Re: Hot Standby, release candidate?
От | Simon Riggs |
---|---|
Тема | Re: Hot Standby, release candidate? |
Дата | |
Msg-id | 1260790525.1955.588.camel@ebony обсуждение исходный текст |
Ответ на | Re: Hot Standby, release candidate? (Robert Haas <robertmhaas@gmail.com>) |
Ответы |
Re: Hot Standby, release candidate?
|
Список | pgsql-hackers |
On Mon, 2009-12-14 at 06:21 -0500, Robert Haas wrote: > On Mon, Dec 14, 2009 at 6:11 AM, Simon Riggs <simon@2ndquadrant.com> wrote: > > On Mon, 2009-12-14 at 11:09 +0100, Magnus Hagander wrote: > >> On Mon, Dec 14, 2009 at 10:54, Heikki Linnakangas > >> <heikki.linnakangas@enterprisedb.com> wrote: > >> > * Please remove any spurious whitespace. "git diff --color" makes them > >> > stand out like a sore thumb, in red. (pgindent will fix them but always > >> > better to fix them before committing, IMO). > >> > >> +1 in general, not particularly for this patch (haven't checked that > >> in this patch). > >> > >> Actually, how about we add that to the page at > >> http://wiki.postgresql.org/wiki/Submitting_a_Patch? > > > > If we can define "spurious whitespace" it would help decide whether > > there is any action to take, and when. > > git defines it as either (1) extra whitespace at the end of a line or > (2) an initial indent that uses spaces followed by tabs (typically > something like space-tab, where tab alone would have produced the same > result). git diff --check master tends to be useful here. (2) is a problem that has been discussed before on hackers, anything like that should be changed. Why is (1) important, and if it is important, why is it being mentioned only now? Are we saying that all previous reviewers of my work (and others') removed these without ever mentioning they had done so? -- Simon Riggs www.2ndQuadrant.com
В списке pgsql-hackers по дате отправления: