Changed to use a cmake option and made the define a little more direct about it's role.

Author: robertosfield

CMakeLists.txt

@@ -98,7 +98,7 @@ if (VSG_SUPPORTS_Windowing)
     endif()
 endif()
 
-set(VSG_USE_DYNAMIC_CAST 0 CACHE STRING "Use dynamic_cast in vsg::Object::cast<T>(), 0 for off, 1 for enabled." )
+option(VSG_USE_dynamic_cast "Use dynamic_cast in vsg::Object::cast<T>(), 0 for off, 1 for enabled." OFF)
 
 # this line needs to be after the call to setup_build_vars()
 configure_file("${VSG_SOURCE_DIR}/src/vsg/core/Version.h.in" "${VSG_VERSION_HEADER}")

include/vsg/core/Object.h

@@ -85,7 +85,7 @@ namespace vsg
         virtual const std::type_info& type_info() const noexcept { return typeid(Object); }
         virtual bool is_compatible(const std::type_info& type) const noexcept { return typeid(Object) == type; }
 
-        #if VSG_USE_DYNAMIC_CAST == 1
+        #if VSG_USE_dynamic_cast
             template<class T>
             T* cast() { return dynamic_cast<T*>(this); }
 

src/vsg/core/Version.h.in

@@ -45,8 +45,8 @@ extern "C"
     /// Native Windowing support provided with vsg::Window::create(windowTraits) enabled when 1, disabled when 0
     #cmakedefine01 VSG_SUPPORTS_Windowing
 
-    /// Define VSG_USE_DYNAMIC_CAST to avoid typeid(T) comparisons in vsg::Object::cast<T>()
-    #define VSG_USE_DYNAMIC_CAST @VSG_USE_DYNAMIC_CAST@
+    /// Define VSG_USE_dynamic_cast to avoid typeid(T) comparisons in vsg::Object::cast<T>()
+    #cmakedefine01 VSG_USE_dynamic_cast
 
     struct VsgVersion
     {