added cstdint header to printing_toolbox.hpp by JanRotti · Pull Request #2099 · su2code/SU2
Proposed Changes
Adding cstdint header to printing_toolbox.hpp.
Reason
I was building SU2 from source this morning. Running ninja threw an compilation error with a comment to import cstdint in printing_toolbox.hpp since uint8_t is not defined in the scope:
...
../SU2_CFD/src/output/filewriter/CParaviewXMLFileWriter.cpp:240:10: error: ‘uint8_t’ was not declared in this scope
240 | vector<uint8_t> typeBuf(myElem);
| ^~~~~~~
../SU2_CFD/src/output/filewriter/CParaviewXMLFileWriter.cpp:30:1: note: ‘uint8_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
29 | #include "../../../../Common/include/toolboxes/printing_toolbox.hpp"
+++ |+#include <cstdint>
30 |
...
Adding the header solves the issue.
Here some additional information:
OS: Fedora 38
Configuration: MPI, pythonwrapper, autodiff
Compiler: g++ 13.1.1
PR Checklist
Put an X by all that apply. You can fill this out after submitting the PR. If you have any questions, don't hesitate to ask! We want to help. These are a guide for you to know what the reviewers will be looking for in your contribution.
- I am submitting my contribution to the develop branch.
- My contribution generates no new compiler warnings (try with --warnlevel=3 when using meson).
- My contribution is commented and consistent with SU2 style (https://su2code.github.io/docs_v7/Style-Guide/).
- I have added a test case that demonstrates my contribution, if necessary.
- I have updated appropriate documentation (Tutorials, Docs Page, config_template.cpp), if necessary.