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
---------------------------(end of broadcast)--------------------------- TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]
