[libcxx-commits] [PATCH] D117962: [libc++] Fix LWG3422 "Issues of seed_seq's constructors"

Arthur O'Dwyer via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Sun Jan 23 09:42:49 PST 2022


Quuxplusone marked an inline comment as done.
Quuxplusone added a comment.

@ldionne PTAL when you get a chance.



================
Comment at: libcxx/include/__random/seed_seq.h:44
 #ifndef _LIBCPP_CXX03_LANG
-    template<class _Tp>
+    template<class _Tp, __enable_if_t<is_integral<_Tp>::value>* = nullptr>
         _LIBCPP_INLINE_VISIBILITY
----------------
Mordante wrote:
> Please include `__nullptr` to make sure this is available.
Actually this is under `#ifndef _LIBCPP_CXX03_LANG` so it should be fine. (I'm not sure we would `#include <__nullptr>` even if it weren't, but I've raised that tangent over on the Discord.)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117962/new/

https://reviews.llvm.org/D117962



More information about the libcxx-commits mailing list