Re: pgsql: Small stylistic improvement in recent FORCE QUOTE * code - use a

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Small stylistic improvement in recent FORCE QUOTE * code - use a
Date: 2009-07-25 17:07:51
Message-ID: 12355.1248541671@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

adunstan(at)postgresql(dot)org (Andrew Dunstan) writes:
> Log Message:
> -----------
> Small stylistic improvement in recent FORCE QUOTE * code - use a bool instead of a magic value.

This actually broke things a bit. I agree that the original coding with
a phony list header was horridly ugly, but it did have one redeeming
value: namely that the various existing tests for force_quote not empty
still worked as-is. With the separate bool you need to extend all those
checks. On balance it still seems cleaner that way, though, so I made
the necessary fixes.

regards, tom lane

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andrew Dunstan 2009-07-25 17:21:04 Re: pgsql: Small stylistic improvement in recent FORCE QUOTE * code - use a
Previous Message Tom Lane 2009-07-25 17:04:20 pgsql: Code review for FORCE QUOTE * patch: fix error checking to