printf("%lf") is not portable, so omit the "l".
authorTom Lane
Sun, 20 May 2018 15:40:54 +0000 (11:40 -0400)
committerTom Lane
Sun, 20 May 2018 15:40:54 +0000 (11:40 -0400)
commite52cabff7054005d2f7157c236d996dcd79baf5c
tree2b8c43f314159087595c36bf10e62403b03e40fb
parent8109f201da7745477519e0e151b900a6aeca6e69
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: https://postgr.es/m/21670.1526769114@sss.pgh.pa.us
Discussion: https://postgr.es/m/c085e1da-0d64-1c15-242d-c921f32e0d5c@dunslane.net
doc/src/sgml/ecpg.sgml
src/interfaces/ecpg/test/compat_informix/sqlda.pgc
src/interfaces/ecpg/test/expected/compat_informix-sqlda.c
src/interfaces/ecpg/test/expected/preproc-outofscope.c
src/interfaces/ecpg/test/expected/sql-sqlda.c
src/interfaces/ecpg/test/preproc/outofscope.pgc
src/interfaces/ecpg/test/sql/sqlda.pgc