Commit 1cd7128b by Dino Radakovic Committed by Copybara-Service

`layout`: Delete outdated comments about ElementType alias not being used because of MSVC

Code below those comments does use ElementType.

PiperOrigin-RevId: 617854731
Change-Id: I7996b8cbaa2fb65855a801f634a57d821408b1f3
parent e4c00cc6
...@@ -542,9 +542,6 @@ class LayoutImpl< ...@@ -542,9 +542,6 @@ class LayoutImpl<
// std::tie(ints, doubles) = x.Pointers(p); // std::tie(ints, doubles) = x.Pointers(p);
// //
// Requires: `p` is aligned to `Alignment()`. // Requires: `p` is aligned to `Alignment()`.
//
// Note: We're not using ElementType alias here because it does not compile
// under MSVC.
template <class Char> template <class Char>
auto Pointers(Char* p) const { auto Pointers(Char* p) const {
return std::tuple<CopyConst<Char, ElementType<OffsetSeq>>*...>( return std::tuple<CopyConst<Char, ElementType<OffsetSeq>>*...>(
...@@ -599,9 +596,6 @@ class LayoutImpl< ...@@ -599,9 +596,6 @@ class LayoutImpl<
// //
// Requires: `p` is aligned to `Alignment()`. // Requires: `p` is aligned to `Alignment()`.
// //
// Note: We're not using ElementType alias here because it does not compile
// under MSVC.
//
// Note: We mark the parameter as unused because GCC detects it is not used // Note: We mark the parameter as unused because GCC detects it is not used
// when `SizeSeq` is empty [-Werror=unused-but-set-parameter]. // when `SizeSeq` is empty [-Werror=unused-but-set-parameter].
template <class Char> template <class Char>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment