aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWladimir J. van der Laan <laanwj@protonmail.com>2020-06-29 16:50:54 +0200
committerWladimir J. van der Laan <laanwj@protonmail.com>2020-06-29 16:50:59 +0200
commitdbadf746e26840b68cb3b01a690aa46256ae2191 (patch)
treefbb93e061bc83997f94f3509cdfc82ad5c8d0c74
parent1269cab21a9ac32f9ee8adfb9041950756eff3e4 (diff)
parentfa3b35a189c4a4fd9667ef0af1c7059471ac8b01 (diff)
Merge #19333: refactor: Fix clang compile failure
fa3b35a189c4a4fd9667ef0af1c7059471ac8b01 ci: Add test for clang-3.8 C++11 support (MarcoFalke) faa7431fee45b26f7ac2f5fd0b8874cb6afafbd4 refactor: Fix clang compile failure (MarcoFalke) Pull request description: Fix ``` script/standard.cpp:278:22: error: default initialization of an object of const type 'const (anonymous namespace)::CScriptVisitor' without a user-provided default constructor const CScriptVisitor g_script_visitor; ^ {} 1 error generated. ACKs for top commit: laanwj: ACK fa3b35a189c4a4fd9667ef0af1c7059471ac8b01 Tree-SHA512: b3251208945b44530224aadbc10fef1260b479c0b43a5e345501fbfd3579a9fe354b946090e023232852bbb99759da4429b58b137b7b286ddac6bd7960851f7f
-rw-r--r--ci/test/00_setup_env_native_nowallet.sh4
-rw-r--r--src/script/standard.cpp5
2 files changed, 3 insertions, 6 deletions
diff --git a/ci/test/00_setup_env_native_nowallet.sh b/ci/test/00_setup_env_native_nowallet.sh
index 553dab1491..1a0b14b62b 100644
--- a/ci/test/00_setup_env_native_nowallet.sh
+++ b/ci/test/00_setup_env_native_nowallet.sh
@@ -8,7 +8,7 @@ export LC_ALL=C.UTF-8
export CONTAINER_NAME=ci_native_nowallet
export DOCKER_NAME_TAG=ubuntu:16.04 # Use xenial to have one config run the tests in python3.5, see doc/dependencies.md
-export PACKAGES="python3-zmq"
+export PACKAGES="python3-zmq clang-3.8 llvm-3.8" # Use clang-3.8 to test C++11 compatibility, see doc/dependencies.md
export DEP_OPTS="NO_WALLET=1"
export GOAL="install"
-export BITCOIN_CONFIG="--enable-glibc-back-compat --enable-reduce-exports"
+export BITCOIN_CONFIG="--enable-glibc-back-compat --enable-reduce-exports CC=clang-3.8 CXX=clang++-3.8"
diff --git a/src/script/standard.cpp b/src/script/standard.cpp
index 39dd4ff39f..251be8d271 100644
--- a/src/script/standard.cpp
+++ b/src/script/standard.cpp
@@ -290,14 +290,11 @@ public:
return CScript() << CScript::EncodeOP_N(id.version) << std::vector<unsigned char>(id.program, id.program + id.length);
}
};
-
-const CScriptVisitor g_script_visitor;
-
} // namespace
CScript GetScriptForDestination(const CTxDestination& dest)
{
- return boost::apply_visitor(::g_script_visitor, dest);
+ return boost::apply_visitor(CScriptVisitor{}, dest);
}
CScript GetScriptForRawPubKey(const CPubKey& pubKey)