tiny TODOs

Author: devshgraphicsprogramming

include/nbl/config/BuildConfigOptions.h.in

@@ -1,10 +1,12 @@
 #ifndef __NBL_BUILD_CONFIG_OPTIONS_H_INCLUDED__
 #define __NBL_BUILD_CONFIG_OPTIONS_H_INCLUDED__
 
+// TODO: figure out _ and _ suffixes and prefixes
+
 #cmakedefine _NBL_DEBUG
 #cmakedefine _NBL_RELWITHDEBINFO
 
-// architecture
+// architecture (TODO: rethink)
 #cmakedefine __NBL_COMPILE_WITH_ARM_SIMD_
 
 // libraries
@@ -17,6 +19,7 @@
 #cmakedefine _NBL_PLATFORM_OSX_
 #cmakedefine _NBL_PLATFORM_ANDROID_
 
+// TODO: X11 vs Wayland shouldn't even be an option (dlopen at runtime!)
 #cmakedefine _NBL_BUILD_WITH_WAYLAND
 
 // graphics API backend
@@ -57,7 +60,6 @@
 #cmakedefine _NBL_COMPILE_WITH_GLTF_WRITER_
 
 // compute interop
-#cmakedefine _NBL_COMPILE_WITH_OPENCL_
 #cmakedefine _NBL_COMPILE_WITH_CUDA_
 #cmakedefine _NBL_BUILD_OPTIX_
 

include/nbl/system/IApplicationFramework.h

@@ -38,6 +38,7 @@ class IApplicationFramework : public core::IReferenceCounted
     protected:
         ~IApplicationFramework() {}
 
+        // TODO: why aren't these pure virtual, and why do we even need a `_impl()`suffix?
         virtual void onAppInitialized_impl() {}
         virtual void onAppTerminated_impl() {}
 

include/nbl/system/IThreadHandler.h

@@ -26,7 +26,7 @@ template <typename CRTP, typename InternalStateType = void>
 class IThreadHandler
 {
 private:
-    // TODO: @Crisspl factor this out somewhere? `nbl/core/reflection` ?
+    // TODO: @AnastzIuk factor this out somewhere? `nbl/core/reflection` ?
 #define _NBL_IMPL_MEMBER_FUNC_PRESENCE_CHECKER(member_func_name)\
     class has_##member_func_name\
     {\