> I took a quick look at the patch and this part seems a bit dodgy:
>
> + while (*p && !isspace(*p) && *str != ',')
> + p++;
>
> Shouldn't that *str actually be *p?
Ack, indeed you are right. I originally modified str directly then later
decided to introduce a new variable but forgot to change that :)

Dominick Meglio
Received on 2004-02-23