Reset sequence number on player start

Fix a bug where seeking through a module resulted in a sequence change,
caused by the sequence number not being reset on player start.

Signed-off-by: Claudio Matsuoka <cmatsuoka@gmail.com>
Move IT comment loading to end of loader

Signed-off-by: Claudio Matsuoka <cmatsuoka@gmail.com>
Change file depacker to be non-recursive

Reduce complexity: the recursive file depacker passed a linked list to
stores the temporary file chain to be deleted after the module is loaded,
because Windows can't unlink open files. The non-recursive depacker
simply stores the temporary file name.

Signed-off-by: Claudio Matsuoka <cmatsuoka@gmail.com>
Fix crash in external command line assembly

Signed-off-by: Claudio Matsuoka <cmatsuoka@gmail.com>
Limit note number to prevent crash

Note values that are too high end up crashing the player, so limit them
at 149. See 6nations.it for an example of problematic module. Reported by
Bastian Pflieger.

Signed-off-by: Claudio Matsuoka <cmatsuoka@gmail.com>
Fix stray notes if last instrument is invalid

In FT2, if a note has no instrument and the last specified instrument is
invalid, don't play the note. See last notes of mothershipftw-stroberev2.xm.
Reported by Andreas Argirakis.

Signed-off-by: Claudio Matsuoka <cmatsuoka@gmail.com>
[test] Add test for FT2 new note after invalid ins

Signed-off-by: Claudio Matsuoka <cmatsuoka@gmail.com>
Update changelog

Signed-off-by: Claudio Matsuoka <cmatsuoka@gmail.com>
[test] Remove generated file from git

Signed-off-by: Claudio Matsuoka <cmatsuoka@gmail.com>
Update version number

Signed-off-by: Claudio Matsuoka <cmatsuoka@gmail.com>