On 07/13/2015 04:56 PM, Laurent Laborde wrote:
Friendly greetings !
in file src/bin/scripts/print.c line 421 :
need_recordsep = false;
then set to true line 424.
Now i'm pretty sure it's a meaningless "bug" without any consequence (the
commit that introduced it is 15 years old).
There is a lot of (apparently) dead assignment here and there but some
assigment could be used for debugging purpose so ... why not. But this one ?
The code in question looks like this:
for (f = footers; f; f = f->next)
{
if (need_recordsep)
{
print_separator(cont->opt->recordSep, fout);
need_recordsep = false;
}
fputs(f->data, fout);
need_recordsep = true;
}
Hmm. It does kind of make sense. Right after printing the separator, you
don't need to print a separator because you just printed one. But as
soon as you print the field, you need a separator again. It would be
quite understandable without that dead assignment too, and that's
probably how I would've written it in the first place. But since that's
how it is and has been for 15 years, I'm inclined to just leave it so.
- Heikki
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
