Added the missing <limits> include

seomoz / url-cpp

C++ bindings for url parsing and sanitization

MIT License

19 stars 11 forks source link

Added the missing <limits> include #41

Closed dcfreire closed 8 months ago

dcfreire commented 2 years ago

Not all compilers add the <limits> library, which contains std::numeric_limits.

tdufala commented 8 months ago

Jumping into this project.

This PR seems valid to me, but the test environment itself seems to be broken, seemingly due to some issues setting up the environment. With that said, I'm fairly confident this won't break anything upon reviewing the code. Rather than blocking this on fixing the test environment, going to favor merging.