Alvaro Herrera <alvhe...@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


Reply via email to