Skip to content

Commit

Permalink
fix MSVC warnings and syntax errors when 32bit compiling recent new f…
Browse files Browse the repository at this point in the history
…iles
  • Loading branch information
hurchalla committed May 29, 2024
1 parent 2d0f4e3 commit c13ac53
Show file tree
Hide file tree
Showing 7 changed files with 102 additions and 75 deletions.
11 changes: 6 additions & 5 deletions include/hurchalla/util/BitpackedUintVector.h
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,8 @@ struct BitpackedUintVector
static_assert(!std::numeric_limits<U>::is_signed, "");
static_assert(element_bitlen <= std::numeric_limits<U>::digits, "");
static_assert(0 < element_bitlen && element_bitlen <= 32, "");
using size_type =
typename ImplBitpackedUintVector<U, element_bitlen>::size_type;
using size_type = typename
detail::ImplBitpackedUintVector<U, element_bitlen>::size_type;

BitpackedUintVector(size_type count) : impl_buv(count) {}

Expand Down Expand Up @@ -58,7 +58,8 @@ struct BitpackedUintVector
// returns the maximum value that fits within element_bitlen bits.
HURCHALLA_FORCE_INLINE static constexpr U max_allowed_value()
{
return ImplBitpackedUintVector<U, element_bitlen>::max_allowed_value();
return detail::
ImplBitpackedUintVector<U, element_bitlen>::max_allowed_value();
}

HURCHALLA_FORCE_INLINE std::size_t dataSizeBytes() const
Expand All @@ -78,11 +79,11 @@ struct BitpackedUintVector
// data (for the BitpackedUintVector constructor).
HURCHALLA_FORCE_INLINE static constexpr uint64_t getFormatID()
{
return ImplBitpackedUintVector<U, element_bitlen>::getFormatID();
return detail::ImplBitpackedUintVector<U,element_bitlen>::getFormatID();
}

private:
ImplBitpackedUintVector<U, element_bitlen> impl_buv;
detail::ImplBitpackedUintVector<U, element_bitlen> impl_buv;
};


Expand Down
1 change: 1 addition & 0 deletions include/hurchalla/util/Unroll.h
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@

#include "hurchalla/util/compiler_macros.h"
#include <cstdint>
#include <cstddef>

namespace hurchalla {

Expand Down
Loading

0 comments on commit c13ac53

Please sign in to comment.