log: Cleanup incorrect eol usages (#1684)

Author: mwestphal

library/src/scene_impl.cxx

@@ -158,8 +158,11 @@ class scene_impl::internals
       this->Window.getCamera().resetToBounds();
     }
 
-    // Print info about scene and coloring
+    // Display coloring information
     this->Window.PrintColoringDescription(log::VerboseLevel::DEBUG);
+    log::debug("");
+
+    // Print scene description
     this->Window.PrintSceneDescription(log::VerboseLevel::DEBUG);
   }
 

vtkext/private/module/F3DColoringInfoHandler.cxx

@@ -120,7 +120,7 @@ std::optional<F3DColoringInfoHandler::ColoringInfo> F3DColoringInfoHandler::SetC
 
     if (!quiet)
     {
-      F3DLog::Print(F3DLog::Severity::Debug, "No array to color with\n");
+      F3DLog::Print(F3DLog::Severity::Debug, "No array to color with");
     }
   }
   else if (!arrayName.has_value())
@@ -138,7 +138,7 @@ std::optional<F3DColoringInfoHandler::ColoringInfo> F3DColoringInfoHandler::SetC
       this->CurrentColoringIter.reset();
       if (!quiet)
       {
-        F3DLog::Print(F3DLog::Severity::Warning, "Unknown scalar array: \"" + arrayName.value() + "\"\n");
+        F3DLog::Print(F3DLog::Severity::Warning, "Unknown scalar array: \"" + arrayName.value() + "\"");
       }
     }
   }

vtkext/private/module/vtkF3DRenderer.cxx

@@ -149,7 +149,7 @@ vtkSmartPointer<vtkTexture> GetTexture(const std::string& filePath, bool isSRGB
     std::string fullPath = vtksys::SystemTools::CollapseFullPath(filePath);
     if (!vtksys::SystemTools::FileExists(fullPath))
     {
-      F3DLog::Print(F3DLog::Severity::Warning, "Texture file does not exist " + fullPath + "\n");
+      F3DLog::Print(F3DLog::Severity::Warning, "Texture file does not exist " + fullPath);
     }
     else
     {
@@ -171,7 +171,7 @@ vtkSmartPointer<vtkTexture> GetTexture(const std::string& filePath, bool isSRGB
       }
       else
       {
-        F3DLog::Print(F3DLog::Severity::Warning, "Cannot open texture file " + fullPath + "\n");
+        F3DLog::Print(F3DLog::Severity::Warning, "Cannot open texture file " + fullPath);
       }
     }
   }
@@ -614,7 +614,7 @@ void vtkF3DRenderer::ConfigureGridUsingCurrentActors()
       std::stringstream stream;
       stream << "Using grid unit square size = " << tmpUnitSquare << "\n"
              << "Grid origin set to [" << gridPos[0] << ", " << gridPos[1] << ", " << gridPos[2]
-             << "]\n\n";
+             << "]\n";
       this->GridInfo = stream.str();
 
       vtkNew<vtkF3DOpenGLGridMapper> gridMapper;
@@ -1877,7 +1877,7 @@ void vtkF3DRenderer::ConfigureActorsProperties()
     if (this->SurfaceColor.value().size() != 3)
     {
       F3DLog::Print(F3DLog::Severity::Warning,
-        std::string("Invalid surface color provided, not applying\n"));
+        std::string("Invalid surface color provided, not applying"));
     }
     else
     {
@@ -1891,7 +1891,7 @@ void vtkF3DRenderer::ConfigureActorsProperties()
     if (this->EmissiveFactor.value().size() != 3)
     {
       F3DLog::Print(F3DLog::Severity::Warning,
-        std::string("Invalid emissive factor provided, not applying\n"));
+        std::string("Invalid emissive factor provided, not applying"));
     }
     else
     {
@@ -2416,11 +2416,11 @@ std::string vtkF3DRenderer::GetColoringDescription()
   {
     stream << "Coloring using " << (this->UseCellColoring ? "cell" : "point") << " array named "
            << info.value().Name << (this->EnableColoring ? ", " : " (forced), ")
-           << vtkF3DRenderer::ComponentToString(this->ComponentForColoring) << "\n";
+           << vtkF3DRenderer::ComponentToString(this->ComponentForColoring);
   }
   else
   {
-    stream << "Not coloring\n";
+    stream << "Not coloring";
   }
   return stream.str();
 }
@@ -2555,7 +2555,7 @@ void vtkF3DRenderer::ConfigureRangeAndCTFForColoring(
   if (this->ComponentForColoring >= info.MaximumNumberOfComponents)
   {
     F3DLog::Print(F3DLog::Severity::Warning,
-      std::string("Invalid component index: ") + std::to_string(this->ComponentForColoring) + "\n");
+      std::string("Invalid component index: ") + std::to_string(this->ComponentForColoring));
     return;
   }
 
@@ -2572,7 +2572,7 @@ void vtkF3DRenderer::ConfigureRangeAndCTFForColoring(
     else
     {
       F3DLog::Print(F3DLog::Severity::Warning,
-        std::string("Invalid scalar range provided, using automatic range\n"));
+        std::string("Invalid scalar range provided, using automatic range"));
     }
   }
 

vtkext/private/module/vtkF3DRenderer.h

@@ -158,6 +158,7 @@ class vtkF3DRenderer : public vtkOpenGLRenderer
   /**
    * Return description about the current rendering status
    * Currently contains information about the camera and the grid if any
+   * Returns a multiline string containing the scene description
    */
   std::string GetSceneDescription();
 
@@ -321,6 +322,7 @@ class vtkF3DRenderer : public vtkOpenGLRenderer
 
   /**
    * Get information about the current coloring
+   * Returns a single line string containing the coloring description
    */
   virtual std::string GetColoringDescription();
 
@@ -406,7 +408,8 @@ class vtkF3DRenderer : public vtkOpenGLRenderer
 
   /**
    * Generate a padded metadata description
-   * using the internal importer
+   * using the internal importer.
+   * Returns a multiline string containing the meta data description
    */
   std::string GenerateMetaDataDescription();