diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2021-06-16 14:04:12 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2021-06-16 18:37:57 +0200 |
| commit | d32d2fd301357df94d1930420b89415e1fe69688 (patch) | |
| tree | d036f36f5013c79c38777c187b5779b6ab639fcb /include/o3tl | |
| parent | c9ba420bfe5c31e67a7555c8f4a16d0bd7954a51 (diff) | |
Move enable_if from dummy template parameter to return type
...and drop the no-longer necessary loplugin:implicitboolconversion exclusion
(which had been necessary to avoid
> [CXX] vcl/source/window/dockingarea.cxx
> In file included from vcl/source/window/dockingarea.cxx:26:
> In file included from vcl/inc/svdata.hxx:26:
> include/o3tl/hash_combine.hxx:14:78: error: implicit conversion (Dependent) from 'bool' to 'std::enable_if_t<(sizeof(N) == 4), bool>' (aka 'typename enable_if<(sizeof(N) == 4), bool>::type') [loplugin:implicitboolconversion]
> template <typename T, typename N, std::enable_if_t<(sizeof(N) == 4), bool> = false>
> ^~~~~
etc.)
Change-Id: If764365213cf29f7dec6fdd6a773dc9894dd15bd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117330
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'include/o3tl')
| -rw-r--r-- | include/o3tl/hash_combine.hxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/include/o3tl/hash_combine.hxx b/include/o3tl/hash_combine.hxx index 17419b3e2c0f..139ee981699c 100644 --- a/include/o3tl/hash_combine.hxx +++ b/include/o3tl/hash_combine.hxx @@ -11,15 +11,15 @@ namespace o3tl { -template <typename T, typename N, std::enable_if_t<(sizeof(N) == 4), bool> = false> -inline void hash_combine(N& nSeed, T const& nValue) +template <typename T, typename N> +inline std::enable_if_t<(sizeof(N) == 4)> hash_combine(N& nSeed, T const& nValue) { static_assert(sizeof(nSeed) == 4); nSeed ^= std::hash<T>{}(nValue) + 0x9E3779B9u + (nSeed << 6) + (nSeed >> 2); } -template <typename T, typename N, std::enable_if_t<(sizeof(N) == 8), bool> = false> -inline void hash_combine(N& nSeed, T const& nValue) +template <typename T, typename N> +inline std::enable_if_t<(sizeof(N) == 8)> hash_combine(N& nSeed, T const& nValue) { static_assert(sizeof(nSeed) == 8); nSeed ^= std::hash<T>{}(nValue) + 0x9E3779B97F4A7C15llu + (nSeed << 12) + (nSeed >> 4); |