Thread
-
small makeVar refactoring
Peter Eisentraut <peter_e@gmx.net> — 2010-08-21T21:13:23Z
While hacking around, I noticed that a lot of makeVar() calls could be refactored into some convenience functions, to save some redundancy and so that the unusual call patterns stand out better. Useful?
-
Re: small makeVar refactoring
Tom Lane <tgl@sss.pgh.pa.us> — 2010-08-21T21:49:10Z
Peter Eisentraut <peter_e@gmx.net> writes: > While hacking around, I noticed that a lot of makeVar() calls could be > refactored into some convenience functions, to save some redundancy and > so that the unusual call patterns stand out better. Useful? I'm not real thrilled with importing catalog/pg_attribute.h into makefuncs.h; that seems like a lot of namespace pollution for not much return. So -1 to makeVarFromAttribute. The other is okay although I'm not convinced it's useful enough to bother with. regards, tom lane