mirror of
https://git.savannah.gnu.org/git/bison.git
synced 2026-03-09 20:33:03 +00:00
This is based on what is recommended by both Scott Meyers, in 'Effective
C++', and Andrei Alexandrescu and Herb Sutter in 'C++ Coding Standards'.
Use a static_cast on void* rather than directly use a reinterpret_cast,
which can have nefarious effects on objects. However, even though following
this guideline is good practice in general, I am not quite sure how relevant
it is when applied to conversions from POD to objects. Actually, it might
very well be the opposite: isn't this exactly what reinterpret_cast is for?
What we really want *is* to transmit the memory map as a series of bytes,
which, if I am correct, falls into the kind of "low level" hack for which
this cast is meant.
In any case, this silences the warning, which will be greatly appreciated by
anyone using variants with a compiler supporting -fstrict-aliasing.
* data/variant.hh (as): Here.
* tests/c++.at (Exception safety, C++ Variant-based Symbols, Variants):
Don't use NO_STRICT_ALIAS_CXXFLAGS (revert commit ddb9db15), as type punning
is no longer an issue.
* tests/atlocal.in, configure.ac (NO_STRICT_ALIAS_CXXFLAGS): Remove
definition.
* examples/local.mk (NO_STRICT_ALIAS_CXXFLAGS): Remove from AM_CXXFLAGS.
* doc/bison.texi: Don't mention type punning issues.
44 lines
1.5 KiB
Makefile
44 lines
1.5 KiB
Makefile
# Copyright (C) 2005, 2008-2013 Free Software Foundation, Inc.
|
|
#
|
|
# This program is free software: you can redistribute it and/or modify
|
|
# it under the terms of the GNU General Public License as published by
|
|
# the Free Software Foundation, either version 3 of the License, or
|
|
# (at your option) any later version.
|
|
#
|
|
# This program is distributed in the hope that it will be useful,
|
|
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
# GNU General Public License for more details.
|
|
#
|
|
# You should have received a copy of the GNU General Public License
|
|
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
|
|
|
dist_noinst_SCRIPTS = examples/extexi examples/test
|
|
TEST_LOG_COMPILER = $(top_srcdir)/examples/test
|
|
|
|
AM_CXXFLAGS = \
|
|
$(WARN_CXXFLAGS) $(WARN_CXXFLAGS_TEST) $(WERROR_CXXFLAGS)
|
|
|
|
## ------------ ##
|
|
## Extracting. ##
|
|
## ------------ ##
|
|
|
|
doc = $(top_srcdir)/doc/bison.texi
|
|
extexi = $(top_srcdir)/examples/extexi
|
|
extract = VERSION="$(VERSION)" $(PERL) -f $(extexi) $(doc) --
|
|
extracted =
|
|
CLEANFILES += $(extracted) examples/extracted.stamp
|
|
examples/extracted.stamp: $(doc) $(extexi)
|
|
$(AM_V_GEN)rm -f $@ $@.tmp
|
|
$(AM_V_at)touch $@.tmp
|
|
$(AM_V_at)$(extract) $(extracted)
|
|
$(AM_V_at)mv $@.tmp $@
|
|
|
|
$(extracted): examples/extracted.stamp
|
|
@test -f $@ || rm -f examples/extracted.stamp
|
|
@test -f $@ || $(MAKE) $(AM_MAKEFLAGS) examples/extracted.stamp
|
|
|
|
include examples/calc++/local.mk
|
|
include examples/mfcalc/local.mk
|
|
include examples/rpcalc/local.mk
|