PTParser: RhythmSlash bug?

[Simone Tellini]

Hi,

I think there's a small bug in the RhythmSlash class. It defines:

// Single Note String Number Constants
const wxByte RhythmSlash::MIN_SINGLE_NOTE_STRING_NUMBER = 1;
const wxByte RhythmSlash::MAX_SINGLE_NOTE_STRING_NUMBER = 7;

which are used by:

static bool IsValidStringNumber(wxByte stringNumber) {return
((stringNumber >= MIN_SINGLE_NOTE_STRING_NUMBER) && (stringNumber <=
MAX_SINGLE_NOTE_STRING_NUMBER));}

As far as I've seen, though, string numbers are 0-based, thus either
the constants or IsValidStringNumber() should be modified.

--
Simone Tellini
http://www.tellini.org

[Dummy]

Fixed in the latest source.

Thanks Simone.