weaselab/conflict-set

Disallow copying ConflictSet in C++98/C++03 #51

Merged

andrew merged 2 commits from weaselbot/conflict-set:weaselbot/issue-48 into main

2026-06-22 23:45:01 +00:00

Author	SHA1	Message	Date
andrew	ccd637deab	We already know __cplusplus is defined CI / release (arm64, ubuntu-latest-arm64) (pull_request) Successful in 3m21s Details CI / pre-commit (pull_request) Successful in 2m6s Details CI / test (-DCMAKE_BUILD_TYPE=Debug, debug) (pull_request) Successful in 3m35s Details CI / test (-DCMAKE_CXX_FLAGS=-DUSE_64_BIT=1, 64-bit-versions) (pull_request) Successful in 3m27s Details CI / test (-DCMAKE_C_COMPILER=gcc -DCMAKE_CXX_COMPILER=g++, gcc) (pull_request) Successful in 3m39s Details CI / test (-DUSE_SIMD_FALLBACK=ON, simd-fallback) (pull_request) Successful in 3m34s Details CI / release (amd64, ubuntu-latest-amd64) (pull_request) Successful in 4m53s Details CI / coverage (pull_request) Successful in 3m36s Details	2026-06-22 14:18:33 -04:00
weaselbot	7eaac2a184	Make ConflictSet non-copyable in C++98/C++03 CI / release (arm64, ubuntu-latest-arm64) (pull_request) Successful in 3m20s Details CI / pre-commit (pull_request) Successful in 2m5s Details CI / test (-DCMAKE_BUILD_TYPE=Debug, debug) (pull_request) Successful in 3m37s Details CI / test (-DCMAKE_CXX_FLAGS=-DUSE_64_BIT=1, 64-bit-versions) (pull_request) Successful in 3m27s Details CI / test (-DCMAKE_C_COMPILER=gcc -DCMAKE_CXX_COMPILER=g++, gcc) (pull_request) Successful in 3m37s Details CI / test (-DUSE_SIMD_FALLBACK=ON, simd-fallback) (pull_request) Successful in 3m27s Details CI / release (amd64, ubuntu-latest-amd64) (pull_request) Successful in 4m52s Details CI / coverage (pull_request) Successful in 3m37s Details `ConflictSet(const ConflictSet&)` and `operator=(const ConflictSet&)` were only deleted for C++11 and later. In C++98/C++03 the compiler implicitly generated public copy operations, so copying a ConflictSet shared the opaque `Impl*` and caused a double-free on destruction. Declare both operations private and leave them undefined when `__cplusplus <= 199711L`, matching the standard pre-C++11 idiom for move-only types. Guard the declarations with `defined(__cplusplus)` so they are not exposed to C90 compilation units. Closes #48	2026-06-22 13:43:04 -04:00