printf("%lf") is not portable, so omit the "l".
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 20 May 2018 15:40:55 +0000 (11:40 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 20 May 2018 15:40:55 +0000 (11:40 -0400)
commit0aa97b86f9021f2d4a5cd1aab04e0d33794d120b
tree9160623d687c4578987b922e92448f9d8b607696
parent91f3bcc9e423b605046827bba549ac09fbb53db8
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