[wesnoth-commits] [wesnoth/wesnoth] 7ba24d: Move ms_after= to the right place in music macros ...

[Jyrki Vesterinen]

  Branch: refs/heads/1.14
  Home:   https://github.com/wesnoth/wesnoth
  Commit: 7ba24d60d8fbe1beb712c102a85ca43be2ebf392
      https://github.com/wesnoth/wesnoth/commit/7ba24d60d8fbe1beb712c102a85ca43be2ebf392
  Author: Jyrki Vesterinen <sandgtx at gmail.com>
  Date:   2019-10-13 (Sun, 13 Oct 2019)

  Changed paths:
    M data/core/macros/sound-utils.cfg

  Log Message:
  -----------
  Move ms_after= to the right place in music macros (#4459)

I originally added it in #719. At that point ms_after= affected the *previous*
track due to a bug; however, I had assumed the behavior to be intentional and
compensated for it by adding ms_after= to the first track of a scenario instead
of the story screen music.

In April, I saw the documentation for ms_after= which said that it's supposed
to affect the track itself, not the previous one. Hence, I fixed the bug in
commit b023b818c742cee0bc75460235c9d1088c0b495d. However, I had forgotten that the changes I had made
in #719 depended on the old broken behavior of ms_after=.

This commit moves ms_after= to the right place, restoring the fade-out on
scenario start and removing the pointless pause after the first track.


  Commit: b0b467dfefae77371add962de89d5f72f51c12af
      https://github.com/wesnoth/wesnoth/commit/b0b467dfefae77371add962de89d5f72f51c12af
  Author: Jyrki Vesterinen <sandgtx at gmail.com>
  Date:   2019-10-13 (Sun, 13 Oct 2019)

  Changed paths:
    M changelog.md

  Log Message:
  -----------
  Changelog entry for commit 7ba24d60d8fbe1beb712c102a85ca43be2ebf392


Compare: https://github.com/wesnoth/wesnoth/compare/81bef7f68997...b0b467dfefae