Re: Macros for time magic values

Simon Riggs wrote:
> On Sat, 2011-03-12 at 22:29 +0200, Peter Eisentraut wrote:
> > On fre, 2011-03-11 at 12:50 -0500, Bruce Momjian wrote:
> > > It has bothered me that many of our time routines use special magic
> > > constants for time values, e.g. 24, 12, 60, etc.
> > > 
> > > The attached patch changes these magic constants to macros to clarify
> > > the code.  I would like to apply this for 9.1 as a cleanup.
> > 
> > I think it's much clearer with the plain numbers.
> 
> Yeh. It's not like the values 24, 12 or 60 were going to change.

Well, I find the macro names clearer because it removes domain-specific
knowledge from code that is already complex, e.g '24' represents what?
It is similar to the argument of why use C when you can use assembler. 
I find the macros a pure code clarity win.  If most people don't we
should remove them consistently.

-- 
  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + It's impossible for everything to be true. +