On Fri, 9 Apr 2021 at 13:52, Michael Paquier <[email protected]> wrote: > > On Thu, Apr 08, 2021 at 09:21:30PM -0400, Alvaro Herrera wrote: > > On 2021-Apr-08, Tom Lane wrote: > >> Maybe like > >> > >> case T_ScalarArrayOpExpr: > >> /* ScalarArrayOpExpr's result is boolean ... */ > >> coll = InvalidOid; /* ... so it has no collation > >> */ > >> break; > > > > This is much clearer, yeah. > > +1.
Yeah, that's much better.
For the exprSetCollation case, I ended up with:
case T_ScalarArrayOpExpr:
/* ScalarArrayOpExpr's result is boolean ... */
Assert(!OidIsValid(collation)); /* ... so never
set a collation */
I wanted something more like /* ... so we must never set a collation
*/ but that put the line longer than 80. I thought wrapping to a 2nd
line was excessive, so I shortened it to that.
David
improve_possibly_misleading_comments_in_nodefuncs.patch
Description: Binary data
