From: Jesse F. Williamson Date: Mon, 3 Aug 2026 14:47:47 +0000 (-0700) Subject: Fixes compilation error with some versions of libstdc++ X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=e3943b58266cd04f4f23d3a0c5f473aff3d0ac9a;p=ceph.git Fixes compilation error with some versions of libstdc++ Some versions of GCC don't expose append_range()/insert_range(), but when they did, this was b0rked. This patch checks no_range_insertions(). I introduced this compilation error in: 9b3679cc94d853b534cd97d3573e0045d3611dd2 Oops... :-/ Assisted-by: Codex:GPT-5 Signed-off-by: Jesse F. Williamson --- diff --git a/src/test/test_concepts.cc b/src/test/test_concepts.cc index c601fa240609..72aa97a6555d 100644 --- a/src/test/test_concepts.cc +++ b/src/test/test_concepts.cc @@ -92,6 +92,10 @@ struct insert_range_only final : vector_backed_range { } }; +struct no_range_insertions final : vector_backed_range { + using vector_backed_range::vector_backed_range; +}; + // Only exposes erase(I), to help verify can_remove_front() and pop_front() // can work without pop_front() directly being available: struct front_erasable final : vector_backed_range { @@ -532,13 +536,13 @@ TEST_CASE("optional sizing helpers call supported operations only", X(insert_after, (ceph::concepts::has_insert_after), \ (ceph::concepts::has_insert_after)) \ X(append_range, (ceph::concepts::has_append_range), \ - (ceph::concepts::has_append_range)) \ + (ceph::concepts::has_append_range)) \ X(insert_range_append, (ceph::concepts::has_insert_range_append), \ - (ceph::concepts::has_insert_range_append)) \ + (ceph::concepts::has_insert_range_append)) \ X(insert_iterator_range_append, (ceph::concepts::has_insert_iterator_range_append), \ (ceph::concepts::has_insert_iterator_range_append)) \ X(insert_range, (ceph::concepts::has_insert_range), \ - (ceph::concepts::has_insert_range)) \ + (ceph::concepts::has_insert_range)) \ X(insert_iterator_range, (ceph::concepts::has_insert_iterator_range), \ (ceph::concepts::has_insert_iterator_range)) \ X(emplace_front, (ceph::concepts::has_emplace_front), \