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)
commit9c515f77d6cffb6f2576c5e9895cc0f29cc6279c
tree501c3dcccef6325d47142fa13bc71a1afebf3aaa
parent22d22e4bddf65559453506f4090a6cae0f010b96
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