Cast expression to void to sidestep a warning

The expression in question is clearly intentionally unused.

dune/solvers/common/algorithm.hh

@@ -243,7 +243,9 @@ namespace Imp {
   template<class Range, class F, class Index, Index... i>
   constexpr void forEachIndex(Range&& range, F&& f, std::integer_sequence<Index, i...>)
   {
-    std::initializer_list<int>{(f(Hybrid::elementAt(range, std::integral_constant<Index,i>())), 0)...};
+    (void)std::initializer_list<int>{
+      ((void)f(Hybrid::elementAt(range, std::integral_constant<Index,i>())), 0)...
+    };
   }
 
   template<class Range, class F,

[graeser]

I understand the cast of the unused result. But is the cast of the argument really necessary?

[graeser]

My clang-3.8 is happy if we only do (void) std::initializer_list<int>....

[pipping]

I didn't test that. I found it easier to understand the code this way. But those lines are gone now anyway, right?