Michael Paesold wrote
> Tom Lane wrote:
> > Per SQL spec, the backend thinks that /* .. */ nests:
> >
> > regression=# /* some /* comment */ comment */ select 1;
> >  ?column?
> > ----------
> >         1
> > (1 row)
> >
> > As it stands, is_transact_command will get confused by this.

It is now fixed in the attached patch.

> > I'd split out the code that locates "the next keyword" into a separate
> > function that could be called twice.
>
> I have read this mail after making the attached patch, so I have used a
> little different approach. I have moved the code to skip over whitespace
and
> comments into it's own function.

Well, reading this again after some sleep, we probably meant the same.

Concerning my other patch that also touches psql/common.c (Rollback of only
last statement), have you looked into it?

Best Regards,
Michael Paesold

Attachment: psql-autocommit.diff
Description: Binary data

---------------------------(end of broadcast)---------------------------
TIP 2: you can get off all lists at once with the unregister command
    (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED])

Reply via email to