As described in issue #6, there were some header includes missing.
I have added the includes where they were needed, and removed them from places where they were not.
For this, I have tried to stick to the Google C++ Guidelines.
If you prefer a different style, do let me know and I will adapt the changes in this PR.
I also took the liberty to fix a warning related to the usage of LONG_MAX for a float value, replacing it with std::numeric_limits<float>::max().
With these changes, the library builds without errors or warning for my particular compiler settings.
medalotte
commented
2 years ago
As described in issue #6, there were some header includes missing. I have added the includes where they were needed, and removed them from places where they were not. For this, I have tried to stick to the Google C++ Guidelines. If you prefer a different style, do let me know and I will adapt the changes in this PR.
I also took the liberty to fix a warning related to the usage of
LONG_MAXfor a float value, replacing it withstd::numeric_limits<float>::max().With these changes, the library builds without errors or warning for my particular compiler settings.