Wouldn't it make more sense to simply do this here:
```c++
int moraleMalus = -50 * ((float) kill_count / maxKills);
```
to get a more linear progression? You could do the same thing for the duration a few lines below this with:
```c++
int duration = 300 * ((float) kill_count / maxKills);
```