From 1c85080a1e912f240159a3ae30e822d37b936c0e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jan=20M=C3=B6bius?= <moebius@cs.rwth-aachen.de>
Date: Fri, 15 Apr 2016 15:57:07 +0200
Subject: [PATCH] - Updated changelog - Always emit warning

---
 Doc/changelog.docu                       |  1 +
 src/OpenMesh/Core/IO/reader/PLYReader.cc | 17 +++++++++--------
 2 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/Doc/changelog.docu b/Doc/changelog.docu
index f336bf26..57845d21 100644
--- a/Doc/changelog.docu
+++ b/Doc/changelog.docu
@@ -17,6 +17,7 @@
 <b>IO</b>
 <ul>
 <li>Obj reader: added texCoord3d functions to objloader</li>
+<li>PLY Reader: Don't emit warning for every complex face but warn only once in the end.</li>
 <li>Importer: Integrate non-manifold faces while importing and not at the end. Fixes missing properties on non-manifolds. (Thanks to Max Limper for the patch, Merge Request 51)<li>
 </ul>
 
diff --git a/src/OpenMesh/Core/IO/reader/PLYReader.cc b/src/OpenMesh/Core/IO/reader/PLYReader.cc
index bb357990..2d04f68a 100644
--- a/src/OpenMesh/Core/IO/reader/PLYReader.cc
+++ b/src/OpenMesh/Core/IO/reader/PLYReader.cc
@@ -456,11 +456,11 @@ bool _PLYReader_::read_ascii(std::istream& _in, BaseImporter& _bi, const Options
 
     }
 
-    if (err_enabled) {
+    if (err_enabled) 
       omerr().enable();
-      if (complex_faces)
-        omerr() << complex_faces << " invalid faces could not be added.\n";
-    }
+
+    if (complex_faces)
+      omerr() << complex_faces << "The reader encountered invalid faces, that could not be added.\n";
 
     // File was successfully parsed.
     return true;
@@ -646,11 +646,12 @@ bool _PLYReader_::read_binary(std::istream& _in, BaseImporter& _bi, bool /*_swap
       }
     }
 
-    if (err_enabled) {
+    if (err_enabled) 
       omerr().enable();
-      if (complex_faces)
-        omerr() << complex_faces << " invalid faces could not be added.\n";
-    }
+
+   if (complex_faces)
+      omerr() << complex_faces << "The reader encountered invalid faces, that could not be added.\n";
+
 
     return true;
 }
-- 
GitLab