common/classes.cpp
 

shouldn't it be
int8 GetEQArrayEQClass(int8 eqclass) {
if (eqclass >= WARRIOR && eqclass <= BERSERKER)
return eqclass - WARRIOR;
if (eqclass >= WARRIORGM && eqclass <= BERSERKERGM)
return eqclass - WARRIORGM;
return WARRIOR;
}
at line 332

instead of
int8 GetEQArrayEQClass(int8 eqclass) {
if (eqclass >= WARRIOR && eqclass <= BEASTLORD)
return eqclass - WARRIOR;
if (eqclass >= WARRIORGM && eqclass <= BEASTLORDGM)
return eqclass - WARRIORGM;
return WARRIOR;
}

hmm... looks fine, will make the change. thats only used for spell casting, and it looks like the spell loading code was already updated to handle them, so we'll see how it goes. Items on the other hand store the "berserker allowed" bit in a different way (seperate field), in case you wander over there too.