RunPlanVector JSON IO minimum viable product (untested) #4160
Annotations
10 errors and 1 warning
|
Lint:
include/flamegpu/io/JSONRunPlanReader.h#L20
Add #include <string> for string
|
|
Lint:
include/flamegpu/io/JSONRunPlanWriter.h#L30
Missing spaces around =
|
|
Lint:
include/flamegpu/io/JSONRunPlanWriter.h#L21
Add #include <memory> for unique_ptr<>
|
|
Lint:
include/flamegpu/io/JSONRunPlanWriter.h#L30
Add #include <string> for string
|
|
Lint:
src/flamegpu/io/JSONRunPlanReader.cpp#L6
Found C system header after C++ system header. Should be: JSONRunPlanReader.h, c system, c++ system, other.
|
|
Lint:
src/flamegpu/io/JSONRunPlanReader.cpp#L7
Found C system header after C++ system header. Should be: JSONRunPlanReader.h, c system, c++ system, other.
|
|
Lint:
src/flamegpu/io/JSONRunPlanReader.cpp#L8
Found C system header after C++ system header. Should be: JSONRunPlanReader.h, c system, c++ system, other.
|
|
Lint:
src/flamegpu/io/JSONRunPlanReader.cpp#L234
Namespace should be terminated with "// namespace io"
|
|
Lint:
src/flamegpu/io/JSONRunPlanReader.cpp#L235
Namespace should be terminated with "// namespace flamegpu"
|
|
Lint:
src/flamegpu/io/JSONRunPlanWriter.cpp#L5
Found C system header after C++ system header. Should be: JSONRunPlanWriter.h, c system, c++ system, other.
|
|
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
Loading