From 507c699ac0a569632e5913287d58999762572e1c Mon Sep 17 00:00:00 2001 From: Stephen Seo Date: Wed, 25 Aug 2021 11:57:54 +0900 Subject: [PATCH] WIP output PNG image --- example02_threaded_raytracing/.clang-tidy | 14 +++++--------- example02_threaded_raytracing/CMakeLists.txt | 4 ++++ .../clang_tidy_generation.sh | 2 +- example02_threaded_raytracing/src/rayTracer.cpp | 3 +++ example02_threaded_raytracing/src/rayTracer.hpp | 2 +- 5 files changed, 14 insertions(+), 11 deletions(-) diff --git a/example02_threaded_raytracing/.clang-tidy b/example02_threaded_raytracing/.clang-tidy index 0b70847..bee86fb 100644 --- a/example02_threaded_raytracing/.clang-tidy +++ b/example02_threaded_raytracing/.clang-tidy @@ -1,5 +1,5 @@ --- -Checks: 'clang-diagnostic-*,clang-analyzer-*,*,-llvmlibc-*,-fuchsia-*,-readability-magic-numbers,-cppcoreguidelines-avoid-magic-numbers,-modernize-use-trailing-return-type,-llvm-header-guard,-cppcoreguidelines-pro-bounds-pointer-arithmetic,-altera-struct-pack-align,-misc-non-private-member-variables-in-classes,-modernize-use-nodiscard,-cppcoreguidelines-pro-type-union-access,-llvm-else-after-return,-readability-else-after-return' +Checks: 'clang-diagnostic-*,clang-analyzer-*,*,-llvmlibc-*,-fuchsia-*,-readability-magic-numbers,-cppcoreguidelines-avoid-magic-numbers,-modernize-use-trailing-return-type,-llvm-header-guard,-cppcoreguidelines-pro-bounds-pointer-arithmetic,-altera-struct-pack-align,-misc-non-private-member-variables-in-classes,-modernize-use-nodiscard,-cppcoreguidelines-pro-type-union-access,-llvm-else-after-return,-readability-else-after-return,-cppcoreguidelines-owning-memory,-android-cloexec-fopen' WarningsAsErrors: '' HeaderFilterRegex: '' AnalyzeTemporaryDtors: false @@ -8,12 +8,12 @@ User: stephen CheckOptions: - key: cppcoreguidelines-no-malloc.Reallocations value: '::realloc' - - key: cppcoreguidelines-owning-memory.LegacyResourceConsumers - value: '::free;::realloc;::freopen;::fclose' - key: modernize-use-auto.MinTypeNameLength value: '5' - key: bugprone-reserved-identifier.Invert value: 'false' + - key: modernize-replace-disallow-copy-and-assign-macro.MacroName + value: DISALLOW_COPY_AND_ASSIGN - key: bugprone-narrowing-conversions.PedanticMode value: 'false' - key: bugprone-unused-return-value.CheckedFunctions @@ -30,8 +30,6 @@ CheckOptions: value: 'false' - key: cert-dcl37-c.AllowedIdentifiers value: '' - - key: modernize-replace-disallow-copy-and-assign-macro.MacroName - value: DISALLOW_COPY_AND_ASSIGN - key: hicpp-use-emplace.SmartPointers value: '::std::shared_ptr;::std::unique_ptr;::std::auto_ptr;::std::weak_ptr' - key: hicpp-member-init.IgnoreArrays @@ -316,8 +314,6 @@ CheckOptions: value: 'false' - key: modernize-use-emplace.TupleMakeFunctions value: '::std::make_pair;::std::make_tuple' - - key: cppcoreguidelines-owning-memory.LegacyResourceProducers - value: '::malloc;::aligned_alloc;::realloc;::calloc;::fopen;::freopen;::tmpfile' - key: hicpp-uppercase-literal-suffix.IgnoreMacros value: 'true' - key: bugprone-argument-comment.StrictMode @@ -334,12 +330,12 @@ CheckOptions: value: '' - key: modernize-avoid-bind.PermissiveParameterList value: 'false' - - key: hicpp-use-override.OverrideSpelling - value: override - key: modernize-use-override.FinalSpelling value: final - key: cert-err09-cpp.MaxSize value: '-1' + - key: hicpp-use-override.OverrideSpelling + value: override - key: hicpp-use-equals-default.IgnoreMacros value: 'true' - key: modernize-use-noexcept.ReplacementString diff --git a/example02_threaded_raytracing/CMakeLists.txt b/example02_threaded_raytracing/CMakeLists.txt index 8976e93..7531d22 100644 --- a/example02_threaded_raytracing/CMakeLists.txt +++ b/example02_threaded_raytracing/CMakeLists.txt @@ -40,6 +40,10 @@ else() endif() target_include_directories(Example02 PUBLIC ${GLM_INCLUDE_DIRS}) +find_package(PNG REQUIRED) +target_include_directories(Example02 PUBLIC ${PNG_INCLUDE_DIRS}) +target_link_libraries(Example02 PUBLIC ${PNG_LIBRARIES}) + find_program(CLANG_FORMAT "clang-format") if(CLANG_FORMAT) add_custom_target( diff --git a/example02_threaded_raytracing/clang_tidy_generation.sh b/example02_threaded_raytracing/clang_tidy_generation.sh index 6037041..cdf12e4 100755 --- a/example02_threaded_raytracing/clang_tidy_generation.sh +++ b/example02_threaded_raytracing/clang_tidy_generation.sh @@ -2,4 +2,4 @@ set -ev -clang-tidy src/main.cpp -checks=*,-llvmlibc-*,-fuchsia-*,-readability-magic-numbers,-cppcoreguidelines-avoid-magic-numbers,-modernize-use-trailing-return-type,-llvm-header-guard,-cppcoreguidelines-pro-bounds-pointer-arithmetic,-altera-struct-pack-align,-misc-non-private-member-variables-in-classes,-modernize-use-nodiscard,-cppcoreguidelines-pro-type-union-access,-llvm-else-after-return,-readability-else-after-return -dump-config -- -std=c++17 > .clang-tidy +clang-tidy src/main.cpp -checks=*,-llvmlibc-*,-fuchsia-*,-readability-magic-numbers,-cppcoreguidelines-avoid-magic-numbers,-modernize-use-trailing-return-type,-llvm-header-guard,-cppcoreguidelines-pro-bounds-pointer-arithmetic,-altera-struct-pack-align,-misc-non-private-member-variables-in-classes,-modernize-use-nodiscard,-cppcoreguidelines-pro-type-union-access,-llvm-else-after-return,-readability-else-after-return,-cppcoreguidelines-owning-memory,-android-cloexec-fopen -dump-config -- -std=c++17 > .clang-tidy diff --git a/example02_threaded_raytracing/src/rayTracer.cpp b/example02_threaded_raytracing/src/rayTracer.cpp index 50036d0..8f08da7 100644 --- a/example02_threaded_raytracing/src/rayTracer.cpp +++ b/example02_threaded_raytracing/src/rayTracer.cpp @@ -2,6 +2,7 @@ #include #include +#include #include #include @@ -9,6 +10,8 @@ #include #include +#include + const float PI = std::acos(-1.0F); Ex02::RT::Pixel::Pixel() : r(0), g(0), b(0) {} diff --git a/example02_threaded_raytracing/src/rayTracer.hpp b/example02_threaded_raytracing/src/rayTracer.hpp index 2f5307d..1571328 100644 --- a/example02_threaded_raytracing/src/rayTracer.hpp +++ b/example02_threaded_raytracing/src/rayTracer.hpp @@ -31,7 +31,7 @@ public: Pixel &getPixel(unsigned int x, unsigned int y); const Pixel &getPixel(unsigned int x, unsigned int y) const; - // returns actual output filename + // returns actual output filename (it appends the file extension) std::string writeToFile(const std::string &filename) const; private: