Fix non-C89-compatible coding in pgbench.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 19 May 2014 04:06:26 +0000 (00:06 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 19 May 2014 04:06:49 +0000 (00:06 -0400)
commite41cc470bb57a0f3a765fa44b77ef104b5a5a01e
tree26c61813a04f53393cb0ed41527495c543fe072b
parent4d30d4b9d725211859c63a18e458015636d24b94
Fix non-C89-compatible coding in pgbench.

C89 says that compound initializers may only contain constant expressions;
a restriction violated by commit 89d00cbe.  While we've had no actual field
complaints about this, C89 is still the project standard, and it's not
saving all that much code to break compatibility here.  So let's adhere to
the old restriction.

In passing, replace a bunch of hardwired constants "256" with
sizeof(target-variable), just because the latter is more readable and
less breakable.  And const-ify where possible.

Back-patch to 9.3 where the nonportable code was added.

Andres Freund and Tom Lane
contrib/pgbench/pgbench.c