Skip to content

Commit c0cf10a

Browse files
authored
Move 'src/library/digest/murmur' unit tests to 'tests/' (#182) (#247)
1 parent abe02a8 commit c0cf10a

File tree

5 files changed

+17
-20
lines changed

5 files changed

+17
-20
lines changed

src/library/digest/murmur/murmur.h

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -37,12 +37,12 @@ class TMurmurHash2A {
3737
using TValue = typename TTraits::TValue;
3838

3939
public:
40-
inline TMurmurHash2A(TValue seed = 0)
40+
TMurmurHash2A(TValue seed = 0)
4141
: Hash(seed)
4242
{
4343
}
4444

45-
inline TMurmurHash2A& Update(const void* buf, size_t len) noexcept {
45+
TMurmurHash2A& Update(const void* buf, size_t len) noexcept {
4646
Size += len;
4747

4848
MixTail(buf, len);
@@ -58,8 +58,8 @@ class TMurmurHash2A {
5858
return *this;
5959
}
6060

61-
inline TValue Value() const noexcept {
62-
TValue hash = Mix(Mix(Hash, Tail), (TValue)Size);
61+
TValue Value() const noexcept {
62+
TValue hash = Mix(Mix(Hash, Tail), static_cast<TValue>(Size));
6363

6464
hash ^= hash >> TTraits::R2;
6565
hash *= TTraits::Multiplier;
@@ -69,7 +69,7 @@ class TMurmurHash2A {
6969
}
7070

7171
private:
72-
static inline TValue Mix(TValue h, TValue k) noexcept {
72+
static TValue Mix(TValue h, TValue k) noexcept {
7373
k *= TTraits::Multiplier;
7474
k ^= k >> TTraits::R1;
7575
k *= TTraits::Multiplier;
@@ -78,9 +78,9 @@ class TMurmurHash2A {
7878
return h;
7979
}
8080

81-
inline void MixTail(const void*& buf, size_t& len) noexcept {
81+
void MixTail(const void*& buf, size_t& len) noexcept {
8282
while (len && (len < sizeof(TValue) || Count)) {
83-
Tail |= (TValue) * ((const unsigned char*&)buf)++ << (Count++ * 8);
83+
Tail |= (TValue) * (reinterpret_cast<const unsigned char*&>(buf))++ << (Count++ * 8);
8484

8585
--len;
8686

src/library/digest/murmur/ut/ya.make

Lines changed: 0 additions & 13 deletions
This file was deleted.

tests/library/digest/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
add_subdirectory(argonish)
22
add_subdirectory(lower_case)
33
add_subdirectory(md5)
4+
add_subdirectory(murmur)
Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
add_ydb_test(NAME library-digest-murmur-ut
2+
SOURCES
3+
ut.cpp
4+
INCLUDE_DIRS
5+
${YDB_SDK_SOURCE_DIR}/src/library/digest/murmur
6+
LINK_LIBRARIES
7+
cpp-digest-murmur
8+
cpp-testing-unittest_main
9+
)

0 commit comments

Comments
 (0)