Eliminate the only use of VOIDFLAGS, as part of STMT_START in perl.h
authorNicholas Clark <nick@ccl4.org>
Wed, 11 Sep 2013 09:54:42 +0000 (11:54 +0200)
committerNicholas Clark <nick@ccl4.org>
Mon, 16 Sep 2013 09:58:45 +0000 (11:58 +0200)
commit1400a53b58cffb93a158ce6193542245eaa39b7d
tree820ea8af0a7fb648bd7e2f72a3d5d5b7f6cada71
parentde2d3e8dce1543b8d4baf3f4f33ce6bc4fc8b7d1
Eliminate the only use of VOIDFLAGS, as part of STMT_START in perl.h

STMT_START has used VOIDFLAGS as part of its conditional compilation since it
was added by commit 728e280303ae3d3b (March 1996). The code originally read:

   /* Now which other defined()s do we need here ??? */

Since then it has been amended to avoid entering that definition for GCC
on Solaris. Given that all current Solaris compilers are C89 conformant,
VOIDFLAGS will always be true (actually 15), so the test is redundant.
Even back in 1996, it's possible that VOIDFLAGS was always non-zero on
SunOS, rendering the test obsolete from the start.

Spotted by Brian Fraser, and extracted from a larger patch of his.
perl.h