From e0ffb9b59be89bc0434ebeacafa2bb0160306423 Mon Sep 17 00:00:00 2001 From: nobu Date: Thu, 19 Apr 2012 07:33:55 +0000 Subject: * strftime.c (rb_strftime_with_timespec): fix carrir-up bug and overwrite '+' with '-' if negative offset less than a hour. [ruby-core:44447][Bug #6323] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@35399 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 6 ++++++ strftime.c | 7 +++++-- test/ruby/test_time.rb | 16 ++++++++++++++++ 3 files changed, 27 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 6d40e8471e..5bdfe046c3 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +Thu Apr 19 16:33:53 2012 Nobuyoshi Nakada + + * strftime.c (rb_strftime_with_timespec): fix carrir-up bug and + overwrite '+' with '-' if negative offset less than a hour. + [ruby-core:44447][Bug #6323] + Thu Apr 19 09:39:57 2012 Nobuyoshi Nakada * ext/-test-/win32/dln/extconf.rb: need import library for ordinal diff --git a/strftime.c b/strftime.c index ea369deaa0..8be95a310b 100644 --- a/strftime.c +++ b/strftime.c @@ -493,9 +493,12 @@ rb_strftime_with_timespec(char *s, size_t maxsize, const char *format, rb_encodi sign = +1; } i = snprintf(s, endp - s, (padding == ' ' ? "%+*ld" : "%+.*ld"), - precision + 2, sign * (off / 360 + 1)); + precision + 1, sign * (off / 3600)); if (i < 0) goto err; - s += i - 1; + if (sign < 0 && off < 3600) { + *(padding == ' ' ? s + i - 2 : s) = '-'; + } + s += i; off = off % 3600; if (1 <= colons) *s++ = ':'; diff --git a/test/ruby/test_time.rb b/test/ruby/test_time.rb index 573b563177..10ee8fe798 100644 --- a/test/ruby/test_time.rb +++ b/test/ruby/test_time.rb @@ -684,6 +684,22 @@ class TestTime < Test::Unit::TestCase assert_equal("-000005:00", t.strftime("%:10z"), bug4458) assert_equal(" -5:00:00", t.strftime("%_::10z"), bug4458) assert_equal("-005:00:00", t.strftime("%::10z"), bug4458) + + bug6323 = '[ruby-core:44447]' + t = T2000.getlocal("+00:36") + assert_equal(" +036", t.strftime("%_10z"), bug6323) + assert_equal("+000000036", t.strftime("%10z"), bug6323) + assert_equal(" +0:36", t.strftime("%_:10z"), bug6323) + assert_equal("+000000:36", t.strftime("%:10z"), bug6323) + assert_equal(" +0:36:00", t.strftime("%_::10z"), bug6323) + assert_equal("+000:36:00", t.strftime("%::10z"), bug6323) + t = T2000.getlocal("-00:55") + assert_equal(" -055", t.strftime("%_10z"), bug6323) + assert_equal("-000000055", t.strftime("%10z"), bug6323) + assert_equal(" -0:55", t.strftime("%_:10z"), bug6323) + assert_equal("-000000:55", t.strftime("%:10z"), bug6323) + assert_equal(" -0:55:00", t.strftime("%_::10z"), bug6323) + assert_equal("-000:55:00", t.strftime("%::10z"), bug6323) end def test_delegate -- cgit v1.2.1