lighttransport / nanort

NanoRT, single header only modern ray tracing kernel.
MIT License
1.08k stars 91 forks source link

Fix warning about shadowed variable `i`. (also ran clang-format) #47

Closed Ybalrid closed 5 years ago

Ybalrid commented 5 years ago

Found this while working on raycasted mouse selection in gltf-insight:

This is on the codepath activated by using these defines

   #define NANORT_USE_CPP11_FEATURE
   #define NANORT_ENABLE_PARALLEL_BUILD

I don't think current CI setup for NanoRT is using these. It's low priority, but I may add a build configuration in Travis to take this scenario into account.