Re: Record comparison compiler warning

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: Record comparison compiler warning
Дата
Msg-id 9700.1383405511@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: Record comparison compiler warning  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Список pgsql-hackers
Alvaro Herrera <alvherre@2ndquadrant.com> writes:
> Stefan Kaltenbrunner wrote:
>> http://www.postgresql.org/message-id/raw/1381949353.78943.YahooMailNeo@web162902.mail.bf1.yahoo.com

> I would blame Bruce's MUA, or surrounding configuration, for this
> problem.  It looks fine in mine, and as far as I can see, Kevin's
> message correctly declares the email to be in Latin-1 quoted-printable
> encoding, which declares A0 to mean non-breaking space.

I agree with Bruce: this patch is broken.  A0 may be a non-breaking space,
but the fact remains that it isn't a space, and since we're talking about
a diff, the whitespace needs to be the same as what it is in the original
file.  (I believe that some of those whitespace runs involve tabs not just
spaces, making the diff even more wrong.)

Admittedly, if you're just eyeballing it, it might look fine.  But try
feeding it to "patch" and you'll find out it ain't.
        regards, tom lane



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Handle LIMIT/OFFSET before select clause (was: Feature request: optimizer improvement)
Следующее
От: Sawada Masahiko
Дата:
Сообщение: Re: Patch for fail-back without fresh backup