Re: pgindent && weirdness

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: pgindent && weirdness
Дата
Msg-id 6012.1579041021@sss.pgh.pa.us
обсуждение исходный текст
Ответ на pgindent && weirdness  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Ответы Re: pgindent && weirdness
Re: pgindent && weirdness
Список pgsql-hackers
Alvaro Herrera <alvherre@2ndquadrant.com> writes:
> I just ran pgindent over some patch, and noticed that this hunk ended up
> in my working tree:
 
> -    if (IsA(leftop, Var) && IsA(rightop, Const))
> +    if (IsA(leftop, Var) &&IsA(rightop, Const))

Yeah, it's been doing that for decades.  I think the triggering
factor is the typedef name (Var, here) preceding the &&.

It'd be nice to fix properly, but I've tended to take the path
of least resistance by breaking such lines to avoid the ugliness:

    if (IsA(leftop, Var) &&
        IsA(rightop, Const))

            regards, tom lane



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

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Rearranging ALTER TABLE to avoid multi-operations bugs
Следующее
От: Tom Lane
Дата:
Сообщение: Re: planner support functions: handle GROUP BY estimates ?