On Mon, Aug  6, 2012 at 03:20:18PM -0400, Bruce Momjian wrote:
> On Sat, Aug  4, 2012 at 10:34:14AM -0400, Bruce Momjian wrote:
> > > I am thinking it is too late to apply this for 9.2 because users might
> > > have already tested their applications, though I doubt many are using BC
> > > dates.  Feedback?
> > 
> > There is never just one bug in formatting.c --- the input side was also
> > broken for BC/negative centuries.  The attached patch fixes the input
> > side as well, and shows the old/fixed output for BC centuries.  The test
> > queries were:
> > 
> >     SELECT to_date('-6  4', 'CC YY');
> >     SELECT to_date(' 6  4', 'CC YY');
> >     SELECT to_date('-6 00', 'CC YY');
> >     SELECT to_date(' 6 00', 'CC YY');
> >     SELECT to_date('-6', 'CC');
> >     SELECT to_date(' 6', 'CC');
> > 
> > I believe this is all for 9.3-only.
> 
> OK, I found another bug in our code;  the output of:
> 
>   SELECT to_date(' 6 BC', 'CC BC');
> 
> returned 501BC, instead of the start of the century, 600BC, like SELECT
> to_date('-6', 'CC') does.
> 
> I also allowed negative BC dates to map to AD dates, just like negative
> AD dates map to BC dates.
> 
> Attached is an updated patch and output diff.

Applied.

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

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

-- 
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs

Reply via email to