Re: pgsql: to_char(): have format 'OF' only show the leading negative sign

From: Stephen Frost <sfrost(at)snowman(dot)net>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: to_char(): have format 'OF' only show the leading negative sign
Date: 2015-04-29 14:11:02
Message-ID: 20150429141102.GX30322@tamriel.snowman.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Bruce,

* 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?

Thanks!

Stephen

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2015-04-29 16:59:43 pgsql: psql: Improve tab completion for ALTER FOREIGN TABLE.
Previous Message Michael Paquier 2015-04-29 12:01:36 Re: shared_libperl, shared_libpython