pgsql: printf("%lf") is not portable, so omit the "l".

Started by Tom Laneabout 8 years ago1 messagescomitters

tgl@sss.pgh.pa.us

about 8 years ago

printf("%lf") is not portable, so omit the "l".

The "l" (ell) width spec means something in the corresponding scanf usage,
but not here. While modern POSIX says that applying "l" to "f" and other
floating format specs is a no-op, SUSv2 says it's undefined. Buildfarm
experience says that some old compilers emit warnings about it, and at
least one old stdio implementation (mingw's "ANSI" option) actually
produces wrong answers and/or crashes.

Discussion: /messages/by-id/21670.1526769114@sss.pgh.pa.us
Discussion: /messages/by-id/c085e1da-0d64-1c15-242d-c921f32e0d5c@dunslane.net

Branch
------
REL_10_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/fa2cfb962c05df730f6bf586614eb39a4b0b5238

Modified Files
--------------
doc/src/sgml/ecpg.sgml | 2 +-
src/interfaces/ecpg/test/compat_informix/sqlda.pgc | 2 +-
src/interfaces/ecpg/test/expected/compat_informix-sqlda.c | 2 +-
src/interfaces/ecpg/test/expected/preproc-outofscope.c | 2 +-
src/interfaces/ecpg/test/expected/sql-sqlda.c | 2 +-
src/interfaces/ecpg/test/preproc/outofscope.pgc | 2 +-
src/interfaces/ecpg/test/sql/sqlda.pgc | 2 +-
7 files changed, 7 insertions(+), 7 deletions(-)