Andrew Dunstan <[EMAIL PROTECTED]> writes:
> addlitchar(unescape_single_char(yytext[1]));
> +                                     if 
> (IS_HIGHBIT_SET(literalbuf[literallen]))
> +                                             saw_high_bit = true;

Isn't that array subscript off-by-one?  Probably better to put the test
inside unescape_single_char(), anyway.

Otherwise it looks sane, though maybe shy a comment or so.

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 5: don't forget to increase your free space map settings

Reply via email to