search

lighttransport / nanort

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

fixing clang warning #48

Closed Ybalrid closed 5 years ago

Ybalrid commented 5 years ago

While working on glTF_insight, this warning prevent passing CI there : 

/home/travis/build/lighttransport/gltf-insight/third_party/nanort/nanort.h:2454:13: error: unused variable 'kMaxStackDepth' [-Werror,-Wunused-variable]
  const int kMaxStackDepth = 512;
            ^
1 error generated.

This variable is only used in an assert... So I just casted it to void. (Don't know if that was the best thing to do.)

Ybalrid commented 5 years ago

There are way more changes in that branch that I found that prevented CI to go green on gltf insight. They are all committed here