>>>>> "Tom" == Tom Lane <t...@sss.pgh.pa.us> writes:

 Tom> Hm. transformRuleStmt already does special-case utility statements
 Tom> to some extent, so my inclination would be to make it do more of
 Tom> that. However, it looks like that might end up with rather
 Tom> spaghetti-ish code, as that function is kind of messy already.

 Tom> Or we could abandon the notion of adding the assertion. I don't
 Tom> know how much work it's worth.

Fixing transformRuleStmt just pushes the issue along another step:
InsertRule wants to do recordDependencyOnExpr on the rule actions,
which just does find_expr_references_walker.

I'm going to leave the assertion out for now and put in a comment for
future reference.

-- 
Andrew (irc:RhodiumToad)


Reply via email to