search

jermp / pthash

Fast and compact minimal perfect hash functions in C++.
MIT License
190 stars 25 forks source link

missing include #15

Closed yhhshb closed 10 months ago

yhhshb commented 11 months ago

Hi @jermp , I think pthash/include/encoders/util.hpp is missing the cstdint and cassert headers. g++ 12.3 complains about uintX_t definitions and asserts.

jermp commented 11 months ago

Hi @yhhshb, which header?

jermp commented 11 months ago

I do not have this issue. It just compiles fine on gcc version 12.3.0. I think the problem you're seeing is because you're taking just the file util.hpp and place it in another project. Right?

jermp commented 11 months ago

Hi @yhhshb, can you please confirm?

yhhshb commented 10 months ago

Hi @jermp ,

No, I did not take util.hpp alone. I use the whole repository as a dependency of other projects.

jermp commented 10 months ago

Ah ok. Strange that I cannot replicate the issue from PTHash itself. Anyway, I've just added the missing includes.