From 12acd9db2be614ea7cf9ae0d423a2b69e9c6a285 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Carsten=20Gr=C3=A4ser?= <graeser@dune-project.org>
Date: Wed, 9 Sep 2015 14:05:48 +0200
Subject: [PATCH] [c++11] Add comment on index_sequence
Document an incompatibility issue of index_sequence
and add a reminder to fix this as soon as we allow
template aliases.
This can only be fixed
---
dune/common/std/utility.hh | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/dune/common/std/utility.hh b/dune/common/std/utility.hh
index a2b661f7f..3655caa12 100644
--- a/dune/common/std/utility.hh
+++ b/dune/common/std/utility.hh
@@ -51,6 +51,15 @@ namespace Dune
// index_sequence
// --------------
+ // While this is similar to std::index_sequence, there is one
+ // major problem with the present implementation. Our index_sequence
+ // is only derived from integer_sequence but not equal to one.
+ // As a consequence template specializations that are common
+ // in nontrivial use cases will fail unless we take special
+ // care about index_sequence. The correct implementation is
+ // a template alias. We should switch to the correct implementation
+ // (i.e. a template alias) as soon as we allow this c++11 feature
+ // in dune-common.
/** \brief a function similar to std::index_sequence to be introduced in
* C++14
--
GitLab