Andreas Pflug wrote:
> Christopher Kings-Lynne wrote:
>
>>>> need_paren = (PRETTY_PAREN(context) ?
>>>> !IsA(op->rarg, RangeTblRef) : true);
>>>
>>>
>>>
>>>
>>> In a quick glance this code seems close to completely brain dead :-(
>>
This probably was about catching
expr_A UNION (expr_B INTERSECT expr_C)
cases, falsely assuming left-to-right won't ever need parentheses.
Apparently the current version already fixes this completely,
suppressing parentheses also with non-pretty.
Regards,
Andreas
