[libcxx-commits] [libcxx] [libcxx] Removal of narrowing conversions in `flat_[multi]{set, map}` tests for compatibility with MSVC (PR #170909)

[via libcxx-commits]

=?utf-8?q?Vojt=C4=9Bch?= Michal <vojta.michall at email.cz>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/170909 at github.com>


================
@@ -57,7 +57,7 @@ constexpr void test_one() {
   i                            = m.begin(); // move-assignment
   typename M::const_iterator k = i;         // converting constructor
   assert(i == k);                           // comparison
-  for (int j = 0; j < 9; ++j, ++i) {        // pre-increment
+  for (int j = 0; j < 9; ++j, (void)++i) {  // pre-increment
----------------
huixie90 wrote:

why is this needed?

https://github.com/llvm/llvm-project/pull/170909