jll63 / yomm2

Fast, orthogonal, open multi-methods. Solve the Expression Problem in C++17.
Boost Software License 1.0
348 stars 18 forks source link

Possibly missing include in header? #26

Closed derpda closed 3 years ago

derpda commented 3 years ago

It seems like include/yorel/yomm2.hpp is missing an include for std::uintptr_t! At least on my system the compilation now fails.

std::uintptr_t is defined in <cstdint>, and adding that include fixes the issue for me.

jll63 commented 3 years ago

Yes indeed. Hmmm that would not have arrived in D. I am going to try iwyu.

jll63 commented 3 years ago

@derpda If you have a minute, please review and test in your context. https://github.com/jll63/yomm2/pull/27

derpda commented 3 years ago

27 indeed fixes this!