Jeff Trawick wrote: >Jeff Trawick writes: > > > >>brianp@apache.org writes: >> >> >> >>>brianp 02/05/23 18:07:47 >>> >>> Modified: strmatch apr_strmatch.c >>> Log: >>> Switched to unsigned chars in the searching functions to avoid >>> problems when input strings contain character values > 127 >>> Reported by: Cliff Woolley >>> >>> >>ouch, lots of compile errors with native compiler on AIX (invalid >>operations between pointers to char of different signed-ness) >> >>I'll start playing with this in a minute... >> >> > >I guess this patch is appropriate unless we want to change the API. >Darn signed chars :) > There's only one operation in the whole algorithm that really needs an unsigned char. I'll switch back to signed chars, with a cast to unsigned only in the case where the char value is used as an array index. --Brian