stefandrissen / SAM-MOD-player

fix:

Lines 24 to 25 in 9a47003

    
           ; soundtracker 2.2: 
        
           ; - samples: 31

Hello,

Edit: Soundtracker 2.3 should look like this:

 ; soundtracker 2.3:
 ; - samples: 31
 ; - id: M.K.

As the M.K. signature (Michael Kleps AKA Unknown of DOC) signs that it is a module with 31 samples.

Thanks! I used to rely just on the M.K. signature, but some tracker history doc sent me down this rabbit hole that 2.2 could have 31 samples without an M.K. signature...

How would it determine which type it loads? If 2.2 would be able to handle the 31 sampled modules without the M.K. signature, then it would mean, that it can only handle 31 sampled modules. But if you check the images on pouet.net:

https://content.pouet.net/files/screenshots/00048/00048007.jpg
https://content.pouet.net/files/screenshots/00034/00034741.png

you can see, that 2.2 has one digit for samples and 2.3 has two.

As stated, I was led down a rabbit hole by some MOD format history document claiming 2.2 supported 31 samples without M.K.
Based on your comment, this claim seems false

My player performs sanity checks on the file contents, the easiest one being the presence of a signature like M.K. / FLT4 / M!K!.
If there is no signature, the sample headers are checked for sanity.
If there are more than 15 valid sample headers, I classed it as 2.2.

Thankfully this is nonsense.

I see. Sanity check can work, but i doubt Soundtracker 2.2 (or ANY Sound/Noise/ProTracker) did that. :)

Soundtracker 2.3 was first with 31 samples, not 2.2