From: | Stephen Frost <sfrost(at)snowman(dot)net> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Bruce Momjian <bruce(at)momjian(dot)us>, pgsql-committers <pgsql-committers(at)postgresql(dot)org> |
Subject: | Re: Re: pgsql: to_char(): have format 'OF' only show the leading negative sign |
Date: | 2015-04-29 18:22:22 |
Message-ID: | 20150429182222.GB30322@tamriel.snowman.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
* Robert Haas (robertmhaas(at)gmail(dot)com) wrote:
> On Wed, Apr 29, 2015 at 10:11 AM, Stephen Frost <sfrost(at)snowman(dot)net> wrote:
> > * Bruce Momjian (bruce(at)momjian(dot)us) wrote:
> >> to_char(): have format 'OF' only show the leading negative sign
> >>
> >> Previously both hours and minutes displayed as negative.
> >>
> >> Report by David Pozsar
> >
> > This is causing the following error for me:
> >
> > /home/sfrost/git/pg/dev/postgresql/src/backend/utils/adt/formatting.c: In function ‘DCH_to_char’:
> > /home/sfrost/git/pg/dev/postgresql/src/backend/utils/adt/formatting.c:2510:6: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 3 has type ‘int’ [-Wformat=]
> > sprintf(s, ":%02ld", abs(tm->tm_gmtoff % SECS_PER_HOUR) / SECS_PER_MINUTE);
> > ^
> >
> > Since abs() is declared to return just an int. I don't see it anywhere
> > in our tree and I'm not sure how portable it actually is, but labs() is
> > supposedly in C89, so perhaps that should be used here instead?
>
> I just removed the "l" for now, since %02d is used to print the result
> of abs() elsewhere. If that isn't right, someone can re-fix my fix.
Works for me. I'm no longer seeing the warning.
Thanks!
Stephen
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2015-04-29 18:30:26 | Re: [COMMITTERS] pgsql: Add transforms feature |
Previous Message | Robert Haas | 2015-04-29 18:17:24 | Re: Re: pgsql: to_char(): have format 'OF' only show the leading negative sign |