The old code was mixing sal_uInt16 & sal_uLong all over the place. Plus,
MenuItemList is now a std::vector<> which uses size_t as index variables.

Note: ImplSelectWithStart() was listed as unused; however, it's used on
Mac OS X. I added a comment stating that we use it, so hopefully someone
else doesn't remove it later without checking first.

I also removed GetCharClass() since it wasn't used and it was in the
unusedcode.easy list.

I don't see any changes in the menus (Mac OS X).