Noah Misch <noah@leadboat.com> writes:
> On Fri, Mar 23, 2012 at 11:31:54AM -0400, Tom Lane wrote:
>> ... I've not looked
>> yet at the existing transform functions, but why would they want to know
>> about the original node at all?
> You suggested[1] passing an Expr instead of an argument list, and your reasons
> still seem good to me. That said, perhaps we should send both the original
> Expr and the simplified argument list. That will help if we ever want to
> fully simplify x - y * 0. (Then again, the feature is undocumented and we
> could change it when that day comes.)
I believe what I had in mind back then was that we'd build a new FuncExpr
containing the simplified argument list. On reflection that's probably
the most future-proof way to do it, since otherwise anytime we change
the contents of FuncExpr, we'll be faced with possibly having to change
the signature of protransform functions.
Will go see what I can do with that.
regards, tom lane
