From 47ebe9b545d2efa9b60a735ee261d78eaa54aa57 Mon Sep 17 00:00:00 2001 From: assiduous Date: Tue, 28 Jan 2020 22:59:57 -0800 Subject: Fixed a number of minor issues --- .../GraphicsEngineOpenGL/interface/EngineFactoryOpenGL.h | 2 +- Graphics/GraphicsEngineOpenGL/readme.md | 6 +++--- Graphics/GraphicsEngineOpenGL/src/EngineFactoryOpenGL.cpp | 13 ++++++++----- 3 files changed, 12 insertions(+), 9 deletions(-) (limited to 'Graphics/GraphicsEngineOpenGL') diff --git a/Graphics/GraphicsEngineOpenGL/interface/EngineFactoryOpenGL.h b/Graphics/GraphicsEngineOpenGL/interface/EngineFactoryOpenGL.h index 0d42a16e..cff83e54 100644 --- a/Graphics/GraphicsEngineOpenGL/interface/EngineFactoryOpenGL.h +++ b/Graphics/GraphicsEngineOpenGL/interface/EngineFactoryOpenGL.h @@ -107,7 +107,7 @@ typedef struct IEngineFactoryOpenGL* (*GetEngineFactoryOpenGLType)(); inline GetEngineFactoryOpenGLType LoadGraphicsEngineOpenGL() { - return (GetEngineFactoryOpenGLType)LoadEngineDll("GraphicsEngineGL", "GetEngineFactoryOpenGL"); + return (GetEngineFactoryOpenGLType)LoadEngineDll("GraphicsEngineOpenGL", "GetEngineFactoryOpenGL"); } #else diff --git a/Graphics/GraphicsEngineOpenGL/readme.md b/Graphics/GraphicsEngineOpenGL/readme.md index 0c3c204b..b59cde75 100644 --- a/Graphics/GraphicsEngineOpenGL/readme.md +++ b/Graphics/GraphicsEngineOpenGL/readme.md @@ -14,9 +14,9 @@ using namespace Diligent; // ... #if ENGINE_DLL - GetEngineFactoryOpenGLType GetEngineFactoryOpenGL; - if(!LoadGraphicsEngineOpenGL(GetEngineFactoryOpenGL)) - return FALSE; + auto GetEngineFactoryOpenGL = LoadGraphicsEngineOpenGL(); + if (GetEngineFactoryOpenGL == nullptr) + return false; #endif RefCntAutoPtr pRenderDevice; RefCntAutoPtr pImmediateContext; diff --git a/Graphics/GraphicsEngineOpenGL/src/EngineFactoryOpenGL.cpp b/Graphics/GraphicsEngineOpenGL/src/EngineFactoryOpenGL.cpp index 024a7bfc..85372f5d 100644 --- a/Graphics/GraphicsEngineOpenGL/src/EngineFactoryOpenGL.cpp +++ b/Graphics/GraphicsEngineOpenGL/src/EngineFactoryOpenGL.cpp @@ -248,17 +248,20 @@ void EngineFactoryOpenGLImpl::AttachToActiveGLContext(const EngineGLCreateInfo& #ifdef DOXYGEN /// Loads OpenGL-based engine implementation and exports factory functions -/// \param [out] GetFactoryFunc - Pointer to the function that returns pointer to the factory for -/// the OpenGL engine implementation -/// See EngineFactoryOpenGLImpl::CreateDeviceAndSwapChainGL(). +/// +/// \return - Pointer to the function that returns pointer to the factory for +/// the OpenGL engine implementation +/// See EngineFactoryOpenGLImpl::CreateDeviceAndSwapChainGL(). +/// /// \remarks Depending on the configuration and platform, the function loads different dll: -/// Platform\\Configuration | Debug | Release +/// +/// Platform\\Configuration | Debug | Release /// --------------------------|------------------------------|---------------------------- /// Win32/x86 | GraphicsEngineOpenGL_32d.dll | GraphicsEngineOpenGL_32r.dll /// Win32/x64 | GraphicsEngineOpenGL_64d.dll | GraphicsEngineOpenGL_64r.dll /// /// To load the library on Android, it is necessary to call System.loadLibrary("GraphicsEngineOpenGL") from Java. -void LoadGraphicsEngineOpenGL(GetEngineFactoryOpenGLType& GetFactoryFunc) +GetEngineFactoryOpenGLType LoadGraphicsEngineOpenGL() { // This function is only required because DoxyGen refuses to generate documentation for a static function when SHOW_FILES==NO # error This function must never be compiled; -- cgit v1.2.3