Skip to content

Commit

Permalink
More build fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
laurynas-biveinis committed Feb 4, 2025
1 parent 8350830 commit f2fc621
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
6 changes: 3 additions & 3 deletions mutex_art.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ class mutex_db final {
/// each visited leaf. The tree remains locked for the duration of
/// the scan.
///
/// @param fromKey is an inclusive lower bound for the starting
/// @param from_key is an inclusive lower bound for the starting
/// point of the scan.
///
/// @param fn A function
Expand All @@ -176,10 +176,10 @@ class mutex_db final {
/// forward order. Otherwise the scan will visit all index entries
/// in the half-open range (fromKey,toKey] in reverse order.
///
/// @param fromKey is an inclusive bound for the starting point of
/// @param from_key is an inclusive bound for the starting point of
/// the scan.
///
/// @param toKey is an exclusive bound for the ending point of the
/// @param to_key is an exclusive bound for the ending point of the
/// scan.
///
/// @param fn A function
Expand Down
3 changes: 2 additions & 1 deletion portability_builtins.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,8 @@ namespace unodb::detail {
template <typename T>
T bswap(T x) {
#ifdef UNODB_DETAIL_MSVC
static_assert(sizeof(std::uint32_t) == sizeof(unsigned long));
static_assert(sizeof(std::uint32_t) ==

Check notice on line 33 in portability_builtins.hpp

View check run for this annotation

codefactor.io / CodeFactor

portability_builtins.hpp#L33

Use int16/int64/etc, rather than the C type short. (runtime/int)
sizeof(unsigned long)); // NOLINT(runtime/int)

Check notice on line 34 in portability_builtins.hpp

View check run for this annotation

codefactor.io / CodeFactor

portability_builtins.hpp#L34

Use int16/int64/etc, rather than the C type long. (runtime/int)
static_assert(std::is_same_v<unsigned short, T> || // NOLINT(runtime/int)
std::is_same_v<std::uint32_t, T> ||

Check notice on line 36 in portability_builtins.hpp

View check run for this annotation

codefactor.io / CodeFactor

portability_builtins.hpp#L36

Use int16/int64/etc, rather than the C type short. (runtime/int)
std::is_same_v<std::uint64_t, T>);
Expand Down

0 comments on commit f2fc621

Please sign in to comment.