Minor additional comments, formatting and fixes.

Signed-off-by: Eric Renaud-Houde <eric.renaud.houde@gmail.com>

Author: num3ric

src/libutils/oglapphelpers/graphicalapp.cpp

@@ -14,6 +14,9 @@
 namespace OCIO_NAMESPACE
 {
 
+// Factory for windowed backends (OGL, DX). For headless Vulkan use
+// VulkanApp::CreateVulkanApp(). Metal is always paired with OGL (MetalApp
+// inherits ScreenOglApp) and is covered by the OCIO_GL_ENABLED branch.
 GraphicalAppRcPtr GraphicalApp::CreateApp(const char * winTitle, int winWidth, int winHeight)
 {
 #ifdef OCIO_GL_ENABLED

src/libutils/oglapphelpers/graphicalapp.h

@@ -7,7 +7,7 @@
 
 namespace OCIO_NAMESPACE
 {
-// Forward declaration  of GraphicalApp.
+// Forward declaration of GraphicalApp.
 class GraphicalApp;
 typedef OCIO_SHARED_PTR<GraphicalApp> GraphicalAppRcPtr;
 
@@ -65,7 +65,7 @@ class GraphicalApp
     virtual void readImage(float* imageBuffer) = 0;
 
     // Helper to print graphics info.
-    void virtual printGraphicsInfo() const noexcept = 0;
+    virtual void printGraphicsInfo() const noexcept = 0;
 
     // Factory: returns a platform-appropriate GraphicalApp (OGL or DX).
     static GraphicalAppRcPtr CreateApp(const char * winTitle, int winWidth, int winHeight);

src/libutils/oglapphelpers/oglapp.cpp

@@ -388,7 +388,7 @@ void HeadlessOglApp::printEGLInfo() const noexcept
 
 void HeadlessOglApp::redisplay()
 {
-    GraphicalApp::redisplay();
+    OglApp::redisplay();
     eglSwapBuffers(m_eglDisplay, m_eglSurface);
 }
 

tests/gpu/GPUUnitTest.cpp

@@ -737,6 +737,8 @@ int main(int argc, const char ** argv)
 #ifdef OCIO_DIRECTX_ENABLED
         if (useDxRenderer)
         {
+            // SM_5_0 controls OCIO shader code generation syntax; DxApp always
+            // compiles to SM 6.0 via DXC regardless of this enum value.
             shadingLanguage = OCIO::GPU_LANGUAGE_HLSL_SM_5_0;
         }
         else