#ifdef __OSMAC__
i = findOperatorIndex(target,0,MAXNAMES-1);
#else
i = findOperatorIndex(target,0,MAXNAMES);
#endif
Just found this code. findOperatorIndex() is a function that performs a binary search of a static global array for a given string. It takes three parameters: 1) the string to search for, 2) the first array index, and 3) the last array index. The array is zero-indexed and contains MAXNAMES elements, and neither the contents nor the length of the array differ between operating systems. Passing MAXNAMES leads to the code walking off the end of the array in certain cases.
As a second WTF, there's no reason why findOperatorIndex() needs the second and third parameters: the function isn't recursive, the length of the array it's searching is globally known, and searching subsets of the array makes no sense.
As a bonus, this code has been shipping for at least a decade and a half.