From: Nicolas RECHATIN <nicolas.rechatin@cea.fr>
Date: Thu, 17 Jun 2021 14:19:28 +0000 (+0200)
Subject: + XML formatting and tooltips
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=ad8f629a9dbf7e7b2a496b5eddd8d2403e691a04;p=modules%2Fshaper.git

+ XML formatting and tooltips
+ CPP Includes sort and lint
---

diff --git a/src/FiltersAPI/FiltersAPI_Feature.cpp b/src/FiltersAPI/FiltersAPI_Feature.cpp
index 095f45775..0b5def82e 100644
--- a/src/FiltersAPI/FiltersAPI_Feature.cpp
+++ b/src/FiltersAPI/FiltersAPI_Feature.cpp
@@ -20,11 +20,11 @@
 #include "FiltersAPI_Feature.h"
 
 #include <ModelAPI_Feature.h>
+#include <ModelAPI_FiltersFactory.h>
+#include <ModelAPI_Session.h>
 
 #include <ModelHighAPI_Dumper.h>
 #include <ModelHighAPI_Tools.h>
-#include <ModelAPI_Session.h>
-#include <ModelAPI_FiltersFactory.h>
 
 FiltersAPI_Feature::FiltersAPI_Feature(
     const std::shared_ptr<ModelAPI_Feature> & theFeature)
diff --git a/src/FiltersAPI/FiltersAPI_Filter.cpp b/src/FiltersAPI/FiltersAPI_Filter.cpp
index 37d7881e6..f1f37393a 100644
--- a/src/FiltersAPI/FiltersAPI_Filter.cpp
+++ b/src/FiltersAPI/FiltersAPI_Filter.cpp
@@ -84,7 +84,6 @@ FiltersAPI_Filter::FiltersAPI_Filter(const std::string& theName,
       myFilterArguments.push_back(FiltersAPI_Argument(aBoolean->value()));
       continue;
     }
-
   }
 }
 
diff --git a/src/FiltersPlugin/FiltersPlugin_ContinuousFaces.cpp b/src/FiltersPlugin/FiltersPlugin_ContinuousFaces.cpp
index 0ab82ea2c..67726eaa1 100644
--- a/src/FiltersPlugin/FiltersPlugin_ContinuousFaces.cpp
+++ b/src/FiltersPlugin/FiltersPlugin_ContinuousFaces.cpp
@@ -22,16 +22,15 @@
 #include <ModelAPI_AttributeSelectionList.h>
 #include <ModelAPI_AttributeSelection.h>
 #include <ModelAPI_AttributeDouble.h>
-
 #include <ModelAPI_Tools.h>
 
 #include <GeomAPI_Edge.h>
 #include <GeomAPI_Shape.h>
-#include <GeomAlgoAPI_ShapeTools.h>
+#include <GeomAPI_ShapeExplorer.h>
 #include <GeomAPI_Wire.h>
 
-#include <GeomAPI_ShapeExplorer.h>
 #include <GeomAlgoAPI_ContinuousFaces.h>
+#include <GeomAlgoAPI_ShapeTools.h>
 
 #include <map>
 #include <math.h>
@@ -75,10 +74,10 @@ static void cacheContinuousFace(const GeomShapePtr theFace,
       if (theCache.find(*aFIt) == theCache.end()) {
        GeomPointPtr aPoint = anEdge->middlePoint();
        if (isContinuousFaces(theFace,
-                              *aFIt,
-                              aPoint,
-                              theAngle,
-                              anError)) {
+                             *aFIt,
+                             aPoint,
+                             theAngle,
+                             anError)) {
           theCache.insert(*aFIt);
           cacheContinuousFace(*aFIt, theEdgeToFaces, theCache, theAngle);
         }
diff --git a/src/FiltersPlugin/FiltersPlugin_EdgeSize.cpp b/src/FiltersPlugin/FiltersPlugin_EdgeSize.cpp
index a5b6ff8fb..854636ab0 100644
--- a/src/FiltersPlugin/FiltersPlugin_EdgeSize.cpp
+++ b/src/FiltersPlugin/FiltersPlugin_EdgeSize.cpp
@@ -21,21 +21,20 @@
 
 #include <ModelAPI_AttributeString.h>
 #include <ModelAPI_AttributeDouble.h>
-
 #include <ModelAPI_ResultBody.h>
 #include <ModelAPI_Tools.h>
 
 #include <GeomAPI_Edge.h>
 #include <GeomAPI_Shape.h>
-#include <GeomAlgoAPI_ShapeTools.h>
 #include <GeomAPI_Wire.h>
 
+#include <GeomAlgoAPI_ShapeTools.h>
+
 #include <Precision.hxx>
 
 #include <map>
 #include <iostream>
 
-
 bool FiltersPlugin_EdgeSize::isSupported(GeomAPI_Shape::ShapeType theType) const
 {
   return theType == GeomAPI_Shape::EDGE;
diff --git a/src/FiltersPlugin/FiltersPlugin_FaceSize.cpp b/src/FiltersPlugin/FiltersPlugin_FaceSize.cpp
index 17923ffba..8479c4a36 100644
--- a/src/FiltersPlugin/FiltersPlugin_FaceSize.cpp
+++ b/src/FiltersPlugin/FiltersPlugin_FaceSize.cpp
@@ -21,14 +21,14 @@
 
 #include <ModelAPI_AttributeString.h>
 #include <ModelAPI_AttributeDouble.h>
-
 #include <ModelAPI_Tools.h>
 
 #include <GeomAPI_Edge.h>
 #include <GeomAPI_Shape.h>
-#include <GeomAlgoAPI_ShapeTools.h>
 #include <GeomAPI_Wire.h>
+
 #include <GeomAlgoAPI_BasicProperties.h>
+#include <GeomAlgoAPI_ShapeTools.h>
 
 #include <Precision.hxx>
 
diff --git a/src/FiltersPlugin/FiltersPlugin_FeatureEdges.cpp b/src/FiltersPlugin/FiltersPlugin_FeatureEdges.cpp
index 4b6ad58cc..1617be13b 100644
--- a/src/FiltersPlugin/FiltersPlugin_FeatureEdges.cpp
+++ b/src/FiltersPlugin/FiltersPlugin_FeatureEdges.cpp
@@ -21,16 +21,16 @@
 
 #include <ModelAPI_AttributeString.h>
 #include <ModelAPI_AttributeDouble.h>
-
 #include <ModelAPI_ResultBody.h>
 #include <ModelAPI_Tools.h>
-#include <GeomAPI_ShapeExplorer.h>
-#include <GeomAlgoAPI_ContinuousFaces.h>
 
 #include <GeomAPI_Edge.h>
 #include <GeomAPI_Shape.h>
+#include <GeomAPI_ShapeExplorer.h>
 #include <GeomAPI_Wire.h>
 
+#include <GeomAlgoAPI_ContinuousFaces.h>
+
 #include <map>
 #include <math.h>
 #include <iostream>
diff --git a/src/FiltersPlugin/FiltersPlugin_Plugin.cpp b/src/FiltersPlugin/FiltersPlugin_Plugin.cpp
index 343cdd267..59d25b9dd 100644
--- a/src/FiltersPlugin/FiltersPlugin_Plugin.cpp
+++ b/src/FiltersPlugin/FiltersPlugin_Plugin.cpp
@@ -17,24 +17,24 @@
 // See http://www.salome-platform.org/ or email : webmaster.salome@opencascade.com
 //
 
-#include "FiltersPlugin_Plugin.h"
-#include "FiltersPlugin_Selection.h"
-#include "FiltersPlugin_HorizontalFace.h"
-#include "FiltersPlugin_VerticalFace.h"
 #include "FiltersPlugin_BelongsTo.h"
-#include "FiltersPlugin_OnPlane.h"
-#include "FiltersPlugin_OnLine.h"
-#include "FiltersPlugin_OnGeometry.h"
-#include "FiltersPlugin_OnPlaneSide.h"
-#include "FiltersPlugin_OppositeToEdge.h"
+#include "FiltersPlugin_ContinuousFaces.h"
 #include "FiltersPlugin_EdgeSize.h"
+#include "FiltersPlugin_ExternalFaces.h"
 #include "FiltersPlugin_FaceSize.h"
 #include "FiltersPlugin_FeatureEdges.h"
-#include "FiltersPlugin_ContinuousFaces.h"
-#include "FiltersPlugin_VolumeSize.h"
+#include "FiltersPlugin_HorizontalFace.h"
+#include "FiltersPlugin_OnGeometry.h"
+#include "FiltersPlugin_OnLine.h"
+#include "FiltersPlugin_OnPlane.h"
+#include "FiltersPlugin_OnPlaneSide.h"
+#include "FiltersPlugin_OppositeToEdge.h"
+#include "FiltersPlugin_Plugin.h"
 #include "FiltersPlugin_RelativeToSolid.h"
-#include "FiltersPlugin_ExternalFaces.h"
+#include "FiltersPlugin_Selection.h"
 #include "FiltersPlugin_Validators.h"
+#include "FiltersPlugin_VerticalFace.h"
+#include "FiltersPlugin_VolumeSize.h"
 
 #include <Config_ModuleReader.h>
 
diff --git a/src/FiltersPlugin/FiltersPlugin_VolumeSize.cpp b/src/FiltersPlugin/FiltersPlugin_VolumeSize.cpp
index 32f04d30f..2a4a961c2 100644
--- a/src/FiltersPlugin/FiltersPlugin_VolumeSize.cpp
+++ b/src/FiltersPlugin/FiltersPlugin_VolumeSize.cpp
@@ -21,14 +21,14 @@
 
 #include <ModelAPI_AttributeString.h>
 #include <ModelAPI_AttributeDouble.h>
-
 #include <ModelAPI_Tools.h>
 
 #include <GeomAPI_Edge.h>
 #include <GeomAPI_Shape.h>
-#include <GeomAlgoAPI_ShapeTools.h>
 #include <GeomAPI_Wire.h>
+
 #include <GeomAlgoAPI_BasicProperties.h>
+#include <GeomAlgoAPI_ShapeTools.h>
 
 #include <Precision.hxx>
 
diff --git a/src/FiltersPlugin/filter-ContinuousFaces.xml b/src/FiltersPlugin/filter-ContinuousFaces.xml
index 5864b282e..45036ef51 100644
--- a/src/FiltersPlugin/filter-ContinuousFaces.xml
+++ b/src/FiltersPlugin/filter-ContinuousFaces.xml
@@ -1,13 +1,14 @@
 <filter id="ContinuousFaces">
   <doublevalue id="ContinuousFaces__value"
-      label="Angle"
-      min="0"
-      step="0.1"
-      default="5">
+               label="Angle"
+               min="0"
+               step="0.1"
+               default="5"
+               tooltip="Continuity angle">
   </doublevalue>
   <multi_selector id="ContinuousFaces__faces"
                   label="Faces:"
-                  tooltip="Select face."
+                  tooltip="Select faces"
                   shape_types="faces">
     <validator id="GeomValidators_ShapeType" parameters="face"/>
   </multi_selector>
diff --git a/src/FiltersPlugin/filter-EdgeSize.xml b/src/FiltersPlugin/filter-EdgeSize.xml
index bb459a633..b6eaed803 100644
--- a/src/FiltersPlugin/filter-EdgeSize.xml
+++ b/src/FiltersPlugin/filter-EdgeSize.xml
@@ -1,70 +1,78 @@
 <filter id="EdgeSize">
-    <switch id="EdgeSize__comparatorType">
+  <switch id="EdgeSize__comparatorType">
     <case id="inf" title="&lt;">
       <doublevalue id="EdgeSize__value"
-                  label="Size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+                   label="Size"
+                   min="0"
+                   step="0.1"
+                   default="1"
+                   tooltip="&lt; to">
         <validator id="GeomValidators_Positive"/>
       </doublevalue>
-    </case> 
+    </case>
     <case id="infEq" title="&lt;=">
-    <doublevalue id="EdgeSize__value"
-                  label="Size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="EdgeSize__value"
+                   label="Size"
+                   min="0"
+                   step="0.1"
+                   default="1"
+                   tooltip="&lt;= to">
         <validator id="GeomValidators_Positive"/>
       </doublevalue>
-     </case>
+    </case>
     <case id="sup" title="&gt;">
-    <doublevalue id="EdgeSize__value"
-                  label="Size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="EdgeSize__value"
+                   label="Size"
+                   min="0"
+                   step="0.1"
+                   default="1"
+                   tooltip="&gt; to">
         <validator id="GeomValidators_Positive" parameters="-1.e-10"/>
       </doublevalue>
-     </case>
+    </case>
     <case id="supEq" title="&gt;=">
-    <doublevalue id="EdgeSize__value"
-                  label="Size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="EdgeSize__value"
+                   label="Size"
+                   min="0"
+                   step="0.1"
+                   default="1"
+                   tooltip="&gt;= to">
         <validator id="GeomValidators_Positive" parameters="-1.e-10"/>
       </doublevalue>
-     </case>
+    </case>
     <case id="isBetween" title="is between">
-     <doublevalue id="EdgeSize__value"
-                  label="Min size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="EdgeSize__value"
+                   label="Min size"
+                   min="0"
+                   step="0.1"
+                   default="1"
+                   tooltip="Min value">
         <validator id="GeomValidators_Positive" parameters="-1.e-10"/>
       </doublevalue>
       <doublevalue id="EdgeSize__valueMax"
-                  label="Max size"
-                  min="0"
-                  step="0.1"
-                  default="10">
+                   label="Max size"
+                   min="0"
+                   step="0.1"
+                   default="10"
+                   tooltip="Max value">
         <validator id="GeomValidators_Positive"/>
       </doublevalue>
     </case>
     <case id="isStrictlyBetween" title="is strictly between">
-     <doublevalue id="EdgeSize__value"
-                  label="Min size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="EdgeSize__value"
+                   label="Min size"
+                   min="0"
+                   step="0.1"
+                   default="1"
+                   tooltip="Min value">
         <validator id="GeomValidators_Positive" parameters="-1.e-10"/>
       </doublevalue>
       <doublevalue id="EdgeSize__valueMax"
-                  label="Max size"
-                  min="0"
-                  step="0.1"
-                  default="10">
+                   label="Max size"
+                   min="0"
+                   step="0.1"
+                   default="10"
+                   tooltip="Max value">
         <validator id="GeomValidators_Positive" />
       </doublevalue>
     </case>
diff --git a/src/FiltersPlugin/filter-FaceSize.xml b/src/FiltersPlugin/filter-FaceSize.xml
index 2eff84f7b..41d75e61f 100644
--- a/src/FiltersPlugin/filter-FaceSize.xml
+++ b/src/FiltersPlugin/filter-FaceSize.xml
@@ -1,69 +1,70 @@
 <filter id="FaceSize">
-    <switch id="FaceSize__comparatorType">
+  <switch id="FaceSize__comparatorType">
     <case id="inf" title="&lt;">
       <doublevalue id="FaceSize__value"
-                  label="Size"
-                  min="0"
-                  default="1">
+                   label="Size"
+                   min="0"
+                   default="1"
+                   tooltip="&lt; to">
         <validator id="GeomValidators_Positive"/>
       </doublevalue>
-    </case> 
+    </case>
     <case id="infEq" title="&lt;=">
-    <doublevalue id="FaceSize__value"
-                  label="Size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="FaceSize__value"
+                   label="Size"
+                   min="0" step="0.1"
+                   default="1"
+                   tooltip="&lt;= to">
         <validator id="GeomValidators_Positive"/>
       </doublevalue>
-     </case>
+    </case>
     <case id="sup" title="&gt;">
-    <doublevalue id="FaceSize__value"
-                  label="Size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="FaceSize__value"
+                   label="Size"
+                   min="0" step="0.1"
+                   default="1"
+                   tooltip="&gt; to">
         <validator id="GeomValidators_Positive" parameters="-1.e-10"/>
       </doublevalue>
-     </case>
+    </case>
     <case id="supEq" title="&gt;=">
-    <doublevalue id="FaceSize__value"
-                  label="Size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="FaceSize__value"
+                   label="Size"
+                   min="0" step="0.1"
+                   default="1"
+                   tooltip="&gt;= to">
         <validator id="GeomValidators_Positive" parameters="-1.e-10"/>
       </doublevalue>
-     </case>
+    </case>
     <case id="isBetween" title="is between">
-     <doublevalue id="FaceSize__value"
-                  label="Min size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="FaceSize__value"
+                   label="Min size"
+                   min="0" step="0.1"
+                   default="1"
+                   tooltip="Min value">
         <validator id="GeomValidators_Positive" parameters="-1.e-10"/>
       </doublevalue>
       <doublevalue id="FaceSize__valueMax"
-                  label="Max size"
-                  min="0"
-                  step="0.1"
-                  default="10">
+                   label="Max size"
+                   min="0" step="0.1"
+                   default="10"
+                   tooltip="Max value">
         <validator id="GeomValidators_Positive"/>
       </doublevalue>
     </case>
     <case id="isStrictlyBetween" title="is strictly between">
-     <doublevalue id="FaceSize__value"
-                  label="Min size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="FaceSize__value"
+                   label="Min size"
+                   min="0" step="0.1"
+                   default="1"
+                   tooltip="Min value">
         <validator id="GeomValidators_Positive" parameters="-1.e-10"/>
       </doublevalue>
       <doublevalue id="FaceSize__valueMax"
-                  label="Max size"
-                  min="0"
-                  step="0.1"
-                  default="10">
+                   label="Max size"
+                   min="0" step="0.1"
+                   default="10"
+                   tooltip="Max value">
         <validator id="GeomValidators_Positive"/>
       </doublevalue>
     </case>
diff --git a/src/FiltersPlugin/filter-FeatureEdges.xml b/src/FiltersPlugin/filter-FeatureEdges.xml
index c4e9071e4..a96933f01 100644
--- a/src/FiltersPlugin/filter-FeatureEdges.xml
+++ b/src/FiltersPlugin/filter-FeatureEdges.xml
@@ -1,8 +1,9 @@
 <filter id="FeatureEdges">
   <doublevalue id="FeatureEdges__value"
-              label="Angle"
-              min="0"
-              step="0.1"
-              default="5">
+               label="Angle"
+               min="0"
+               step="0.1"
+               default="5"
+               tooltip="Edge angle">
   </doublevalue>
 </filter>
diff --git a/src/FiltersPlugin/filter-VolumeSize.xml b/src/FiltersPlugin/filter-VolumeSize.xml
index fcb332097..cb89879c4 100644
--- a/src/FiltersPlugin/filter-VolumeSize.xml
+++ b/src/FiltersPlugin/filter-VolumeSize.xml
@@ -1,69 +1,70 @@
 <filter id="VolumeSize">
-    <switch id="VolumeSize__comparatorType">
+  <switch id="VolumeSize__comparatorType">
     <case id="inf" title="&lt;">
       <doublevalue id="VolumeSize__value"
-                  label="Size"
-                  min="0"
-                  default="1">
+                   label="Size"
+                   min="0"
+                   default="1"
+                   tooltip="< to">
         <validator id="GeomValidators_Positive" />
       </doublevalue>
-    </case> 
+    </case>
     <case id="infEq" title="&lt;=">
-    <doublevalue id="VolumeSize__value"
-                  label="Size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="VolumeSize__value"
+                   label="Size"
+                   min="0" step="0.1"
+                   default="1"
+                   tooltip="<= to">
         <validator id="GeomValidators_Positive"/>
       </doublevalue>
-     </case>
+    </case>
     <case id="sup" title="&gt;">
-    <doublevalue id="VolumeSize__value"
-                  label="Size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="VolumeSize__value"
+                   label="Size"
+                   min="0" step="0.1"
+                   default="1"
+                   tooltip="> to">
         <validator id="GeomValidators_Positive" parameters="-1.e-10"/>
       </doublevalue>
-     </case>
+    </case>
     <case id="supEq" title="&gt;=">
-    <doublevalue id="VolumeSize__value"
-                  label="Size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="VolumeSize__value"
+                   label="Size"
+                   min="0" step="0.1"
+                   default="1"
+                   tooltip="&gt;="> to">
         <validator id="GeomValidators_Positive" parameters="-1.e-10"/>
       </doublevalue>
-     </case>
+    </case>
     <case id="isBetween" title="is between">
-     <doublevalue id="VolumeSize__value"
-                  label="Min size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="VolumeSize__value"
+                   label="Min size"
+                   min="0" step="0.1"
+                   default="1"
+                   tooltip="Min value">
         <validator id="GeomValidators_Positive" parameters="-1.e-10"/>
       </doublevalue>
       <doublevalue id="VolumeSize__valueMax"
-                  label="Max size"
-                  min="0"
-                  step="0.1"
-                  default="10">
+                   label="Max size"
+                   min="0" step="0.1"
+                   default="10"
+                   tooltip="Max value">
         <validator id="GeomValidators_Positive"/>
       </doublevalue>
     </case>
     <case id="isStrictlyBetween" title="is strictly between">
-     <doublevalue id="VolumeSize__value"
-                  label="Min size"
-                  min="0"
-                  step="0.1"
-                  default="1">
+      <doublevalue id="VolumeSize__value"
+                   label="Min size"
+                   min="0" step="0.1"
+                   default="1"
+                   tooltip="Min value">
         <validator id="GeomValidators_Positive" parameters="-1.e-10"/>
       </doublevalue>
       <doublevalue id="VolumeSize__valueMax"
-                  label="Max size"
-                  min="0"
-                  step="0.1"
-                  default="10">
+                   label="Max size"
+                   min="0" step="0.1"
+                   default="10"
+                   tooltip="Max value">
         <validator id="GeomValidators_Positive"/>
       </doublevalue>
     </case>
diff --git a/src/GeomAlgoAPI/GeomAlgoAPI_BasicProperties.cpp b/src/GeomAlgoAPI/GeomAlgoAPI_BasicProperties.cpp
index 174929da1..123370f24 100644
--- a/src/GeomAlgoAPI/GeomAlgoAPI_BasicProperties.cpp
+++ b/src/GeomAlgoAPI/GeomAlgoAPI_BasicProperties.cpp
@@ -26,12 +26,12 @@
 #include <Standard_ErrorHandler.hxx>
 
 //=================================================================================================
-bool GetBasicProperties(  const std::shared_ptr<GeomAPI_Shape>& theShape,
-                          const double theTolerance,
-                          Standard_Real& theLength,
-                          Standard_Real& theSurfArea,
-                          Standard_Real& theVolume,
-                          std::string& theError)
+bool GetBasicProperties(const std::shared_ptr<GeomAPI_Shape>& theShape,
+                        const double theTolerance,
+                        Standard_Real& theLength,
+                        Standard_Real& theSurfArea,
+                        Standard_Real& theVolume,
+                        std::string& theError)
 {
 
   #ifdef _DEBUG
@@ -69,7 +69,5 @@ bool GetBasicProperties(  const std::shared_ptr<GeomAPI_Shape>& theShape,
     theError = aFail.GetMessageString();
     return false;
   }
-
   return true;
-
 }
diff --git a/src/GeomAlgoAPI/GeomAlgoAPI_ContinuousFaces.cpp b/src/GeomAlgoAPI/GeomAlgoAPI_ContinuousFaces.cpp
index a7507e432..436c64d85 100644
--- a/src/GeomAlgoAPI/GeomAlgoAPI_ContinuousFaces.cpp
+++ b/src/GeomAlgoAPI/GeomAlgoAPI_ContinuousFaces.cpp
@@ -109,18 +109,17 @@ bool isContinuousFaces(const GeomShapePtr& theFace1,
   try {
     OCC_CATCH_SIGNALS;
     LocalAnalysis_SurfaceContinuity aLocAnal(aSurf1,
-                                            aPointOnFace1.X(),
-                                            aPointOnFace1.Y(),
-                                            aSurf2,
-                                            aPointOnFace2.X(),
-                                            aPointOnFace2.Y(),
-                                            GeomAbs_Shape::GeomAbs_G1, // Order
-                                            0.001, // EpsNul
-                                            0.001, // EpsC0
-                                            0.001, // EpsC1
-                                            0.001, // EpsC2
-                                            theAngle * PI/ 180.0); //EpsG1
-
+                                             aPointOnFace1.X(),
+                                             aPointOnFace1.Y(),
+                                             aSurf2,
+                                             aPointOnFace2.X(),
+                                             aPointOnFace2.Y(),
+                                             GeomAbs_Shape::GeomAbs_G1, // Order
+                                             0.001, // EpsNul
+                                             0.001, // EpsC0
+                                             0.001, // EpsC1
+                                             0.001, // EpsC2
+                                             theAngle * PI/ 180.0); //EpsG1
     aRes = aLocAnal.IsG1();
   }
   catch (Standard_Failure const& anException) {
diff --git a/src/GeomValidators/GeomValidators_Positive.cpp b/src/GeomValidators/GeomValidators_Positive.cpp
index cfb033f31..4d5527eba 100644
--- a/src/GeomValidators/GeomValidators_Positive.cpp
+++ b/src/GeomValidators/GeomValidators_Positive.cpp
@@ -79,6 +79,5 @@ bool GeomValidators_Positive::isValid(const AttributePtr& theAttribute,
       return false;
     }
   }
-
   return true;
 }