Function queryByRect returns resultsetmembers with negative classindex

[troppenhagen@…]

The MapScript-PHP method queryByRect returns some ResultCacheMemberObjects 
containing a negative classindex
if there are more than 128 classes defined in the queried layer.
The constant MS_MAXCLASSES in map.h is set to 250 but MapScript seems not to 
take account of it.
According to the description of method getClass the classindex starts with 0.
Using the method getClass with a negative classindex results in a returned NULL 
value.

[dmorissette]

I think this must be happening because classindex is defined as a char in the
resultCacheMemberObj definition in map.h. Everywhere else in the code it is used
as a 32 bits signed int, so what happens is that the range of the char variable
overflows, resulting in negative values:

typedef struct {
  long shapeindex;
  int tileindex;
  char classindex;
} resultCacheMemberObj;

I sugest changing this definition to use "int classindex". Is that okay with you
Steve? I would do that before the 4.8 release (in both 4.8 and 4.9-dev).

[sdlime]

Sounds like the right fix (that's a lot of classes). Do you want to make it or 
should I?

Steve 

[dmorissette]

Doing it now.

[dmorissette]

Fixed in both 4.8.0 and 4.9-dev in CVS

[dmorissette]

Note that Sean pointed out that this change will impact on the format of the
files saved by msSaveQuery(). That should not be a problem unless there are
people using saved queries for longer than a given user's session. What do you
think Steve?

[sdlime]

Sean's right, but as you say the impact is minimal given the duration of query 
files. I can't see them ever being written and then read by different versions 
of the software.

Steve