Format code and add tests\GH_004597_self_swap into test.lst

microsoft · May 14, 2024 · fe0d85e · fe0d85e
1 parent 706bb57
commit fe0d85e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/tests/std/test.lst b/tests/std/test.lst
@@ -243,6 +243,7 @@ tests\GH_004201_chrono_formatter
 tests\GH_004275_seeking_fancy_iterators
 tests\GH_004388_unordered_meow_operator_equal
 tests\GH_004477_mdspan_warning_5246
+tests\GH_004597_self_swap
 tests\GH_004618_mixed_operator_usage_keeps_statistical_properties
 tests\GH_004618_normal_distribution_avoids_resets
 tests\LWG2381_num_get_floating_point

diff --git a/tests/std/tests/GH_004597_self_swap/test.cpp b/tests/std/tests/GH_004597_self_swap/test.cpp
@@ -4,7 +4,6 @@
 #include <cassert>
 #include <utility>
 
-// Test GH-4597 "<utility>: Side effects in self-swaps of pair are skipped"
 struct swap_counter {
  unsigned int* pcnt_ = nullptr;
 
@@ -66,15 +65,13 @@ _CONSTEXPR20 bool test_gh_4595() {
  p1.swap(p2);
  return (c1 == 1u) && (c2 == 3u) && (p1.first == 3u) && (p1.second == 3) && (p2.first == 1u) && (p2.second == 1);
  }();
-
-#endif
+#endif // _HAS_CXX23
  return res1 && res2 && res3 && res4;
 }
 
 int main() {
 #if _HAS_CXX20
  static_assert(test_gh_4595());
-#else
+#endif // _HAS_CXX20
  assert(test_gh_4595());
-#endif
 }