Brar Piening <b...@gmx.de> writes:
> On Fri, 13 May 2011 23:34:05 +0200, Brar Piening <b...@gmx.de> wrote:
> [...]
>> I'd appreciate any reviews, tests or comments.
> As the current commitfest is getting more and more active recently i've 
> rechecked my patch and updated it for code drift.
> No real changes.
> See http://www.piening.info/VS2010v7.patch

Please submit your patches on-list, not just by reference to some URL
somewhere.  We have both legal and archival reasons for insisting on
that.

> The huge increase in size is due to running perltidy which also tidied 
> up the new builddoc.pl which seems to have gotten in untidied.

Please do *not* submit a patch that combines actual reviewable changes
with something like that.  It makes the reviewer's life harder and
doesn't have any counterbalancing benefit.  You can either submit a
separate patch that tidies the existing code, or just request that some
committer do it.

                        regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to