fix: C preprocessor defaults undefined macros to 0

arun11299 / cpp-jwt

JSON Web Token library for C++

MIT License

395 stars 111 forks source link

fix: C preprocessor defaults undefined macros to 0 #75

Closed saurik closed 3 years ago

saurik commented 3 years ago

Without this patch, this isn't compiling on my newer version of OpenSSL as I am not using LibreSSL (which is then 0, less than any version)... I imagine that, without this patch, it also wouldn't be working for people using newer LibreSSL.

saurik commented 3 years ago

UGH, I glanced at issues but I failed to glance at pull requests: this is already filed as #73... though without also checking if the OPENSSL_VERSION_NUMBER is defined. (Does LibreSSL always define that? That might be why.) If that's the case I'd probably just merge that one (which was filed 26 days ago).

williamswhy commented 3 years ago

This is exactly the problem I have here #77. Hope this gets merged soon.

arun11299 commented 3 years ago

Thanks guys. I have merged @williamswhy's pull request.

Let me know if something else needs to be done.