#68633 Avoid duplicates in speed adjustments

[alef]

Summary

Bugfixes "Avoid duplicates in speed adjustments"

Purpose of change

Creature::effects can be altered by the debug menu, but Character::mod_speed_bonus is altered during a turn, so in the future. This is probably what created confusion, as by the note comment in previous code.

Describe the solution

I inverted the two loops so that I first fill the std::vector with the std::map.
Then I append speed_bonus_effects but using the std::map to avoid duplicates.

Describe alternatives you've considered

Testing

Only on Windows 10 SDL, under VS debugger. Using Debug menu to add frenzy effect, step through code and observe.

Additional context