Re: A couple comments about datetime

Started by Thomas Lockhartover 26 years ago2 messages
#1Thomas Lockhart
lockhart@alumni.caltech.edu

In both datetime_trunc() and timespan_trunc() in dt.c,
the DTK_MICROSEC case is just like the DTK_MILLISEC case.
I think this is wrong and it ought to look like
fsec = rint(fsec * 1000000) / 1000000;
no?

Tom, I looked at this and your fix is the right thing. I am leaving
for a week of vacation, and don't have time to apply the fix. If you
would like to, be my guest :)

TIA

- Thomas

--
Thomas Lockhart lockhart@alumni.caltech.edu
South Pasadena, California

#2Bruce Momjian
maillist@candle.pha.pa.us
In reply to: Thomas Lockhart (#1)
Re: [HACKERS] Re: A couple comments about datetime

In both datetime_trunc() and timespan_trunc() in dt.c,
the DTK_MICROSEC case is just like the DTK_MILLISEC case.
I think this is wrong and it ought to look like
fsec = rint(fsec * 1000000) / 1000000;
no?

Tom, I looked at this and your fix is the right thing. I am leaving
for a week of vacation, and don't have time to apply the fix. If you
would like to, be my guest :)

Applied.

-- 
  Bruce Momjian                        |  http://www.op.net/~candle
  maillist@candle.pha.pa.us            |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026