[Lcms-user] C++17 "register" warning when including lcms2.h

[Hanno H. <Han...@gm...>]

Hi,

VS 2017 produces a warning when including lcms2.h from C++ code:
"warning C5033: 'register' is no longer a supported storage class".

I've read that "register" is still valid C, just not C++17, see also

https://developercommunity.visualstudio.com/content/problem/117986/unexpected-warning-c5033-for-using-register-storag.html

Out of curiosity, I've removed the keyword from all interfaces and compared the result of the testbed.
Performance of register- and non-register versions were identical on x64 lcms2-2.9 builds,
both on MacOSX (Apple LLVM version 9.0.0) and Windows 10 VS 2017.

So I wonder if it would be possible to remove the keyword and leave this optimization to the compiler, or is it still needed for old compilers / systems?

Best regards

Hanno