Re: Bug with view definitions?

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