diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2018-12-07 11:37:24 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2018-12-07 22:46:49 +0100 |
commit | 7d928d8c6eb03c4e5e0d1961e9b62718ab53fb46 (patch) | |
tree | 43e827f3debcf3cfbc2c101c10ed2043bc880d9a /include/rtl/ustring.hxx | |
parent | 74dd206e67c2efb1e56d817be9e42a1ed82e3239 (diff) |
HAVE_CXX14_CONSTEXPR is always true now
...but for safety, leave the configure.ac check in for some longer.
o3tl::array_view::max_size (include/o3tl/array_view.hxx) and
o3tl::basic_string_view::max_size (include/o3tl/string_view.hxx) started to
produce loplugin:staticmethods warnings, which I silenced by /not/ making the
functions static. Those classes are meant to be temporary drop-in replacements
for standard classes (std::span slated for C++20, prev. std::array_view; and
std::basic_string_view, resp.), so should have the same behavior as their
standard counterparts (and making the functions static would likely cause
loplugin:staticaccess warnings at call sites).
Change-Id: If21674dbf02886f453ca447544e37b184df5a25e
Reviewed-on: https://gerrit.libreoffice.org/64768
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'include/rtl/ustring.hxx')
-rw-r--r-- | include/rtl/ustring.hxx | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/include/rtl/ustring.hxx b/include/rtl/ustring.hxx index 7ab872e9d61d..d803a1ad4e6f 100644 --- a/include/rtl/ustring.hxx +++ b/include/rtl/ustring.hxx @@ -33,7 +33,6 @@ #include "rtl/textenc.h" #ifdef LIBO_INTERNAL_ONLY // "RTL_FAST_STRING" -#include "config_global.h" #include "rtl/stringconcat.hxx" #endif @@ -79,10 +78,8 @@ struct SAL_WARN_UNUSED OUStringLiteral data( libreoffice_internal::ConstCharArrayDetector<T>::toPointer(literal)) { -#if HAVE_CXX14_CONSTEXPR assert( libreoffice_internal::ConstCharArrayDetector<T>::isValid(literal)); -#endif } int size; |