0023947: Eliminate trivial compiler warnings in MSVC++ with warning level 4
authoromy <omy@opencascade.com>
Fri, 12 Jul 2013 08:37:38 +0000 (12:37 +0400)
committeromy <omy@opencascade.com>
Fri, 12 Jul 2013 08:37:38 +0000 (12:37 +0400)
Fix first subset of warnings (about 100)
- while(1) and similar constructs replaced by for(;;)
- some uninitialized variables corrected
Got rid of Warning C4189: local variable is initialised but not referenced
Corrected mistakes after getting rid of C4189 compiler warning
Corrected some mistakes that led to compiling errors
Fixed test case because of improvement message - removed unnecessary TODO.
Small fix: tabs have been replaced with whitespaces.
Added TODO for Windows platform
removed last TODO
Corrected mistakes, returned some #ifdef DEB code, fixed test case .
Restoring a few places which have been modified too much
Small grammar fix
Deleted unnecessary puts in bugs/end

189 files changed:
src/Adaptor3d/Adaptor3d_IsoCurve.cxx
src/Adaptor3d/Adaptor3d_SurfaceOfLinearExtrusion.cxx
src/BOPAlgo/BOPAlgo_ArgumentAnalyzer.cxx
src/BOPAlgo/BOPAlgo_BuilderFace.cxx
src/BOPAlgo/BOPAlgo_BuilderSolid.cxx
src/BOPAlgo/BOPAlgo_Builder_2.cxx
src/BOPAlgo/BOPAlgo_PaveFiller_3.cxx
src/BOPAlgo/BOPAlgo_PaveFiller_6.cxx
src/BOPAlgo/BOPAlgo_Tools.cxx
src/BOPAlgo/BOPAlgo_WireSplitter.cxx
src/BOPCol/BOPCol_Array1.hxx
src/BOPDS/BOPDS_DS.cxx
src/BOPDS/BOPDS_SubIterator.cxx
src/BOPTools/BOPTools_AlgoTools.cxx
src/BOPTools/BOPTools_AlgoTools_2.cxx
src/BRep/BRep_Builder.cxx
src/BRepAlgo/BRepAlgo_Loop.cxx
src/BRepBuilderAPI/BRepBuilderAPI_Collect.cxx
src/BRepClass3d/BRepClass3d_SolidExplorer.cxx
src/BRepFeat/BRepFeat.cxx
src/BRepFeat/BRepFeat_MakeLinearForm.cxx
src/BRepFeat/BRepFeat_MakeRevolutionForm.cxx
src/BRepFill/BRepFill_CompatibleWires.cxx
src/BRepFill/BRepFill_OffsetWire.cxx
src/BRepFill/BRepFill_TrimShellCorner.cxx
src/BRepLib/BRepLib_FindSurface.cxx
src/BRepMesh/BRepMesh_Delaun.cxx
src/BRepMesh/BRepMesh_VertexTool.cxx
src/BRepOffset/BRepOffset_MakeOffset.cxx
src/BRepOffset/BRepOffset_Offset.cxx
src/BRepOffsetAPI/BRepOffsetAPI_MiddlePath.cxx
src/BRepTest/BRepTest_SurfaceCommands.cxx
src/BiTgte/BiTgte_Blend.cxx
src/BndLib/BndLib_AddSurface.cxx
src/ChFi3d/ChFi3d_Builder_0.cxx
src/ChFi3d/ChFi3d_Builder_6.cxx
src/DDataStd/DDataStd_BasicCommands.cxx
src/DDataStd/DDataStd_Sample.cxx
src/DNaming/DNaming_BooleanOperationDriver.cxx
src/DPrsStd/DPrsStd_Sample.cxx
src/Draw/Draw_BasicCommands.cxx
src/Draw/Draw_VariableCommands.cxx
src/Draw/Draw_Window.cxx
src/Extrema/Extrema_FuncExtPS.cxx
src/Extrema/Extrema_GenExtPS.cxx
src/FSD/FSD_CmpFile.cxx
src/FSD/FSD_File.cxx
src/FWOSDriver/FWOSDriver_Driver.cxx
src/GeomFill/GeomFill_NSections.cxx
src/GeometryTest/GeometryTest_CurveCommands.cxx
src/HLRTopoBRep/HLRTopoBRep_OutLiner.cxx
src/IFSelect/IFSelect_SessionFile.cxx
src/IGESFile/igesread.c
src/IntAna/IntAna_Curve.cxx
src/IntPatch/IntPatch_ALineToWLine.cxx
src/IntPatch/IntPatch_PrmPrmIntersection.cxx
src/IntPolyh/IntPolyh_MaillageAffinage.cxx
src/IntTools/IntTools_BeanBeanIntersector.cxx
src/IntTools/IntTools_BeanFaceIntersector.cxx
src/IntTools/IntTools_EdgeEdge.cxx
src/IntTools/IntTools_EdgeEdge_1.cxx
src/IntTools/IntTools_EdgeFace.cxx
src/IntTools/IntTools_FClass2d.cxx
src/IntTools/IntTools_FaceFace.cxx
src/LDOM/LDOMParser.cxx
src/LDOM/LDOM_CharReference.cxx
src/LDOM/LDOM_Element.cxx
src/LDOM/LDOM_XmlReader.cxx
src/Message/Message_MsgFile.cxx
src/NCollection/NCollection_IncAllocator.cxx
src/OSD/OSD.cxx
src/OSD/OSD_DirectoryIterator.cxx
src/OSD/OSD_Disk.cxx
src/OSD/OSD_File.cxx
src/OSD/OSD_FileIterator.cxx
src/OSD/OSD_FileNode.cxx
src/OSD/OSD_Host.cxx
src/OSD/OSD_MAllocHook.cxx
src/OSD/OSD_Path.cxx
src/OSD/OSD_Process.cxx
src/OSD/OSD_SharedLibrary.cxx
src/OSD/OSD_WNT.cxx
src/OSD/OSD_signal_WNT.cxx
src/OpenGl/OpenGl_LayerList.cxx
src/OpenGl/OpenGl_Trihedron.cxx
src/OpenGl/OpenGl_View_2.cxx
src/Prs3d/Prs3d_RadiusAspect.cxx
src/QABugs/QABugs_1.cxx
src/QABugs/QABugs_11.cxx
src/QABugs/QABugs_13.cxx
src/QABugs/QABugs_2.cxx
src/QABugs/QABugs_3.cxx
src/QABugs/QABugs_9.cxx
src/QANewBRepNaming/QANewBRepNaming_Gluing.cxx
src/QANewBRepNaming/QANewBRepNaming_Limitation.cxx
src/QANewDBRepNaming/QANewDBRepNaming.cxx
src/QANewModTopOpe/QANewModTopOpe.cxx
src/QANewModTopOpe/QANewModTopOpe_Intersection.cxx
src/QANewModTopOpe/QANewModTopOpe_Tools.cxx
src/RWHeaderSection/RWHeaderSection_GeneralModule.cxx
src/RWStepAP214/RWStepAP214_RWAppliedApprovalAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAppliedDateAndTimeAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAppliedDateAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAppliedDocumentReference.cxx
src/RWStepAP214/RWStepAP214_RWAppliedOrganizationAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAppliedPersonAndOrganizationAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAppliedSecurityClassificationAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAutoDesignActualDateAndTimeAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAutoDesignActualDateAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAutoDesignApprovalAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAutoDesignDateAndPersonAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAutoDesignDocumentReference.cxx
src/RWStepAP214/RWStepAP214_RWAutoDesignNominalDateAndTimeAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAutoDesignNominalDateAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAutoDesignOrganizationAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAutoDesignPersonAndOrganizationAssignment.cxx
src/RWStepAP214/RWStepAP214_RWAutoDesignSecurityClassificationAssignment.cxx
src/RWStepBasic/RWStepBasic_RWApplicationContextElement.cxx
src/Resource/Resource_Manager.cxx
src/STEPCAFControl/STEPCAFControl_Reader.cxx
src/Select3D/Select3D_SensitiveEntity.cxx
src/ShapeAnalysis/ShapeAnalysis_FreeBounds.cxx
src/ShapeFix/ShapeFix_ComposeShell.cxx
src/Standard/Standard.cxx
src/Standard/Standard_MMgrOpt.cxx
src/StdPrs/StdPrs_Curve.cxx
src/StdPrs/StdPrs_WFRestrictedFace.cxx
src/StepBasic/StepBasic_SiUnit.cxx
src/StepData/StepData_DefaultGeneral.cxx
src/StepData/StepData_DescrGeneral.cxx
src/StepData/StepData_PDescr.cxx
src/StepData/StepData_Simple.cxx
src/StepData/StepData_StepReaderData.cxx
src/StepShape/StepShape_OrientedClosedShell.cxx
src/StepShape/StepShape_OrientedFace.cxx
src/StepShape/StepShape_OrientedOpenShell.cxx
src/Storage/Storage_Schema.cxx
src/TColStd/TColStd_MapIteratorOfPackedMapOfInteger.cxx
src/TColStd/TColStd_PackedMapOfInteger.cxx
src/TCollection/TCollection_AsciiString.cxx
src/TDF/TDF_Attribute.cxx
src/TDataStd/TDataStd_Directory.cxx
src/TDataStd/TDataStd_NoteBook.cxx
src/TDataXtd/TDataXtd_Shape.cxx
src/TFunction/TFunction_Function.cxx
src/TNaming/TNaming_UsedShapes.cxx
src/TPrsStd/TPrsStd_ConstraintTools.cxx
src/TestTopOpe/FILES
src/TestTopOpe/TestTopOpe.cxx
src/TestTopOpe/TestTopOpe_CORCommands.cxx
src/TestTopOpe/TestTopOpe_DSACommands.cxx [deleted file]
src/TestTopOpeDraw/TestTopOpeDraw_OtherCommands.cxx
src/TopOpeBRep/TopOpeBRep_EdgesFiller.cxx
src/TopOpeBRep/TopOpeBRep_EdgesIntersector.cxx
src/TopOpeBRep/TopOpeBRep_FaceEdgeFiller.cxx
src/TopOpeBRep/TopOpeBRep_FacesFiller_1.cxx
src/TopOpeBRep/TopOpeBRep_Hctxee2d.cxx
src/TopOpeBRep/TopOpeBRep_Point2d.cxx
src/TopOpeBRep/TopOpeBRep_ProcessGR.cxx
src/TopOpeBRep/TopOpeBRep_VPointInterIterator.cxx
src/TopOpeBRepTool/TopOpeBRepTool_CORRISO.cxx
src/TopOpeBRepTool/TopOpeBRepTool_TOPOLOGY.cxx
src/TopOpeBRepTool/TopOpeBRepTool_tol.cxx
src/V3d/V3d_View.cxx
src/V3d/V3d_View_3.cxx
src/ViewerTest/ViewerTest_OpenGlCommands.cxx
src/ViewerTest/ViewerTest_ViewerCommands.cxx
src/Voxel/Voxel_FastConverter.cxx
src/VrmlData/VrmlData_Geometry.cxx
src/VrmlData/VrmlData_Scene.cxx
src/VrmlData/VrmlData_ShapeConvert.cxx
src/WNT/EHDC.cxx
src/WNT/WNT_Window.cxx
src/XSDRAW/XSDRAW.cxx
src/XSDRAWSTEP/XSDRAWSTEP.cxx
src/XmlMDataStd/XmlMDataStd_IntegerArrayDriver.cxx
src/XmlMDataStd/XmlMDataStd_NamedDataDriver.cxx
src/XmlMDataStd/XmlMDataStd_RealArrayDriver.cxx
src/XmlObjMgt/XmlObjMgt.cxx
src/gp/gp_GTrsf.cxx
src/gp/gp_GTrsf2d.cxx
src/gp/gp_Mat.cxx
src/gp/gp_Mat2d.cxx
src/gp/gp_Trsf.cxx
src/gp/gp_Trsf2d.cxx
src/math/math_DirectPolynomialRoots.cxx
tests/bugs/begin
tests/bugs/end
tests/bugs/modalg_1/buc60462_2

index f083f24..d2df8c3 100755 (executable)
@@ -805,7 +805,6 @@ gp_Elips Adaptor3d_IsoCurve::Ellipse() const
   default:  
     {
       Standard_NoSuchObject::Raise("Adaptor3d_IsoCurve:Ellipse");
-      return gp_Elips();
     }
     
   }
index d61b05c..9855ab5 100755 (executable)
@@ -492,9 +492,6 @@ GeomAbs_SurfaceType Adaptor3d_SurfaceOfLinearExtrusion::GetType() const
     return GeomAbs_SurfaceOfExtrusion;
 
   }
-  
-  // portage WNT
-  return GeomAbs_SurfaceOfExtrusion;
 }
 
 //=======================================================================
index 65cc142..44349ab 100644 (file)
@@ -333,7 +333,6 @@ void BOPAlgo_ArgumentAnalyzer::TestSelfInterferences()
     Standard_Integer aNb[6] = {aVVs.Extent(), aVEs.Extent(), aEEs.Extent(), 
                                aVFs.Extent(), aEFs.Extent(), aFFs.Extent()};
     //
-    Standard_Integer ind = 0;
     for (Standard_Integer aTypeInt = 0; aTypeInt < 6; ++aTypeInt) {
       for (Standard_Integer i = 0; i < aNb[aTypeInt]; ++i) {
         BOPDS_Interf* aInt = (aTypeInt==0) ? (BOPDS_Interf*)(&aVVs(i)) : 
index 9794138..5ad003e 100644 (file)
@@ -182,7 +182,7 @@ static
   myShapesToAvoid.Clear();
   //
   iCnt=0;
-  while (1) {
+  for(;;) {
     ++iCnt;
     bFound=Standard_False;
     //
index c873bd8..690ac9e 100644 (file)
@@ -166,7 +166,7 @@ static
   myShapesToAvoid.Clear();
   //
   iCnt=0;
-  while (1) {
+  for(;;) {
     ++iCnt;
     bFound=Standard_False;
     //
@@ -178,9 +178,6 @@ static
       if (!myShapesToAvoid.Contains(aF)) {
         BOPTools::MapShapesAndAncestors(aF, TopAbs_EDGE, TopAbs_FACE, aMEF);
       }
-      else {
-        int a=0;
-      }
     }
     aNbE=aMEF.Extent();
     //
index bba4e8a..54f2a9a 100644 (file)
@@ -570,7 +570,7 @@ void MakeBlocksCnx(const BOPCol_IndexedDataMapOfShapeListOfShape& aMILI,
     aMVAdd.Clear();
     //
     aMVP.Add(nV);
-    while(1) {
+    for(;;) {
       aNbVP=aMVP.Extent();
       for (j=1; j<=aNbVP; ++j) {
         const TopoDS_Shape& nVP=aMVP(j);
index 88b3cc3..833259a 100644 (file)
     BOPCol_MapIteratorOfMapOfInteger aIt1;
     //
     aMIP.Add(i);
-    while(1) {
+    for(;;) {
       aNbIP=aMIP.Extent();
       aIt1.Initialize(aMIP);
       for(; aIt1.More(); aIt1.Next()) {
index 9e5522c..ea639d6 100644 (file)
@@ -327,8 +327,6 @@ static void ToleranceFF(const TopoDS_Face& aF1,
     for (j=0; j<aNbC; ++j) {
       BOPDS_Curve& aNC=aVC.ChangeValue(j);
       const IntTools_Curve& aIC=aNC.Curve();
-      // DEBf
-      const Handle(Geom_Curve)& aC3D=aIC.Curve();
       // DEBt
       aNC.InitPaveBlock1();
       //
@@ -798,9 +796,6 @@ static void ToleranceFF(const TopoDS_Face& aF1,
     if (aSI.ShapeType()!=TopAbs_FACE) {
       continue;
     }
-    //
-    const TopoDS_Face& aF=(*(TopoDS_Face*)(&aSI.Shape()));
-    //
     if(!myDS->HasFaceInfo(i)) {
       continue;
     }
index 00f08c9..7c49afa 100644 (file)
@@ -59,7 +59,7 @@
     aMVAdd.Clear();
     //
     aMVP.Add(nV);
-    while(1) {
+    for(;;) {
       aNbVP=aMVP.Extent();
       for (j=1; j<=aNbVP; ++j) {
         nVP=aMVP(j);
     aMVAdd.Clear();
     //
     aMVP.Add(nV);
-    while(1) {
+    for(;;) {
       aNbVP=aMVP.Extent();
       for (j=1; j<=aNbVP; ++j) {
         const Handle(BOPDS_PaveBlock)& nVP=aMVP(j);
index a2ff1b9..0394e42 100644 (file)
     //
     aMVP.Add(aV);
     //
-    while(1) {
+    for(;;) {
       aNbVP=aMVP.Extent();
       for (k=1; k<=aNbVP; ++k) {
         const TopoDS_Shape& aVP=aMVP(k);
index df9b868..a9b4dd7 100644 (file)
@@ -284,7 +284,7 @@ template <class Type>
     BOPCol_PMemBlock pBlock, pNext;
     //
     pBlock=myPBlock1;
-    while(1) {
+    for(;;) {
       pNext=pBlock->myNext;
       if(!pNext) {
         myPBlock=pBlock;
@@ -321,7 +321,7 @@ template <class Type>
     BOPCol_PMemBlock pBlock, pNext;
     //
     pBlock=myPBlock1;
-    while(1) {
+    for(;;) {
       pNext=pBlock->myNext;
       if(!pNext) {
         myPBlock=pBlock;
index 57c1564..9a7cefd 100644 (file)
@@ -1379,7 +1379,7 @@ static
   }
   //
   const BOPCol_MapOfInteger& aMVIn1=aFI1.VerticesIn();
-  aIt.Initialize(aMVOn1);
+  aIt.Initialize(aMVIn1);
   for (; aIt.More(); aIt.Next()) {
     nV=aIt.Value();
     aMI.Add(nV);
@@ -1393,7 +1393,7 @@ static
   }
   //
   const BOPCol_MapOfInteger& aMVIn2=aFI2.VerticesIn();
-  aIt.Initialize(aMVOn1);
+  aIt.Initialize(aMVIn2);
   for (; aIt.More(); aIt.Next()) {
     nV=aIt.Value();
     aMI.Add(nV);
index e10c1fe..0412917 100644 (file)
     i=aIt2.Value();
     //
     const BOPDS_ShapeInfo& aSI=myDS->ShapeInfo(i);
-    const TopoDS_Shape& aSi=aSI.Shape();
     const Bnd_Box& aBoxEx=aSI.Box();
     //
     aSelector.Clear();
index 893c57d..c6e3bf3 100644 (file)
@@ -140,7 +140,7 @@ static
     aMAdd.Clear();
     aMAdd.Add(aF1);
     //
-    while(1) {
+    for(;;) {
       aMAdd1.Clear();
       //
       aNbAdd = aMAdd.Extent();
@@ -374,7 +374,7 @@ static
   const TopoDS_Shape& aF1=theLFIn.First();
   aMAdd.Add(aF1);
   //
-  while(1) {
+  for(;;) {
     aMAdd1.Clear();
     aNbAdd = aMAdd.Extent();
     for (i=1; i<=aNbAdd; ++i) {
index db27bb5..e57f762 100644 (file)
@@ -477,7 +477,7 @@ static
   TopoDS_Iterator aItC;
   //
   aLC.Append (theC1);
-  while(1) {
+  for(;;) {
     aLC1.Clear();
     aIt.Initialize(aLC);
     for (; aIt.More(); aIt.Next()) {
index 333727e..5ab0f4f 100755 (executable)
@@ -115,10 +115,6 @@ static void UpdateCurves(BRep_ListOfCurveRepresentation& lcr,
     GC = Handle(BRep_GCurve)::DownCast(itcr.Value());
     if (!GC.IsNull()) {
       if (GC->IsCurve3D()) {
-#ifdef DEB
-        const Handle(Geom_Curve)& Crv =
-#endif
-                                      GC->Curve3D();
 //      if (!C.IsNull()) { //xpu031198, edge degeneree
 
         // xpu151298 : parameters can be setted for null curves
@@ -185,10 +181,6 @@ static void UpdateCurves(BRep_ListOfCurveRepresentation& lcr,
     GC = Handle(BRep_GCurve)::DownCast(itcr.Value());
     if (!GC.IsNull()) {
       if (GC->IsCurve3D()) {
-#ifdef DEB
-        const Handle(Geom_Curve)& Crv =
-#endif
-                                      GC->Curve3D();
 //      if (!C.IsNull()) { //xpu031198, edge degeneree
 
         // xpu151298 : parameters can be setted for null curves
index 9021eca..c24f6d2 100755 (executable)
@@ -532,7 +532,6 @@ void BRepAlgo_Loop::Perform()
     }
   }
 #endif
-  
   //------------------------------------------------
   // Cut edges
   //------------------------------------------------
@@ -573,11 +572,11 @@ void BRepAlgo_Loop::Perform()
     TopTools_MapOfShape Done;
     for (Mapit.Initialize(MVE); Mapit.More();Mapit.Next()) {
       for (itl.Initialize(Mapit.Value()); itl.More(); itl.Next()) {
-       TopoDS_Edge& E = TopoDS::Edge(itl.Value());
-       if (Done.Add(E)) {
-         sprintf(name,"EEC_%d_%d",NbLoops,NbEdges++);
-         DBRep::Set(name,E);
-       }
+        TopoDS_Edge& E = TopoDS::Edge(itl.Value());
+        if (Done.Add(E)) {
+          sprintf(name,"EEC_%d_%d",NbLoops,NbEdges++);
+          DBRep::Set(name,E);
+        }
       }
     }
   }
@@ -633,14 +632,14 @@ void BRepAlgo_Loop::Perform()
       UsedEdges.Add(CE);
 
       if (!MVE.IsBound(CV) || MVE(CV).IsEmpty()) {
-       End = Standard_True;
+        End = Standard_True;
       }
       else {
-       End = !SelectEdge(myFace,CE,CV,NE,MVE(CV));
-       if (!End) {
-         CE = NE;
-         if (MVE(CV).IsEmpty()) MVE.UnBind(CV);
-       }       
+        End = !SelectEdge(myFace,CE,CV,NE,MVE(CV));
+        if (!End) {
+          CE = NE;
+          if (MVE(CV).IsEmpty()) MVE.UnBind(CV);
+        }
       }
     }
     //--------------------------------------------------
@@ -650,28 +649,28 @@ void BRepAlgo_Loop::Perform()
     TopExp_Explorer explo( NW, TopAbs_VERTEX );
     for (; explo.More(); explo.Next())
       {
-       const TopoDS_Vertex& aV = TopoDS::Vertex( explo.Current() );
-       Handle(BRep_TVertex)& TV = *((Handle(BRep_TVertex)*) &(aV).TShape());
-       TV->Tolerance( Tol );
-       TV->Modified( Standard_True );
+      const TopoDS_Vertex& aV = TopoDS::Vertex( explo.Current() );
+      Handle(BRep_TVertex)& TV = *((Handle(BRep_TVertex)*) &(aV).TShape());
+      TV->Tolerance( Tol );
+      TV->Modified( Standard_True );
       }
     for (explo.Init( NW, TopAbs_EDGE ); explo.More(); explo.Next())
       {
-       const TopoDS_Edge& aE = TopoDS::Edge( explo.Current() );
-       Handle(BRep_TEdge)& TE = *((Handle(BRep_TEdge)*) &(aE).TShape());
-       TE->Tolerance( Tol );
-       TE->Modified( Standard_True );
+      const TopoDS_Edge& aE = TopoDS::Edge( explo.Current() );
+      Handle(BRep_TEdge)& TE = *((Handle(BRep_TEdge)*) &(aE).TShape());
+      TE->Tolerance( Tol );
+      TE->Modified( Standard_True );
       }
 
     if (VF.IsSame(CV) && SamePnt2d(VF,EF,CE,myFace))
       myNewWires.Append (NW);
-    else {
 #ifdef DEB
+    else {
       cout <<"BRepAlgo_Loop: Open Wire"<<endl;
       if (AffichLoop)
-       cout << "OpenWire is : NW_"<<NbLoops<<"_"<<NbWires<<endl;
+        cout << "OpenWire is : NW_"<<NbLoops<<"_"<<NbWires<<endl;
+      }
 #endif
-    }
 #ifdef DRAW
     if (AffichLoop) {
       sprintf(name,"NW_%d_%d",NbLoops,NbWires++);      
@@ -694,7 +693,6 @@ void BRepAlgo_Loop::CutEdge (const TopoDS_Edge&          E,
 {
   TopoDS_Shape aLocalE  = E.Oriented(TopAbs_FORWARD);
   TopoDS_Edge WE = TopoDS::Edge(aLocalE);
-//  TopoDS_Edge WE = TopoDS::Edge(E.Oriented(TopAbs_FORWARD));
 
   Standard_Real                      U1,U2;
   TopoDS_Vertex                      V1,V2;
index 4500f92..893287c 100755 (executable)
@@ -186,7 +186,6 @@ static void DEBControl (const TopTools_DataMapOfShapeListOfShape& MG)
     TopTools_ListIteratorOfListOfShape itl(MG(OS));
     Standard_Integer IV = 1;
     for (; itl.More(); itl.Next()) {
-      const TopoDS_Shape& NS = itl.Value();
       sprintf(name, "SV_%d_%d",IK,IV++);
 #ifdef DRAW
       DBRep::Set(name,NS);
index f7205a1..dedeb9b 100755 (executable)
@@ -410,7 +410,7 @@ Standard_Integer BRepClass3d_SolidExplorer::OtherSegment(const gp_Pnt& P,
   Standard_Integer NbPointsOK=0;
   Standard_Integer NbFacesInSolid=0;
 
-  do { 
+  for(;;) { 
     myFirstFace++; 
     faceexplorer.Init(myShape,TopAbs_FACE);
     // look for point on face starting from myFirstFace
@@ -583,8 +583,7 @@ Standard_Integer BRepClass3d_SolidExplorer::OtherSegment(const gp_Pnt& P,
     else { myParamOnEdge*=0.5; } 
     
     
-  } //-- do { ...  } 
-  while(1); 
+  } //-- for(;;) { ...  } 
   return 0;
 }
 
index 3211e3f..7d6940a 100755 (executable)
@@ -181,21 +181,16 @@ Standard_Real BRepFeat::ParametricBarycenter(const TopoDS_Shape& S,
       // On projette sur CC
       extpc.Perform(pone);
       if (extpc.IsDone() && extpc.NbExt() >= 1) {
-       Standard_Real Dist2Min = extpc.SquareDistance(1);
-       Standard_Integer kmin = 1;
-       for (Standard_Integer k=2; k<=extpc.NbExt(); k++) {
-         Standard_Real Dist2 = extpc.SquareDistance(k);
-         if (Dist2 < Dist2Min) {
-           Dist2Min = Dist2;
-           kmin = k;
-         }
-       }
-       nbp++;
-#ifdef DEB
-       Standard_Real prmp = extpc.Point(kmin).Parameter();
-#else
-        extpc.Point(kmin).Parameter();
-#endif
+           Standard_Real Dist2Min = extpc.SquareDistance(1);
+           Standard_Integer kmin = 1;
+           for (Standard_Integer k=2; k<=extpc.NbExt(); k++) {
+             Standard_Real Dist2 = extpc.SquareDistance(k);
+             if (Dist2 < Dist2Min) {
+               Dist2Min = Dist2;
+               kmin = k;
+             }
+           }
+           nbp++;
       }          
     }
   }
index c47e524..6111b0d 100755 (executable)
@@ -1198,10 +1198,6 @@ static void MajMap(const TopoDS_Shape& theB,
   
   for (exp.Init(theB,TopAbs_EDGE); exp.More(); exp.Next()) {
     if (!theMap.IsBound(exp.Current())) {
-#ifdef DEB
-      const TopoDS_Edge& e = 
-#endif
-      TopoDS::Edge(exp.Current());
       TopTools_ListOfShape thelist2; 
       theMap.Bind(exp.Current(), thelist2);
       theMap(exp.Current()) = theP.Shapes(exp.Current());
index c4cb452..18daa46 100755 (executable)
@@ -1735,10 +1735,6 @@ static void MajMap(const TopoDS_Shape& theB,
 
   for (exp.Init(theB,TopAbs_EDGE); exp.More(); exp.Next()) {
     if (!theMap.IsBound(exp.Current())) {
-#ifdef DEB
-      const TopoDS_Edge& e = 
-#endif
-      TopoDS::Edge(exp.Current());
       TopTools_ListOfShape thelist2;
       theMap.Bind(exp.Current(), thelist2);
       theMap(exp.Current()) = theP.Shapes(exp.Current());
index 898325b..5233647 100755 (executable)
@@ -929,11 +929,6 @@ void BRepFill_CompatibleWires::
     
     // extremity of the first wire
     V1 = TopoDS::Vertex(SeqV.Value(1));        
-    // previous wire 
-#ifdef DEB
-    const TopoDS_Wire& wire2 = 
-#endif
-      TopoDS::Wire(myWork(i-1));
     // loop on vertices of wire1
     for (ii=1;ii<=SeqV.Length();ii++) {
       
index 7c6c8bf..27d2a0c 100755 (executable)
@@ -533,9 +533,8 @@ void BRepFill_OffsetWire::Perform (const Standard_Real Offset,
        Handle(Geom_Curve) G3d = BRep_Tool::Curve(TopoDS::Edge(anE),f,l);
        GeomAdaptor_Curve  AC(G3d,f,l);
 
-       Standard_Boolean dummy = PerformCurve(Parameters, Points,
-                                             AC, aDefl, f, l, Precision::Confusion(),
-                                             2);
+    PerformCurve(Parameters, Points, AC, aDefl, f, 
+                     l, Precision::Confusion(), 2);
 
        Standard_Integer NPnts = Points.Length();
        if(NPnts > 2) {
index 75e1088..984629c 100755 (executable)
@@ -851,7 +851,6 @@ Standard_Boolean SplitUEdges(const Handle(TopTools_HArray2OfShape)&     theUEdge
                              const BOPDS_PDS&                           theDS,
                              TopTools_DataMapOfShapeListOfShape&        theHistMap) {
 
-  const BOPDS_VectorOfInterfVE& aVEs = theDS->InterfVE();
   const BOPDS_VectorOfInterfVV& aVVs = theDS->InterfVV();
 
   BRep_Builder aBB;
index e3cfbf0..34cfe56 100755 (executable)
@@ -210,7 +210,7 @@ void BRepLib_FindSurface::Init(const TopoDS_Shape&    S,
   Standard_Integer i = 0,j;
 
   // iterate on the surfaces of the first edge
-  while ( Standard_True) {
+  for(;;) {
     i++;
     BRep_Tool::CurveOnSurface(E,PC,mySurface,myLocation,f,l,i);
     if (mySurface.IsNull()) {
@@ -220,7 +220,7 @@ void BRepLib_FindSurface::Init(const TopoDS_Shape&    S,
     for (ex.Init(S,TopAbs_EDGE); ex.More(); ex.Next()) {
       if (!E.IsSame(ex.Current())) {
        j = 0;
-       while (Standard_True) {
+       for(;;) {
          j++;
          BRep_Tool::CurveOnSurface(TopoDS::Edge(ex.Current()),
                                    PPC,SS,L,ff,ll,j);
index 4273224..6148b6c 100755 (executable)
@@ -761,7 +761,6 @@ void BRepMesh_Delaun::RemovePivotTriangles( const Standard_Integer theEdgeInfo,
     GetTriangle( anElemId ).Edges( e[0], e[1], e[2],
                                    o[0], o[1], o[2] );
 
-    Standard_Boolean isFind = Standard_False;
     for ( Standard_Integer anIndex = 0; anIndex < 3; ++anIndex )
     {
       if ( e[anIndex] == anMainEdgeId && o[anIndex] == anIsForward )
@@ -1128,7 +1127,6 @@ void BRepMesh_Delaun::MeshLeftPolygonOf( const Standard_Integer theEdgeIndex,
         return;
 
       Standard_Integer aDeadEdgeId = Abs( aPolygon.Last() );
-      const BRepMesh_Edge& aDeadEdge = GetEdge( aDeadEdgeId );
 
       aDealLinks.Add( aDeadEdgeId );
 
index d5dc46a..f488409 100644 (file)
@@ -169,7 +169,7 @@ const BRepMesh_ListOfInteger& BRepMesh_VertexTool::GetListOfDelNodes() const
 //=======================================================================
 Standard_Integer BRepMesh_VertexTool::FindIndex(const BRepMesh_Vertex& theVertex)
 {
-  mySelector.SetCurrent(theVertex.Coord());
+  mySelector.SetCurrent(theVertex.Coord(), Standard_False);
   myCellFilter.Inspect (theVertex.Coord(), mySelector);
   return mySelector.GetCoincidentInd();
 }
index cb14de6..0a5a42f 100755 (executable)
 static void DEBVerticesControl (const TopTools_IndexedMapOfShape& NewEdges,
                                      Handle(BRepAlgo_AsDes)      AsDes)
 {
-  Standard_Integer NVP = 0;
-  Standard_Integer NVM = 0;
-  Standard_Integer NVN = 0;
-
   TopTools_ListOfShape               LVP;
-  //TopTools_MapIteratorOfMapOfShape   it;
   TopTools_ListIteratorOfListOfShape it1LE ;    
   TopTools_ListIteratorOfListOfShape it2LE ;
   
-  //for (it.Initialize(NewEdges) ; it.More(); it.Next()) {
   Standard_Integer i;
   for (i = 1; i <= NewEdges.Extent(); i++) {
     const TopoDS_Edge& NE = TopoDS::Edge(NewEdges(i));
index f38da21..a2893ee 100755 (executable)
@@ -1313,7 +1313,6 @@ void BRepOffset_Offset::Init(const TopoDS_Vertex&        Vertex,
 #endif
     Standard_Integer NbEdges = 1;
     for (it.Initialize(LEdge); it.More(); it.Next()) {
-      const TopoDS_Edge& CurE = TopoDS::Edge(it.Value());
       sprintf(name,"EOnSph_%d_%d",NbOFFSET,NbEdges++);
 #ifdef DRAW
       DBRep::Set(name, CurE);
index e9bcb40..b9ae704 100644 (file)
@@ -531,8 +531,6 @@ void BRepOffsetAPI_MiddlePath::Build()
         E2 = TopoDS::Edge(myPaths((j<=NbPaths)? j : 1)(i));
       TopoDS_Edge E12 = TopoDS::Edge(SectionsEdges(i)(j-1));
       
-      //TopoDS_Vertex PrevVertex = TopoDS::Vertex(VerSeq(j-1));
-      //TopoDS_Vertex CurVertex  = TopoDS::Vertex(VerSeq(j));
       TopoDS_Vertex PrevVertex = (E1.IsNull())? TopoDS::Vertex(myPaths(j-1)(i))
         : TopExp::LastVertex(E1, Standard_True);
       TopoDS_Vertex CurVertex = (E2.IsNull())? TopoDS::Vertex(myPaths((j<=NbPaths)? j : 1)(i))
@@ -540,9 +538,7 @@ void BRepOffsetAPI_MiddlePath::Build()
       
       TopoDS_Edge ProperEdge;
       const TopTools_ListOfShape& LE = VEmap.FindFromKey(PrevVertex);
-      //Temporary
-      Standard_Integer LenList = LE.Extent();
-      ///////////
+
       for (itl.Initialize(LE); itl.More(); itl.Next())
       {
         anEdge = TopoDS::Edge(itl.Value());
@@ -673,7 +669,6 @@ void BRepOffsetAPI_MiddlePath::Build()
           BRepLib::BuildCurve3d(NewEdge2);
           Standard_Boolean good_ne  = IsValidEdge(NewEdge, theFace);
           Standard_Boolean good_ne1 = IsValidEdge(NewEdge1, theFace);
-          Standard_Boolean good_ne2 = IsValidEdge(NewEdge2, theFace);
 
           GeomAbs_CurveType type_E1 = TypeOfEdge(E1);
           GeomAbs_CurveType type_E2 = TypeOfEdge(E2);
index 9bbbc7b..60961b2 100755 (executable)
@@ -444,24 +444,6 @@ static Standard_Integer continuity (Draw_Interpretor& ,
   aFind.Perform();
   aFind.Dump();
 
-  for (i=1; i<=aFind.NbContigousEdges(); i++) {
-#ifdef DEB
-    const TopoDS_Edge& edge =
-#endif
-                              aFind.ContigousEdge(i);
-    const TopTools_ListOfShape& list = aFind.ContigousEdgeCouple(i);
-    const TopoDS_Edge& sec1 = TopoDS::Edge(list.First());
-    const TopoDS_Edge& sec2 = TopoDS::Edge(list.Last());
-#ifdef DEB
-    const TopoDS_Edge& bound1 =
-#endif
-                                aFind.SectionToBoundary(sec1);
-#ifdef DEB
-    const TopoDS_Edge& bound2 =
-#endif
-                                aFind.SectionToBoundary(sec2);
-  }
-
   return 0;
 }
 
index 1cafb07..860527d 100755 (executable)
@@ -485,7 +485,7 @@ static void Touched(const BRepOffset_Analyse&  Analyse,
                    TopTools_MapOfShape& TouchedByCork)
 {
   // currently nothing is done !!
-  if ( Standard_True) {
+  /*if ( Standard_True) {
     return;
   }
   else {
@@ -497,7 +497,8 @@ static void Touched(const BRepOffset_Analyse&  Analyse,
       else if (StopFaces.Contains(L.Last())) 
        TouchedByCork.Add(L.First());
     }
-  }
+  }*/
+  return;
 }
 
 //=======================================================================
index 0721007..6be1cfa 100755 (executable)
@@ -153,8 +153,6 @@ static void TreatInfinitePlane(const gp_Pln        &aPlane,
 {
   // Get 3 coordinate axes of the plane.
   const gp_Dir        &aNorm        = aPlane.Axis().Direction();
-  const gp_Dir        &aXDir        = aPlane.XAxis().Direction();
-  const gp_Dir        &aYDir        = aPlane.YAxis().Direction();
   const Standard_Real  anAngularTol = RealEpsilon();
 
   // Get location of the plane as its barycenter
index 7a6a48a..0ce5646 100755 (executable)
@@ -3301,7 +3301,7 @@ Standard_Boolean ChFi3d_ComputeCurves(Handle(Adaptor3d_HSurface)&   S1,
   // At this stage : 
   // classic intersections have failed, the path is approached in vain.
 //  Standard_Real Step = 0.1;
-  while(1) {
+  for(;;) {
     //Attention the parameters of arrow for the path and
     //the tolerance for the approximation can't be taken as those of the  
     //Builder, so they are reestimated as much as possible.
index 8a950f6..20bec9e 100755 (executable)
@@ -851,7 +851,7 @@ Standard_Boolean ChFi3d_Builder::StoreData(Handle(ChFiDS_SurfData)& Data,
   Standard_Real    aDelta = VLast - VFirst;
   Standard_Integer aDenom = 2;
 
-  while (Standard_True) {
+  for(;;) {
     Standard_Real aDeltav = aDelta/aDenom;
     Standard_Real aParam  = VFirst + aDeltav;
     gp_Pnt2d      PUV     = PCurveOnFace->Value(aParam);
index b2a763c..8a18ac3 100755 (executable)
@@ -960,7 +960,6 @@ static Standard_Integer DDataStd_GetExtStringArray (Draw_Interpretor& di,
   }
   
   for(Standard_Integer i = A->Lower(); i<=A->Upper(); i++){
-    const TCollection_ExtendedString& anExtendedString = A->Value(i);
     TCollection_AsciiString anAsciiString(A->Value(i),'?');
     di << anAsciiString.ToCString();
     if(i<A->Upper())  
@@ -1188,7 +1187,6 @@ static Standard_Integer DDataStd_SetBooleanArray (Draw_Interpretor& di,
 
     TDF_Label label;
     DDF::AddLabel(DF, arg[2], label);
-    Standard_Integer isDelta = Draw::Atoi(arg[3]);
     Standard_Integer From = Draw::Atoi(arg[4]), To = Draw::Atoi( arg[5] ), j;
     di << "Array of Standard_Boolean with bounds from = " << From  << " to = " << To  << "\n";
     Handle(TDataStd_BooleanArray) A = TDataStd_BooleanArray::Set(label, From, To);
index 187145b..26fac13 100755 (executable)
@@ -98,10 +98,6 @@ static void Sample()
     realarray->SetValue(j, M_PI * j); 
   }
 
-  // Getting length of array
-
-  Standard_Integer length = realarray->Length();
-
   // Retrieving value by index
 
   Standard_Real value = realarray->Value(3);
@@ -346,7 +342,6 @@ static void Sample()
 
    Handle(TDataStd_Real) valOfdistance = constraint->GetValue();
 
-   Standard_Real distance = valOfdistance->Get();
   }
 
   // Setting PARALLEL constraint between NS1 and NS2
index 3d4cd1a..bbc5b20 100644 (file)
@@ -351,7 +351,6 @@ Standard_Boolean DNaming_BooleanOperationDriver::CheckAndLoad
     if(!aCheck.IsValid(theMkOpe.Shape())) 
       aResIsValid = FixSameParameter(theMkOpe.Shape(), aCheck);
     if (aResIsValid) {
-      const TDF_Label& aResultLabel = RESPOSITION(theFunction);
       if(theFunction->GetDriverGUID() == FUSE_GUID) {
        LoadNamingDS(RESPOSITION(theFunction), theMkOpe);
       }
index 97ef2d4..28ab624 100755 (executable)
@@ -179,12 +179,6 @@ static void Sample()
                                         //Presentation of the attribute is erased
                                         //from viewer and removed from AIS_InteractiveContext
 
-  //Checking if attribute is displayed in viewer 
-
-  if( ShapeLabel.FindAttribute(TPrsStd_AISPresentation::GetID(), PRS) ) 
-  Standard_Boolean isDisplayed = PRS->IsDisplayed();
-
-  
   Handle(TPrsStd_AISPresentation) P;
   if( ShapeLabel.FindAttribute(TPrsStd_AISPresentation::GetID(), P) ) {
 
index a7355b2..219b3af 100755 (executable)
@@ -405,7 +405,7 @@ static Standard_Integer Draw_wait(Draw_Interpretor& , Standard_Integer n, const
 static unsigned int __stdcall CpuFunc (void * param)
 {
   clock_t aCurrent;
-  while (1)
+  for(;;)
   {
     Sleep (5);
     Standard_Real anUserSeconds, aSystemSeconds;
index da2144d..d9a669f 100755 (executable)
@@ -1110,7 +1110,7 @@ static Standard_Real ParseFactor(char*& name)
 {
   Standard_Real x = ParseValue(name);
 
-  while (1) {
+  for(;;) {
     char c = *name;
     if (c == '\0') return x;
     name++;
@@ -1136,7 +1136,7 @@ static Standard_Real Parse(char*& name)
 {
   Standard_Real x = ParseFactor(name);
 
-  while (1) {
+  for(;;) {
     char c = *name;
     if (c == '\0') return x;
     name++;
index 397cfc8..af3c796 100755 (executable)
@@ -1725,7 +1725,6 @@ void DrawWindow::Clear()
   HDC hDC = GetDC(win);
   HDC aWorkDC = myUseBuffer ? GetMemDC(hDC) : hDC;
 
-  int debug = GetROP2(aWorkDC);
   SaveDC(aWorkDC);
   SelectObject(aWorkDC,GetStockObject(BLACK_PEN));
   Rectangle(aWorkDC, 0, 0, WidthWin(), HeightWin());
@@ -2048,7 +2047,7 @@ static DWORD WINAPI readStdinThreadFunc(VOID)
 {
   if (!Draw_IsConsoleSubsystem) return 1;
 
-  while (1) {
+  for(;;) {
     while (console_semaphore != WAIT_CONSOLE_COMMAND)
       Sleep(100);
     //if (gets(console_command))
index 9af49fb..f80d35e 100755 (executable)
@@ -41,7 +41,6 @@ Extrema_FuncExtPS::Extrema_FuncExtPS (const gp_Pnt& P,
 {
   myP = P;
   myS = (Adaptor3d_SurfacePtr)&S;
-  GeomAbs_SurfaceType aSType = S.GetType();
   myPinit = Standard_True;
   mySinit = Standard_True;
 }
@@ -50,7 +49,6 @@ Extrema_FuncExtPS::Extrema_FuncExtPS (const gp_Pnt& P,
 void Extrema_FuncExtPS::Initialize(const Adaptor3d_Surface& S)
 {
   myS = (Adaptor3d_SurfacePtr)&S;
-  GeomAbs_SurfaceType aSType = S.GetType();
   mySinit = Standard_True;
   myPoint.Clear();
   mySqDist.Clear();
index 98a4fc4..49f97c4 100755 (executable)
@@ -496,7 +496,6 @@ void Extrema_GenExtPS::GetGridPoints( const Adaptor3d_Surface& theSurf)
   {
     Handle(TColStd_HArray1OfReal) anArrKnots;
     Standard_Integer aDegree = 0;
-    GeomAbs_CurveType aType = theSurf.BasisCurve()->Curve().GetType();
     if(theSurf.BasisCurve()->Curve().GetType() == GeomAbs_BSplineCurve)
     {
       Handle(Geom_BSplineCurve) aBspl = theSurf.BasisCurve()->Curve().BSpline();
@@ -810,14 +809,10 @@ void Extrema_GenExtPS::FindSolution(const gp_Pnt& P,
 
   math_Vector errors(1,2);
   math_Vector root(1, 2);
-  Standard_Real eps = 1.e-9;
-  Standard_Integer nbsubsample = 11;
 
   Standard_Integer aNbMaxIter = 100;
 
   gp_Pnt PStart = theParams.Value();
-  Standard_Real DistStart = theParams.GetSqrDistance();
-  Standard_Real DistSol = DistStart;
   
   math_FunctionSetRoot S (myF,UV,Tol,UVinf,UVsup, aNbMaxIter);
   
@@ -962,7 +957,7 @@ void Extrema_GenExtPS::Perform(const gp_Pnt& P)
       Bnd_SphereUBTreeSelectorMin aSelector(mySphereArray, aSol);
       //aSelector.SetMaxDist( RealLast() );
       aSelector.DefineCheckPoint( P );
-      Standard_Integer aNbSel = mySphereUBTree->Select( aSelector );
+      mySphereUBTree->Select( aSelector );
       //TODO: check if no solution in binary tree
       Bnd_Sphere& aSph = aSelector.Sphere();
       Standard_Real aU = myUParams->Value(aSph.U());
@@ -979,13 +974,12 @@ void Extrema_GenExtPS::Perform(const gp_Pnt& P)
       Bnd_SphereUBTreeSelectorMax aSelector(mySphereArray, aSol);
       //aSelector.SetMaxDist( RealLast() );
       aSelector.DefineCheckPoint( P );
-      Standard_Integer aNbSel = mySphereUBTree->Select( aSelector );
+      mySphereUBTree->Select( aSelector );
       //TODO: check if no solution in binary tree
       Bnd_Sphere& aSph = aSelector.Sphere();
       Standard_Real aU = myUParams->Value(aSph.U());
       Standard_Real aV = myVParams->Value(aSph.V());
       Extrema_POnSurfParams aParams(aU, aV, myS->Value(aU, aV));
-
       aParams.SetSqrDistance(P.SquareDistance(aParams.Value()));
       aParams.SetIndices(aSph.U(), aSph.V());
 
index 7dea712..3111e44 100755 (executable)
@@ -392,7 +392,7 @@ void FSD_CmpFile::ReadString(TCollection_AsciiString& buffer)
 
 void FSD_CmpFile::ReadWord(TCollection_AsciiString& buffer)
 {
-  char c;
+  char c = '\0';
   char b[8193],*tmpb;
   Standard_Boolean IsEnd = Standard_False;
   Standard_Integer i;
@@ -1394,5 +1394,4 @@ Storage_Position FSD_CmpFile::Tell()
   }
   default: return -1;
   }
-  return -1;
 }
index d755d6f..33c320c 100755 (executable)
@@ -326,7 +326,7 @@ void FSD_File::ReadString(TCollection_AsciiString& buffer)
 
 void FSD_File::ReadWord(TCollection_AsciiString& buffer)
 {
-  char c;
+  char c = '\0';
   char b[8193],*tmpb;
   Standard_Boolean IsEnd = Standard_False;
   Standard_Integer i;
@@ -1368,5 +1368,4 @@ Storage_Position FSD_File::Tell()
   }
   default: return -1;
   }
-  return -1;
 }
index 868613f..4e19908 100755 (executable)
@@ -94,7 +94,6 @@ Standard_Boolean FWOSDriver_Driver::HasReadPermission(const TCollection_Extended
     default:
       return Standard_False;
     }
-  return Standard_False;
 }
 
 //==============================================================================
index 98a1f33..19f4ddb 100755 (executable)
@@ -528,8 +528,6 @@ GeomFill_NSections::GeomFill_NSections(const TColGeom_SequenceOfCurve& NC,
     Handle(Geom_TrimmedCurve) curvTrim;
     Handle(Geom_BSplineCurve) curvBS, curvBS1;
     Handle(Geom_Curve) curv =  mySections(1);
-    Standard_Real first = curv->FirstParameter(),
-                   last = curv->LastParameter();
 
     for (j=jdeb; j<=jfin; j++) {
 
index c130e0f..aa41e34 100755 (executable)
@@ -847,7 +847,6 @@ static void ComputeDeviation(const Handle(Geom_Curve)& theCurve,
   theUfMax = 0.;
   theUlMax = 0.;
   theImax = 0;
-  Standard_Real ufmax = 0., ulmax = 0.;
 
   //take knots
   Standard_Integer nbp = thePnts->NbKnots();
index 252bee7..8f67e98 100755 (executable)
@@ -152,9 +152,7 @@ void HLRTopoBRep_OutLiner::ProcessFace(const TopoDS_Face& F,
       B.Add(NF,W); // add the new wire in the new face.
     }
   //}
-  
-
-  TopTools_ListOfShape& IntL = myDS.AddIntL(F);
+  myDS.AddIntL(F);
   TopTools_ListOfShape& OutL = myDS.AddOutL(F);
 
   if (myDS.FaceHasIntL(F)) { // get the InternalOutLines on face F
index b0eaa94..3c2be18 100755 (executable)
@@ -114,7 +114,7 @@ static int deja = 0;
 //  read mode : lire les lignes
 //  On charge le fichier dans "thelist"
   Standard_Boolean header = Standard_False;
-  while (1) {
+  for(;;) {
     ligne[0] = '\0';
     fgets(ligne,200,lefic);
     if (feof(lefic)) break;
index 85a515a..4b00b23 100755 (executable)
@@ -71,7 +71,7 @@ char* nomfic; int lesect[6]; int modefnes;
   if (lefic == NULL) return -1;    /*  fichier pas pu etre ouvert  */
   for (i = 1; i < 6; i++) lesect[i] = 0;
   for (j = 0; j < 100; j++) ligne[j] = 0;
-  while (1) {
+  for(;;) {
     numl ++;
     i = iges_lire(lefic,&numsec,ligne,modefnes);
     if (i <= 0) {
index d356438..dcabde6 100755 (executable)
   default:
     return(gp_Pnt(0.0,0.0,0.0));
   }
-  return(gp_Pnt(0.0,0.0,0.0));
 }
 
 //
index 0e4cc67..6d2f879 100755 (executable)
@@ -893,7 +893,7 @@ gp_Pnt DefineDU(const Handle(IntPatch_ALine)& aline,
   gp_Pnt P1 = aline->Value(U), P2, P3;
   gp_Vec V13, V12, V23;
   Standard_Real dU = DU/2.0, curvDef, angDef, m1, m2, m3;
-  do{ //According to class TangentialDeflection from GCPnts
+  for(;;) { //According to class TangentialDeflection from GCPnts
     P2=aline->Value(U+dU);  P3=aline->Value(U+DU);
     V13 = P3.XYZ().Subtracted(P1.XYZ());  m1 = V13.Magnitude();
     V12 = P2.XYZ().Subtracted(P1.XYZ());  m2 = V12.Magnitude();
@@ -903,7 +903,7 @@ gp_Pnt DefineDU(const Handle(IntPatch_ALine)& aline,
     angDef = Abs(V13.Angle(V23));
     if(curvDef < CurvDef && angDef < AngDef) break;
     DU = dU;  dU /= 2.0;  
-  }while(1);
+  }
   return P3;
 }
 //=======================================================================
index 59bd2d9..68a7c36 100755 (executable)
@@ -149,7 +149,6 @@ void IntPatch_PrmPrmIntersection::Perform (const Handle(Adaptor3d_HSurface)&
   done = Standard_True;
   SLin.Clear();  
 
-  Standard_Real Deflection2 = Deflection*Deflection;
   Standard_Integer nbLigSec = Interference.NbSectionLines();
   Standard_Integer nbTanZon = Interference.NbTangentZones();
 
@@ -1326,12 +1325,10 @@ void IntPatch_PrmPrmIntersection::Perform (const Handle(Adaptor3d_HSurface)&
     if(U2<UminLig2) UminLig2=U2;
     if(V2<VminLig2) VminLig2=V2; 
   }
-
-  Standard_Real Deflection2 = Deflection*Deflection;
   
   Standard_Real SeuildPointLigne = 15.0 * Increment * Increment;
   
-  Standard_Integer NbPntOn2SOnLine = 0, NbLigCalculee = 0, ver;
+  Standard_Integer NbLigCalculee = 0, ver;
   Standard_Real pu1,pu2,pv1,pv2, dminiPointLigne;
   Standard_Boolean HasStartPoint,RejetLigne;
   IntSurf_PntOn2S StartPOn2S;
@@ -1896,12 +1893,11 @@ void IntPatch_PrmPrmIntersection::Perform (const Handle(Adaptor3d_HSurface)& Sur
       return;
     }
     
-    Standard_Real Deflection2 = Deflection*Deflection;
     Standard_Integer nbLigSec = Interference.NbSectionLines();
     Standard_Integer nbTanZon = Interference.NbTangentZones();
     Standard_Real SeuildPointLigne = 15.0 * Increment * Increment;
 
-    Standard_Integer NbPntOn2SOnLine = 0, NbLigCalculee = 0, ver;
+    Standard_Integer NbLigCalculee = 0, ver;
     Standard_Real U1,U2,V1,V2, pu1,pu2,pv1,pv2, incidence, dminiPointLigne;
     Standard_Boolean HasStartPoint,RejetLigne;
     IntSurf_PntOn2S StartPOn2S;
@@ -2489,7 +2485,6 @@ void IntPatch_PrmPrmIntersection::Perform (const Handle(Adaptor3d_HSurface)& Sur
   empt = Standard_True;
   done = Standard_True;
   SLin.Clear();  
-  Standard_Real Deflection2 = Deflection*Deflection;
   
   Standard_Integer NbLigCalculee = 0;
   Standard_Real U1,U2,V1,V2;
index 10e9e73..3ff6f40 100755 (executable)
@@ -208,7 +208,6 @@ void IntPolyh_MaillageAffinage::FillArrayOfPnt
   Standard_Real u0, u1, v0, v1, aU, aV, dU, dV;
   //
   const Handle(Adaptor3d_HSurface&) MaSurface=(SurfID==1)? MaSurface1 : MaSurface2;
-  IntPolyh_ArrayOfPoints &TPoints=(SurfID==1)? TPoints1:TPoints2;
   NbSamplesU=(SurfID==1)? NbSamplesU1:NbSamplesU2;
   NbSamplesV=(SurfID==1)? NbSamplesV1:NbSamplesV2;
   //
@@ -260,7 +259,6 @@ void IntPolyh_MaillageAffinage::FillArrayOfPnt
   Standard_Integer NbSamplesU, NbSamplesV, i, aNbSamplesU1, aNbSamplesV1; 
   Standard_Real u0, u1, v0, v1, aU, aV, dU, dV;
   const Handle(Adaptor3d_HSurface)& MaSurface=(SurfID==1)? MaSurface1 : MaSurface2;
-  IntPolyh_ArrayOfPoints &TPoints=(SurfID==1)? TPoints1:TPoints2;
   NbSamplesU=(SurfID==1)? NbSamplesU1:NbSamplesU2;
   NbSamplesV=(SurfID==1)? NbSamplesV1:NbSamplesV2;
 
index 57fbb6d..d23d0ef 100755 (executable)
@@ -1305,7 +1305,7 @@ Standard_Integer FindMaxDistPC(const Standard_Real aT1A,
     return iErr;
   }
   //
-  while(1) {
+  for(;;) {
     if (aYP>aYL) {
       aA=aXL;
       aXL=aXP;
index 8023599..975e5e0 100755 (executable)
@@ -898,7 +898,6 @@ Standard_Integer IntTools_BeanFaceIntersector::FastComputeExactIntersection()
       aTolang2=1.e-16;
       aLin=myCurve.Line();
       const gp_Dir& aDirL=aLin.Direction();
-      const gp_Pnt& aLocL=aLin.Location();
       //
       aCos=aDirC.Dot(aDirL);
       if(aCos >= 0.) {
index a6802eb..3a81a85 100755 (executable)
@@ -585,7 +585,7 @@ IntTools_EdgeEdge::IntTools_EdgeEdge()
   Standard_Integer step = 1, stepcheck = 1000, steplimit = 100000;
   Standard_Real value = (IP==1) ? DistanceFunction(0.5*(a+b)) : DerivativeFunction(0.5*(a+b));
   
-  while (1) {
+  for(;;) {
     x0=.5*(a+b);
 
     if (IP==1)
index 53ef989..d184d4f 100644 (file)
@@ -323,7 +323,7 @@ void IntTools_ComparatorCurve::GetCurveBase(const Handle(Geom_Curve)& aC3D,
     return;
   }
   //
-  while(1) {
+  for(;;) {
     iTrimmed=0;
     iOffset=0;
     aCT3D=Handle(Geom_TrimmedCurve)::DownCast(aC3DB);
@@ -795,7 +795,7 @@ void IntTools_DistCC::FindMaxLocal(const Standard_Real aT11,
     return;
   }
   //
-  while(1) {
+  for(;;) {
     if (fabs(aA-aB)<aEps) {
       aX=0.5*(aA+aB);
       aF=Distance(aX);
index 8136ca0..37a910a 100755 (executable)
@@ -391,7 +391,7 @@ void IntTools_EdgeFace::CheckData()
   anIsProj1=ff1;
   anIsProj2=ff2;
   
-  while (1) {
+  for(;;) {
     if (fabs(t1-t2) < aEpsT) {
       tRoot=(anIsProj1) ? t1 : t2;
       return;
@@ -781,7 +781,7 @@ void IntTools_EdgeFace::CheckData()
   
   a=tA; b=tB; r=fA;
   
-  while (1) {
+  for(;;) {
     x0=.5*(a+b);
 
     if (IP==1)
@@ -825,7 +825,7 @@ void IntTools_EdgeFace::CheckData()
   yl=coeff*DistanceFunction(xl);
   
  
-  while (1) {
+  for(;;) {
     
     if (fabs(b-a) < myEpsT) {
       return .5*(b+a);
index 50e5044..e9bffd6 100755 (executable)
@@ -592,7 +592,7 @@ IntTools_FClass2d::IntTools_FClass2d()
     }//if (IsVPer) {
   }
 
-  while (1) {
+  for(;;) {
     dedans = 1;
     gp_Pnt2d Puv(u,v);
       
index e546edd..9c8b66e 100755 (executable)
@@ -4835,7 +4835,7 @@ Standard_Real FindMaxSquareDistance (const Standard_Real aT1,
   aF2=MaxSquareDistance(aX2, 
                        aC3D, aC2D1, aC2D2, myHS1, myHS2, myFace1, myFace2, myContext);
   //
-  while(1) {
+  for(;;) {
     //
     if (fabs(aA-aB)<aEps) {
       aX=0.5*(aA+aB);
index dd1aeb2..56a0158 100755 (executable)
@@ -166,7 +166,7 @@ Standard_Boolean LDOMParser::ParseDocument ()
   Standard_Boolean      isHeader  = Standard_False;
   Standard_Boolean      isDoctype = Standard_False;
 
-  while (1) {
+  for(;;) {
     LDOM_XmlReader::RecordType aType = ReadRecord (*myReader, myCurrentData);
     switch (aType) {
     case LDOM_XmlReader::XML_HEADER:
@@ -246,7 +246,7 @@ Standard_Boolean LDOMParser::ParseElement ()
   Standard_Boolean  isError = Standard_False;
   const LDOM_BasicElement * aParent = &myReader->GetElement();
   const LDOM_BasicNode    * aLastChild = NULL;
-  while (1) {
+  for(;;) {
     LDOM_Node::NodeType aLocType;
     LDOMBasicString     aTextValue;
     char *aTextStr;
index 7a41f59..9f74c13 100755 (executable)
@@ -54,7 +54,7 @@ char * LDOM_CharReference::Decode (char * theSrc, Standard_Integer& theLen)
 
   char * aSrcPtr = theSrc, * aDstPtr = theSrc;
   Standard_Integer anIncrCount = 0;
-  while (1) {
+  for(;;) {
     char * aPtr = strchr (aSrcPtr, '&');
     if (aPtr == NULL) {
       //        End of the loop
@@ -152,7 +152,7 @@ char * LDOM_CharReference::Encode (const char* theSrc, Standard_Integer& theLen,
   char       * aDest = (char *) theSrc;
   Standard_Integer aCount = 0;
   //    Analyse if there is a non-standard character in the string
-  while (1) {
+  for(;;) {
     const unsigned int iSrc =
       (const unsigned int) * (const unsigned char *) ptrSrc;
     if (iSrc == 0) {
index 2682de3..9d059db 100755 (executable)
@@ -48,7 +48,7 @@ LDOMString LDOM_Element::getAttribute (const LDOMString& aName) const
   if (myLastChild == NULL) {
     const LDOM_BasicNode * aNode = anElem.GetFirstChild();
     if (aNode && aNode -> getNodeType () != LDOM_Node::ATTRIBUTE_NODE)
-      while (1) {
+      for(;;) {
         const LDOM_BasicNode * aSibling = aNode -> GetSibling();
         if (aSibling == NULL)
           return LDOMString ();
@@ -77,7 +77,7 @@ LDOM_Attr LDOM_Element::getAttributeNode (const LDOMString& aName) const
   if (myLastChild == NULL) {
     const LDOM_BasicNode * aNode = anElem.GetFirstChild();
     if (aNode && aNode -> getNodeType () != LDOM_Node::ATTRIBUTE_NODE)
-      while (1) {
+      for(;;) {
         const LDOM_BasicNode * aSibling = aNode -> GetSibling();
         if (aSibling == NULL)
           return LDOM_Attr ();
index b009073..5a274aa 100755 (executable)
@@ -116,7 +116,7 @@ LDOM_XmlReader::RecordType LDOM_XmlReader::ReadRecord
   LDOMBasicString anAttrName, anAttrValue;
   char anAttDelimiter = '\0';
 
-  while (1) {
+  for(;;) {
     //  Check if the current file buffer is exhausted
     // ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     //  There should always be some bytes available in the buffer for analysis
@@ -293,7 +293,7 @@ LDOM_XmlReader::RecordType LDOM_XmlReader::ReadRecord
         // ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     case STATE_COMMENT:
       aPtr = aStartData;
-      while (1) {
+      for(;;) {
         aPtr = (const char *) memchr (aPtr, '-', (myEndPtr - 2) - aPtr);
         if (aPtr == NULL) break;
         if (aPtr[1] != '-') ++ aPtr;
@@ -327,7 +327,7 @@ LDOM_XmlReader::RecordType LDOM_XmlReader::ReadRecord
         // ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     case STATE_CDATA:
       aPtr = aStartData;
-      while (1) {
+      for(;;) {
         aPtr = (const char *) memchr (aPtr, ']', (myEndPtr - 1) - aStartData);
         if (aPtr == NULL) break;
         if (aPtr[1] != ']') {           // ERROR
@@ -530,7 +530,7 @@ static Standard_Boolean isName (const char  * aString,
                                 const char  *& aNameEnd)
 {
   Standard_Boolean aResult;
-  int aCh = aString[0];
+  char aCh = aString[0];
   if (IsAlphabetic(aCh) || aCh == '_' || aCh == ':') {
     const char * aPtr = &aString[1];
     while (aPtr < aStringEnd) {
index 4eb0aaf..6e67972 100755 (executable)
@@ -99,7 +99,7 @@ getString (_Char *&                     thePtr,
     //    Skip whitespaces in the beginning of the string
     aPtr = anEndPtr;
     aLeftSpaces = 0;
-    while (1)
+    for (;;)
     {
       _Char aChar = * aPtr;
       if      (aChar == ' ')  aLeftSpaces++;
index 107a77a..2b4de44 100755 (executable)
@@ -44,6 +44,8 @@ namespace
 
   #define IMEM_FREE(p_bl) (size_t(p_bl->p_end_block - p_bl->p_free_space))
 
+  // auxiliary dummy function used to get a place where break point can be set
+  inline void place_for_breakpoint() {}
 };
 
 #define MaxLookup 16
@@ -82,6 +84,8 @@ Standard_EXPORT void IncAllocator_SetDebugFlag(const Standard_Boolean theDebug)
   IS_DEBUG = theDebug;
 }
 
+#ifdef DEB
+
 //=======================================================================
 /**
  * Static value of the current allocation ID. It provides unique
@@ -102,12 +106,9 @@ static void Debug_Create(Standard_Address theAlloc)
   aMutex.Lock();
   StorageIDMap().Bind(theAlloc, ++CurrentID);
   StorageIDSet().Add(CurrentID);
-  aMutex.Unlock();
   if (CurrentID == CATCH_ID)
-  {
-    // Place for break point for creation of investigated allocator
-    int a = 1;
-  }
+    place_for_breakpoint();
+  aMutex.Unlock();
 }
 
 //=======================================================================
@@ -128,6 +129,8 @@ static void Debug_Destroy(Standard_Address theAlloc)
   aMutex.Unlock();
 }
 
+#endif /* DEB */
+
 //=======================================================================
 //function : IncAllocator_PrintAlive
 //purpose  : Outputs the alive numbers to the file inc_alive.d
index 26de136..a6a913d 100755 (executable)
@@ -42,7 +42,8 @@ Standard_Boolean OSD::RealToCString(const Standard_Real aReal,
 
   // Suppress "e+00" and unsignificant 0's 
 
-  if ((p = strchr(aString,'e'))) {
+  p = strchr(aString,'e');
+  if (p) {
     if (!strcmp(p,"e+00"))
       *p = 0 ;
     for (q = p-1 ; *q == '0' ; q--) ;
index d2f0c74..7a25456 100755 (executable)
@@ -327,7 +327,7 @@ Standard_Integer OSD_DirectoryIterator :: Error () const {
 OSD_DirectoryIterator::OSD_DirectoryIterator() {}
 
 void OSD_DirectoryIterator::Initialize(
-                             const OSD_Path& where,
-                             const TCollection_AsciiString& Mask){}
+                             const OSD_Path& /*where*/,
+                             const TCollection_AsciiString& /*Mask*/){}
 
 #endif
index cf73a48..0de301f 100755 (executable)
@@ -329,7 +329,7 @@ Standard_Integer OSD_Disk :: DiskQuota () {
 
 }  // end OSD_Disk :: DiskQuota
 
-void OSD_Disk :: SetDiskQuota ( const Standard_Integer QuotaSize ) {
+void OSD_Disk :: SetDiskQuota ( const Standard_Integer /*QuotaSize*/ ) {
 
  SetLastError (  ( DWORD )STG_E_UNIMPLEMENTEDFUNCTION  );
 
index bbbbc71..23a187b 100755 (executable)
@@ -960,9 +960,8 @@ void OSD_File :: Build (
 // Open a file
 // ---------------------------------------------------------------------
 
-void OSD_File :: Open (
-                  const OSD_OpenMode Mode, const OSD_Protection& Protect
-                 ) {
+void OSD_File :: Open (const OSD_OpenMode Mode, const OSD_Protection& /*Protect*/)
+{
 
  TCollection_AsciiString fName;
 
@@ -1516,7 +1515,7 @@ void OSD_File :: Seek (
                   const Standard_Integer Offset, const OSD_FromWhere Whence
                  ) {
 
- DWORD dwMoveMethod;
+ DWORD dwMoveMethod = 0;
 
  TEST_RAISE(  TEXT( "Seek" )  );
 
@@ -1940,7 +1939,7 @@ PSECURITY_DESCRIPTOR __fastcall _osd_wnt_protection_to_sd (
  PTOKEN_OWNER         pTkOwner        = NULL;
  PTOKEN_GROUPS        pTkGroups       = NULL;
  PTOKEN_PRIMARY_GROUP pTkPrimaryGroup = NULL;
- PSECURITY_DESCRIPTOR retVal;
+ PSECURITY_DESCRIPTOR retVal = NULL;
  PSECURITY_DESCRIPTOR pfSD = NULL;
  BOOL                 fDummy;
  PFILE_ACE            pFileACE;
@@ -2388,7 +2387,7 @@ static int __fastcall _get_buffer (
 
 static DWORD __fastcall _get_access_mask ( OSD_SingleProtection prt ) {
 
- DWORD retVal;
+ DWORD retVal = 0;
 
  switch ( prt ) {
  
@@ -2500,7 +2499,7 @@ static DWORD __fastcall _get_access_mask ( OSD_SingleProtection prt ) {
 
 static DWORD __fastcall _get_dir_access_mask ( OSD_SingleProtection prt ) {
 
- DWORD retVal;
+ DWORD retVal = 0;
 
  switch ( prt ) {
  
@@ -2617,7 +2616,7 @@ static HANDLE __fastcall _open_file (
                          ) {
 
  HANDLE retVal = INVALID_HANDLE_VALUE;
- DWORD  dwDesiredAccess;
+ DWORD  dwDesiredAccess = 0;
  DWORD  dwCreationDistribution;
 
  switch ( oMode ) {
@@ -2680,7 +2679,7 @@ Standard_Integer __fastcall _get_file_type (
                              Standard_CString fName, Standard_Integer fileHandle
                             ) {
 
- Standard_Integer retVal;
+ Standard_Integer retVal = 0;
  DWORD            dwType;
  int              fileType;
 
@@ -3049,9 +3048,9 @@ static OSD_SingleProtection __fastcall _get_protection_dir ( DWORD mask ) {
 BOOL __fastcall _osd_print (const Standard_PCharacter pName, Standard_CString fName ) {
 
  BOOL   fOK, fJob;                
- HANDLE hPrinter;
+ HANDLE hPrinter = NULL;
  BYTE   jobInfo[ MAX_PATH + sizeof ( DWORD ) ];
- DWORD  dwNeeded, dwCode;
+ DWORD  dwNeeded, dwCode = 0;
 
  fOK = fJob = FALSE;
 
@@ -3187,7 +3186,7 @@ Standard_Boolean OSD_File::ReadLastLine(TCollection_AsciiString& aLine,const Sta
 
   if (Count <= 0)
       return Standard_False ;
-  while(1) {
+  for(;;) {
     ReadLine(aLine, MaxLength, Len) ;
     if (!aLine.IsEmpty()) 
       return Standard_True ;
index aed1864..a36a7b7 100755 (executable)
@@ -411,7 +411,7 @@ Standard_Integer OSD_FileIterator :: Error () const {
 OSD_FileIterator::OSD_FileIterator() {}
 
 void OSD_FileIterator::Initialize(
-                       const OSD_Path& where,
-                       const TCollection_AsciiString& Mask){}
+                       const OSD_Path& /*where*/,
+                       const TCollection_AsciiString& /*Mask*/){}
  
 #endif
index 0f48703..4ba95e9 100755 (executable)
@@ -851,7 +851,7 @@ Quantity_Date OSD_FileNode::CreationMoment () {
 Standard_Integer OSD_FileNode::UserId () {
 
  PSID                    pSIDowner = NULL;
- PSID                    retVal;
+ PSID                    retVal = NULL;
  BOOL                    fDefaulted;
  TCollection_AsciiString fName;
  PSECURITY_DESCRIPTOR    pSD;
@@ -1002,7 +1002,7 @@ static BOOL __fastcall _get_file_time (
  FILETIME   ftCreationTime;
  FILETIME   ftLastWriteTime;
  LPFILETIME lpftPtr;
- HANDLE     hFile;
+ HANDLE     hFile = INVALID_HANDLE_VALUE;
 
  __try {
 
index 5fb0925..cb420d2 100755 (executable)
@@ -246,7 +246,7 @@ OSD_Host :: OSD_Host () {
 
  DWORD              nSize;
  Standard_Character szHostName[ MAX_COMPUTERNAME_LENGTH + 1 ];
- char*              hostAddr;
+ char*              hostAddr = 0;
  MEMORYSTATUS       ms;
  WSADATA            wd;
  PHOSTENT           phe;
@@ -258,6 +258,7 @@ OSD_Host :: OSD_Host () {
   nSize                         = MAX_COMPUTERNAME_LENGTH + 1;
   osVerInfo.dwOSVersionInfoSize = sizeof ( OSVERSIONINFO );
 
+  ZeroMemory (&ms, sizeof(ms));
   ZeroMemory (  szHostName, sizeof ( Standard_Character ) *  (MAX_COMPUTERNAME_LENGTH + 1) );
 
   if (  !GetVersionEx ( &osVerInfo )  ) {
index 3ce94e6..517ed6d 100755 (executable)
 
 static OSD_MAllocHook::Callback* MypCurrentCallback = NULL;
 
+namespace {
+  // dummy function to call at place where break point might be needed
+  inline void place_for_breakpoint () {}
+};
+
 //=======================================================================
 //function : GetCallback
 //purpose  :
@@ -414,11 +419,9 @@ void OSD_MAllocHook::LogFileHandler::AllocEvent
     myMutex.Lock();
     myLogFile << "alloc "<< std::setw(10) << theRequestNum
               << std::setw(20) << theSize << std::endl;
-    myMutex.Unlock();
     if (myBreakSize == theSize)
-    {
-      int a = 1;
-    }
+      place_for_breakpoint();
+    myMutex.Unlock();
   }
 }
 
@@ -562,9 +565,7 @@ void OSD_MAllocHook::CollectBySize::AllocEvent
                     long        /*theRequestNum*/)
 {
   if (myBreakSize == theSize)
-  {
-    int a = 1;
-  }
+    place_for_breakpoint();
   if (theSize > 0)
   {
     myMutex.Lock();
index bc4156f..c1be338 100755 (executable)
@@ -1209,7 +1209,7 @@ void OSD_Path :: SystemName (
 }  // end OSD_Path :: SystemName
 
 Standard_Boolean OSD_Path :: IsValid (
-                              const TCollection_AsciiString& aDependentName,
+                              const TCollection_AsciiString& /*aDependentName*/,
                               const OSD_SysType aSysType
                              ) const {
 
@@ -1263,7 +1263,7 @@ Standard_Integer OSD_Path :: TrekLength () const {
 
   return retVal;
 
- while ( Standard_True ) {
+ for (;;) {
  
   if (  myTrek.Token (  TEXT( "|" ), i++  ).IsEmpty ()  )
 
index 8ce6b52..9344a2e 100755 (executable)
@@ -340,7 +340,7 @@ Standard_Boolean OSD_Process :: IsSuperUser () {
  Standard_Boolean retVal = FALSE;
  PSID             pSIDadmin;
  HANDLE           hProcessToken = INVALID_HANDLE_VALUE;
- PTOKEN_GROUPS    pTKgroups;
+ PTOKEN_GROUPS    pTKgroups = NULL;
 
  if (  !OpenProcessToken (
          GetCurrentProcess (),
index 333f9c1..174fdaa 100755 (executable)
@@ -298,7 +298,7 @@ Standard_CString OSD_SharedLibrary :: Name () const {
 
 }  // end OSD_SharedLibrary :: Name
 
-Standard_Boolean OSD_SharedLibrary :: DlOpen ( const OSD_LoadMode Mode ) {
+Standard_Boolean OSD_SharedLibrary :: DlOpen ( const OSD_LoadMode /*Mode*/ ) {
 
  Standard_Boolean retVal = Standard_True;
 
index fdaa472..6bd3522 100755 (executable)
@@ -911,7 +911,7 @@ BOOL MoveDirectory ( LPCTSTR oldDir, LPCTSTR newDir ) {
  LPTSTR              pathSrc,  pathDst;
  HANDLE              hFindFile;
  BOOL                fFind;
- BOOL                retVal;
+ BOOL                retVal = FALSE;
  DIR_RESPONSE        response;
  DWORD               level;
 
@@ -1043,16 +1043,14 @@ retry:
 
      if ( pFD -> dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY ) {
 
-      if (   !(  retVal = MoveDirectory ( pFullNameSrc, pFullNameDst )  )   ) break;
+       retVal = MoveDirectory ( pFullNameSrc, pFullNameDst );
+       if (!retVal) break;
    
      } else {
 retry_1:   
-      if (   !(  retVal = MoveFileEx (
-                           pFullNameSrc, pFullNameDst,
-                           MOVEFILE_REPLACE_EXISTING | MOVEFILE_COPY_ALLOWED
-                          )
-              )
-      ) {
+      retVal = MoveFileEx (pFullNameSrc, pFullNameDst,
+                           MOVEFILE_REPLACE_EXISTING | MOVEFILE_COPY_ALLOWED);
+      if (! retVal) {
       
        if ( _response_dir_proc != NULL ) {
       
@@ -1142,7 +1140,7 @@ retry_2:
 /***/
 BOOL CopyDirectory ( LPCTSTR dirSrc, LPCTSTR dirDst ) {
 
- PWIN32_FIND_DATA    pFD;
+ PWIN32_FIND_DATA    pFD = NULL;
  LPTSTR              pName = NULL;
  LPTSTR              pFullNameSrc = NULL;
  LPTSTR              pFullNameDst = NULL;
@@ -1203,11 +1201,13 @@ BOOL CopyDirectory ( LPCTSTR dirSrc, LPCTSTR dirDst ) {
 
      if ( pFD -> dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY ) {
 
-      if (   !(  retVal = CopyDirectory ( pFullNameSrc, pFullNameDst )  )   ) break;
+       retVal = CopyDirectory ( pFullNameSrc, pFullNameDst );
+       if ( ! retVal ) break;
    
      } else {
 retry:   
-      if (   !(  retVal = CopyFile ( pFullNameSrc, pFullNameDst, FALSE )  )   ) {
+      retVal = CopyFile ( pFullNameSrc, pFullNameDst, FALSE );
+      if ( ! retVal ) {
       
        if ( _response_dir_proc != NULL ) {
       
index 0cd15cb..8f526d2 100755 (executable)
@@ -403,7 +403,8 @@ void OSD::SetSignal (const Standard_Boolean theFloatingSignal)
   SetConsoleCtrlHandler (&_osd_ctrl_break_handler, TRUE);
 
 #ifdef _MSC_VER
-  _se_translator_function pOldSeFunc = _set_se_translator (TranslateSE);
+//  _se_translator_function pOldSeFunc = 
+    _set_se_translator (TranslateSE);
 #endif
 
   fFltExceptions = theFloatingSignal;
@@ -533,7 +534,7 @@ LONG _osd_debug ( void ) {
 
  if ( !fDbgLoaded ) {
  
-  HKEY                hKey;
+  HKEY                hKey = NULL;
   HANDLE              hEvent = INVALID_HANDLE_VALUE;
   DWORD               dwKeyType;
   DWORD               dwValueLen;
@@ -620,12 +621,12 @@ LONG _osd_debug ( void ) {
 
 //void OSD::Handler(const OSD_Signals aSig,
 //                  const OSD_Signals aCode){}
-void OSD::Handler(const OSD_Signals theSignal,
-                 const Standard_Address theSigInfo,
-                 const Standard_Address theContext) {}
+void OSD::Handler(const OSD_Signals /*theSignal*/,
+                 const Standard_Address /*theSigInfo*/,
+                 const Standard_Address /*theContext*/) {}
 
-void OSD::SegvHandler(const OSD_Signals aSig,
-                      const Standard_Address code,
-                      const Standard_Address scp){}
+void OSD::SegvHandler(const OSD_Signals /*aSig*/,
+                      const Standard_Address /*code*/,
+                      const Standard_Address /*scp*/){}
 
 #endif // WNT
index 333d59b..cc25f07 100644 (file)
@@ -222,7 +222,6 @@ void OpenGl_LayerList::ChangeLayer (const OpenGl_Structure *theStructure,
     OpenGl_SequenceOfLayers::Iterator anIts;
     for (anIts.Init (myLayers); anIts.More (); anIts.Next (), aSeqId++)
     {
-      OpenGl_PriorityList& aScanList = anIts.ChangeValue ();
       if (aSeqPos == aSeqId)
         continue;
   
index e367c7d..6bc7d53 100644 (file)
@@ -158,7 +158,6 @@ void OpenGl_Trihedron::redraw (const Handle(OpenGl_Workspace)& theWorkspace) con
   /*
   * Creation du triedre
   */
-  const OpenGl_AspectLine *AspectLine = theWorkspace->AspectLine( Standard_True );
 
   /* Fotis Sioutis 2007-11-14 15:06
   I have also seen in previous posts that the view trihedron in V3d_WIREFRAME mode
index 4ad9445..3ce2a6c 100644 (file)
@@ -1179,8 +1179,6 @@ void OpenGl_View::RenderStructs (const Handle(OpenGl_Workspace) &AWorkspace)
 
   glPushAttrib ( GL_DEPTH_BUFFER_BIT );
 
-  const OpenGl_AspectLine *aspect_line = AWorkspace->AspectLine( Standard_True );
-
   //TsmPushAttri(); /* save previous graphics context */
 
   if ( (AWorkspace->NamedStatus & OPENGL_NS_2NDPASSNEED) == 0 )
index 0e9c524..8234b07 100755 (executable)
@@ -18,7 +18,7 @@
 
 #include <Prs3d_RadiusAspect.ixx>
 
-Prs3d_RadiusAspect::Prs3d_RadiusAspect(const Quantity_NameOfColor aColor,
+Prs3d_RadiusAspect::Prs3d_RadiusAspect(const Quantity_NameOfColor /*aColor*/,
                                       const Aspect_TypeOfLine aType,
                                       const Standard_Real aWidth) {
 }
index 7489123..ae91ac8 100644 (file)
@@ -447,8 +447,6 @@ static Standard_Integer OCC74bug_get (Draw_Interpretor& di, Standard_Integer arg
     di << "ERROR : Usage : " << argv[0] << " shape; get selection mode" << "\n";
     return 1;
   }
-  
-  Standard_Boolean updateviewer = Standard_True, PutInCollector = Standard_True;
 
   ViewerTest_DoubleMapOfInteractiveAndName& aMap = GetMapOfAIS();
   
index 11a6763..6ea73da 100755 (executable)
@@ -2694,8 +2694,7 @@ static Standard_Integer OCC7141 (Draw_Interpretor& di, Standard_Integer argc, co
   try {
     OCC_CATCH_SIGNALS
     if( writer.Transfer(document, mode)) {
-      IFSelect_ReturnStatus stat =
-       writer.Write(aFilePath.ToCString());
+       writer.Write(aFilePath.ToCString());
     }
   }
   catch(OSD_Exception_STACK_OVERFLOW) {
index 29ed685..209111b 100755 (executable)
@@ -62,7 +62,6 @@ static Standard_Integer OCC332bug (Draw_Interpretor& di, Standard_Integer argc,
 {
   // Used to Display Geometry or Topolgy
   char name[255];
-  char *pname = name;
   Standard_Boolean check = Standard_True;
 
   // Set default arguments
@@ -426,8 +425,6 @@ static Standard_Integer OCC544 (Draw_Interpretor& di, Standard_Integer argc, con
 
   // Used to Display Geometry or Topolgy
   char name[255];
-  char *pname = name;
-  //bool check = true;
   Standard_Boolean check = Standard_True;
 
   // Set default arguments
index 001715e..378c1b9 100755 (executable)
@@ -135,7 +135,6 @@ static Standard_Integer OCC527(Draw_Interpretor& di, Standard_Integer argc, cons
            if (!aResult.IsNull())
              {
             double lmaxdist = 0.0;
-            double lmaxtoler = 0.0;
             TopExp_Explorer aExp2;
             for (aExp2.Init(aResult,TopAbs_VERTEX); aExp2.More(); aExp2.Next())
             {
index 26db4c9..72792d4 100644 (file)
@@ -1348,7 +1348,7 @@ static Standard_Integer BUC60856(Draw_Interpretor& di, Standard_Integer /*argc*/
   }
 
   gp_Ax2  Cone_Ax;                                                                
-  double R1=8, R2=16, H1=20, H2=40, angle;                                       
+  double R1=8, R2=16, angle;                                       
   gp_Pnt P0(0,0,0),                                                              
   P1(0,0,20), P2(0,0,45);                                                        
   angle = 2*M_PI;                                                                  
@@ -1583,7 +1583,7 @@ static Standard_Integer statface (Draw_Interpretor& di,Standard_Integer /*argc*/
   Handle(Dico_DictionaryOfInteger) aDico = new Dico_DictionaryOfInteger();
   Handle(TColStd_HSequenceOfAsciiString) aSequence = new TColStd_HSequenceOfAsciiString;
   Standard_CString aString;
-  Standard_Integer i=1,j=1,k=1,l=1,aa=1;
+  Standard_Integer i=1,j=1,l=1,aa=1;
   TopExp_Explorer expl;
   Standard_Real f3d,l3d;
   for(expl.Init(aShape,TopAbs_FACE);expl.More();expl.Next())
index 5dfe86f..ce662f0 100755 (executable)
@@ -37,7 +37,7 @@
 static Standard_Integer BUC60857 (Draw_Interpretor& di, Standard_Integer /*argc*/,const char ** argv)
 {
   gp_Ax2  Cone_Ax;
-  double R1=8, R2=16, H1=20, H2=40, angle;
+  double R1=8, R2=16, angle;
   gp_Pnt P0(0,0,0), P1(0,0,20), P2(0,0,45);
   angle = 2*M_PI;
 
index 1033276..817339b 100755 (executable)
@@ -89,8 +89,6 @@ void QANewBRepNaming_Gluing::Init(const TDF_Label& ResultLabel) {
 void QANewBRepNaming_Gluing::Load(QANewModTopOpe_Glue& theMkGluing) {
   TopoDS_Shape aResShape  = theMkGluing.Shape();
   const TopoDS_Shape& anObjShape = theMkGluing.Shape1();
-  const TopoDS_Shape& aToolShape = theMkGluing.Shape2();
-  const TopAbs_ShapeEnum& anObjectType = ShapeType(anObjShape);
 
   if (aResShape.IsNull()) {
 #ifdef MDTV_DEB
index a16a0fc..ba89996 100755 (executable)
@@ -59,7 +59,6 @@ void QANewBRepNaming_Limitation::Load(QANewModTopOpe_Limitation& MS) const {
   TopoDS_Shape ResSh = MS.Shape();
 
   const TopoDS_Shape& ObjSh = MS.Shape1();
-  const TopoDS_Shape& ToolSh = MS.Shape2();
   const TopAbs_ShapeEnum& ObjType = ShapeType(ObjSh);
 
   if (ResSh.IsNull()) {
index a2c33fa..ffc6eb7 100755 (executable)
@@ -148,10 +148,8 @@ static Standard_Integer QANewDBRepNaming_CheckNaming(Draw_Interpretor& di,Standa
 
   TopTools_IndexedMapOfShape allSubShapes;
   TopExp::MapShapes(aSelectedShape, allSubShapes);
-//  cout<<"SELECTION ..."<<endl;
-//  cout << "i = " << allSubShapes.Extent() << endl;
-  Standard_Integer count = 0, i;
-  char aDrawName[80] = "Select";
+
+  Standard_Integer i;
 
   for (i = 1; i <= allSubShapes.Extent(); i++) {
     const TDF_Label& aSubLabel = L.FindChild(i);
@@ -237,7 +235,6 @@ static Standard_Integer QANewDBRepNaming_CheckNaming(Draw_Interpretor& di,Standa
 //      cout<<"Wrong selection ..."<<endl;
       return 1;
     }
-    const TopAbs_ShapeEnum TypeOfNaming = aNS->Get().ShapeType();
 //    cout<<i<<"  ";TopAbs::Print(TypeOfNaming, cout); cout<<"  ";
 //     DBRep::Set("Solved", aNS->Get());
     TNaming_Selector SLSolving(itr.Value());
index 60ebf4c..9d9bcd4 100755 (executable)
@@ -578,7 +578,6 @@ Standard_Boolean QANewModTopOpe::IsConnected(const TopoDS_Shape& TheS)
   if(nbs <= 1) return aRes;
 
   math_Matrix aMat(1, nbs, 1, nbs, 0);
-  Standard_Integer aMax=nbs*nbs;
 
   TopTools_MapIteratorOfMapOfShape anMIter(aSMap);
   Standard_Integer n = 1;
@@ -616,7 +615,7 @@ Standard_Boolean QANewModTopOpe::IsConnected(const TopoDS_Shape& TheS)
   }
   //cout << "QANewModTopOpe::IsConnected(3)Lnbs=" <<nbs<< endl;
 
-  Standard_Integer k,i,ncount=0;
+  Standard_Integer k,i;
   Standard_Real p; 
   Standard_Boolean aNotChanged;
 
index d62f67c..1edaa81 100755 (executable)
@@ -333,7 +333,6 @@ Standard_Boolean QANewModTopOpe_Intersection::IsDeleted(const TopoDS_Shape& aS)
 //=======================================================================
 Standard_Boolean QANewModTopOpe_Intersection::HasDeleted() const
 {
-  Standard_Real aresult = Standard_False;
   TopExp_Explorer anExp;
 
   for(Standard_Integer argit = 0; argit < 2; argit++) {
index aeadf4a..06833d3 100755 (executable)
@@ -547,7 +547,7 @@ Standard_Boolean QANewModTopOpe_Tools::BoolOpe(const TopoDS_Shape& theFace1,
   const BOPDS_PDS& pDS = aDSFiller.PDS();
 
   Standard_Integer aNb = 0, aNbSps;
-  Standard_Integer i = 0, j = 0;
+  Standard_Integer i = 0;
   TopTools_IndexedMapOfShape aMapV;
 
   {
index 0774e9f..b8f919e 100755 (executable)
@@ -55,7 +55,7 @@ void RWHeaderSection_GeneralModule::FillSharedCase(const Standard_Integer CN, co
 void RWHeaderSection_GeneralModule::CheckCase(const Standard_Integer CN,
                                               const Handle(Standard_Transient)& ent,
                                               const Interface_ShareTool& shares,
-                                              Handle(Interface_Check)& ach) const
+                                              Handle(Interface_Check)& /*ach*/) const
 {
 }
 
index d98d361..15fd6b2 100755 (executable)
@@ -45,10 +45,7 @@ void RWStepAP214_RWAppliedApprovalAssignment::ReadStep
   // --- inherited field : assignedApproval ---
   
   Handle(StepBasic_Approval) aAssignedApproval;
-#ifdef DEB
-  Standard_Boolean stat1 = 
-#endif
-    data->ReadEntity(num, 1,"assigned_approval", ach, STANDARD_TYPE(StepBasic_Approval), aAssignedApproval);
+  data->ReadEntity(num, 1,"assigned_approval", ach, STANDARD_TYPE(StepBasic_Approval), aAssignedApproval);
 
   // --- own field : items ---
   
index 7b0410a..8687827 100755 (executable)
@@ -46,18 +46,12 @@ void RWStepAP214_RWAppliedDateAndTimeAssignment::ReadStep
   // --- inherited field : assignedDateAndTime ---
   
   Handle(StepBasic_DateAndTime) aAssignedDateAndTime;
-#ifdef DEB
-  Standard_Boolean stat1 = 
-#endif
-    data->ReadEntity(num, 1,"assigned_date_and_time", ach, STANDARD_TYPE(StepBasic_DateAndTime), aAssignedDateAndTime);
+  data->ReadEntity(num, 1,"assigned_date_and_time", ach, STANDARD_TYPE(StepBasic_DateAndTime), aAssignedDateAndTime);
   
   // --- inherited field : role ---
   
   Handle(StepBasic_DateTimeRole) aRole;
-#ifdef DEB
-  Standard_Boolean stat2 = 
-#endif
-    data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_DateTimeRole), aRole);
+  data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_DateTimeRole), aRole);
   
   // --- own field : items ---
   
index 8ce18dc..d1f8322 100755 (executable)
@@ -46,18 +46,12 @@ void RWStepAP214_RWAppliedDateAssignment::ReadStep
   // --- inherited field : assignedDate ---
   
   Handle(StepBasic_Date) aAssignedDate;
-#ifdef DEB
-  Standard_Boolean stat1 = 
-#endif
-    data->ReadEntity(num, 1,"assigned_date", ach, STANDARD_TYPE(StepBasic_Date), aAssignedDate);
+  data->ReadEntity(num, 1,"assigned_date", ach, STANDARD_TYPE(StepBasic_Date), aAssignedDate);
   
   // --- inherited field : role ---
   
   Handle(StepBasic_DateRole) aRole;
-#ifdef DEB
-  Standard_Boolean stat2 = 
-#endif
-    data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_DateRole), aRole);
+  data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_DateRole), aRole);
 
   // --- own field : items ---
 
index ac7d332..6f385c5 100755 (executable)
@@ -38,10 +38,7 @@ void  RWStepAP214_RWAppliedDocumentReference::ReadStep
   // --- inherited field : assigned_document
   
   Handle(StepBasic_Document) adoc;
-#ifdef DEB
-  Standard_Boolean stat1 = 
-#endif
-    data->ReadEntity
+  data->ReadEntity
       (num, 1,"assigned_document", ach, STANDARD_TYPE(StepBasic_Document), adoc);
   
   // --- inherited field : source ---
index 2991386..08824ec 100755 (executable)
@@ -46,18 +46,12 @@ void RWStepAP214_RWAppliedOrganizationAssignment::ReadStep
   // --- inherited field : assignedOrganization ---
   
   Handle(StepBasic_Organization) aAssignedOrganization;
-#ifdef DEB
-  Standard_Boolean stat1 = 
-#endif
-    data->ReadEntity(num, 1,"assigned_organization", ach, STANDARD_TYPE(StepBasic_Organization), aAssignedOrganization);
+  data->ReadEntity(num, 1,"assigned_organization", ach, STANDARD_TYPE(StepBasic_Organization), aAssignedOrganization);
   
   // --- inherited field : role ---
 
   Handle(StepBasic_OrganizationRole) aRole;
-#ifdef DEB
-  Standard_Boolean stat2 = 
-#endif
-    data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_OrganizationRole), aRole);
+  data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_OrganizationRole), aRole);
   
   // --- own field : items ---
 
index 4764e8e..32deeec 100755 (executable)
@@ -46,18 +46,12 @@ void RWStepAP214_RWAppliedPersonAndOrganizationAssignment::ReadStep
   // --- inherited field : assignedPersonAndOrganization ---
   
   Handle(StepBasic_PersonAndOrganization) aAssignedPersonAndOrganization;
-#ifdef DEB
-  Standard_Boolean stat1 = 
-#endif
-    data->ReadEntity(num, 1,"assigned_person_and_organization", ach, STANDARD_TYPE(StepBasic_PersonAndOrganization), aAssignedPersonAndOrganization);
+  data->ReadEntity(num, 1,"assigned_person_and_organization", ach, STANDARD_TYPE(StepBasic_PersonAndOrganization), aAssignedPersonAndOrganization);
   
   // --- inherited field : role ---
   
   Handle(StepBasic_PersonAndOrganizationRole) aRole;
-#ifdef DEB
-  Standard_Boolean stat2 = 
-#endif
-    data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_PersonAndOrganizationRole), aRole);
+  data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_PersonAndOrganizationRole), aRole);
   
   // --- own field : items ---
   
index e7aa80c..1569f8d 100755 (executable)
@@ -45,10 +45,7 @@ void RWStepAP214_RWAppliedSecurityClassificationAssignment::ReadStep
   // --- inherited field : assignedSecurityClassification ---
   
   Handle(StepBasic_SecurityClassification) aAssignedSecurityClassification;
-#ifdef DEB
-  Standard_Boolean stat1 = 
-#endif
-    data->ReadEntity(num, 1,"assigned_security_classification", ach, STANDARD_TYPE(StepBasic_SecurityClassification), aAssignedSecurityClassification);
+  data->ReadEntity(num, 1,"assigned_security_classification", ach, STANDARD_TYPE(StepBasic_SecurityClassification), aAssignedSecurityClassification);
   
   // --- own field : items ---
   
index ae97240..d1ab638 100755 (executable)
@@ -46,18 +46,12 @@ void RWStepAP214_RWAutoDesignActualDateAndTimeAssignment::ReadStep
        // --- inherited field : assignedDateAndTime ---
 
        Handle(StepBasic_DateAndTime) aAssignedDateAndTime;
-#ifdef DEB
-       Standard_Boolean stat1 = 
-#endif
-         data->ReadEntity(num, 1,"assigned_date_and_time", ach, STANDARD_TYPE(StepBasic_DateAndTime), aAssignedDateAndTime);
+    data->ReadEntity(num, 1,"assigned_date_and_time", ach, STANDARD_TYPE(StepBasic_DateAndTime), aAssignedDateAndTime);
 
        // --- inherited field : role ---
 
        Handle(StepBasic_DateTimeRole) aRole;
-#ifdef DEB
-       Standard_Boolean stat2 = 
-#endif
-         data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_DateTimeRole), aRole);
+    data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_DateTimeRole), aRole);
 
        // --- own field : items ---
 
index 3aa2513..01b062e 100755 (executable)
@@ -46,18 +46,12 @@ void RWStepAP214_RWAutoDesignActualDateAssignment::ReadStep
        // --- inherited field : assignedDate ---
 
        Handle(StepBasic_Date) aAssignedDate;
-#ifdef DEB
-       Standard_Boolean stat1 = 
-#endif
-         data->ReadEntity(num, 1,"assigned_date", ach, STANDARD_TYPE(StepBasic_Date), aAssignedDate);
+    data->ReadEntity(num, 1,"assigned_date", ach, STANDARD_TYPE(StepBasic_Date), aAssignedDate);
 
        // --- inherited field : role ---
 
        Handle(StepBasic_DateRole) aRole;
-#ifdef DEB
-       Standard_Boolean stat2 = 
-#endif
-         data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_DateRole), aRole);
+    data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_DateRole), aRole);
 
        // --- own field : items ---
 
index f1b31db..80dc92d 100755 (executable)
@@ -45,10 +45,7 @@ void RWStepAP214_RWAutoDesignApprovalAssignment::ReadStep
        // --- inherited field : assignedApproval ---
 
        Handle(StepBasic_Approval) aAssignedApproval;
-#ifdef DEB
-       Standard_Boolean stat1 = 
-#endif
-         data->ReadEntity(num, 1,"assigned_approval", ach, STANDARD_TYPE(StepBasic_Approval), aAssignedApproval);
+    data->ReadEntity(num, 1,"assigned_approval", ach, STANDARD_TYPE(StepBasic_Approval), aAssignedApproval);
 
        // --- own field : items ---
 
index c88301b..fadad73 100755 (executable)
@@ -46,18 +46,12 @@ void RWStepAP214_RWAutoDesignDateAndPersonAssignment::ReadStep
        // --- inherited field : assignedPersonAndOrganization ---
 
        Handle(StepBasic_PersonAndOrganization) aAssignedPersonAndOrganization;
-#ifdef DEB
-       Standard_Boolean stat1 = 
-#endif
-         data->ReadEntity(num, 1,"assigned_person_and_organization", ach, STANDARD_TYPE(StepBasic_PersonAndOrganization), aAssignedPersonAndOrganization);
+    data->ReadEntity(num, 1,"assigned_person_and_organization", ach, STANDARD_TYPE(StepBasic_PersonAndOrganization), aAssignedPersonAndOrganization);
 
        // --- inherited field : role ---
 
        Handle(StepBasic_PersonAndOrganizationRole) aRole;
-#ifdef DEB
-       Standard_Boolean stat2 = 
-#endif
-         data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_PersonAndOrganizationRole), aRole);
+    data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_PersonAndOrganizationRole), aRole);
 
        // --- own field : items ---
 
index 8448dee..8b0c751 100755 (executable)
@@ -38,10 +38,7 @@ void  RWStepAP214_RWAutoDesignDocumentReference::ReadStep
        // --- inherited field : assigned_document
 
        Handle(StepBasic_Document) adoc;
-#ifdef DEB
-       Standard_Boolean stat1 = 
-#endif
-         data->ReadEntity
+    data->ReadEntity
            (num, 1,"assigned_document", ach, STANDARD_TYPE(StepBasic_Document), adoc);
 
        // --- inherited field : source ---
index 84a0a0a..cefde0e 100755 (executable)
@@ -46,18 +46,12 @@ void RWStepAP214_RWAutoDesignNominalDateAndTimeAssignment::ReadStep
        // --- inherited field : assignedDateAndTime ---
 
        Handle(StepBasic_DateAndTime) aAssignedDateAndTime;
-#ifdef DEB
-       Standard_Boolean stat1 = 
-#endif
-         data->ReadEntity(num, 1,"assigned_date_and_time", ach, STANDARD_TYPE(StepBasic_DateAndTime), aAssignedDateAndTime);
+    data->ReadEntity(num, 1,"assigned_date_and_time", ach, STANDARD_TYPE(StepBasic_DateAndTime), aAssignedDateAndTime);
 
        // --- inherited field : role ---
 
        Handle(StepBasic_DateTimeRole) aRole;
-#ifdef DEB
-       Standard_Boolean stat2 = 
-#endif
-         data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_DateTimeRole), aRole);
+    data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_DateTimeRole), aRole);
 
        // --- own field : items ---
 
index 6a7c1b5..12bba23 100755 (executable)
@@ -46,18 +46,12 @@ void RWStepAP214_RWAutoDesignNominalDateAssignment::ReadStep
        // --- inherited field : assignedDate ---
 
        Handle(StepBasic_Date) aAssignedDate;
-#ifdef DEB
-       Standard_Boolean stat1 = 
-#endif
-         data->ReadEntity(num, 1,"assigned_date", ach, STANDARD_TYPE(StepBasic_Date), aAssignedDate);
+    data->ReadEntity(num, 1,"assigned_date", ach, STANDARD_TYPE(StepBasic_Date), aAssignedDate);
 
        // --- inherited field : role ---
 
        Handle(StepBasic_DateRole) aRole;
-#ifdef DEB
-       Standard_Boolean stat2 = 
-#endif
-         data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_DateRole), aRole);
+    data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_DateRole), aRole);
 
        // --- own field : items ---
 
index aacd91a..48e3729 100755 (executable)
@@ -46,18 +46,12 @@ void RWStepAP214_RWAutoDesignOrganizationAssignment::ReadStep
        // --- inherited field : assignedOrganization ---
 
        Handle(StepBasic_Organization) aAssignedOrganization;
-#ifdef DEB
-       Standard_Boolean stat1 = 
-#endif
-         data->ReadEntity(num, 1,"assigned_organization", ach, STANDARD_TYPE(StepBasic_Organization), aAssignedOrganization);
+    data->ReadEntity(num, 1,"assigned_organization", ach, STANDARD_TYPE(StepBasic_Organization), aAssignedOrganization);
 
        // --- inherited field : role ---
 
        Handle(StepBasic_OrganizationRole) aRole;
-#ifdef DEB
-       Standard_Boolean stat2 = 
-#endif
-         data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_OrganizationRole), aRole);
+    data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_OrganizationRole), aRole);
 
        // --- own field : items ---
 
index ec8e5d2..9e32a4a 100755 (executable)
@@ -46,18 +46,12 @@ void RWStepAP214_RWAutoDesignPersonAndOrganizationAssignment::ReadStep
        // --- inherited field : assignedPersonAndOrganization ---
 
        Handle(StepBasic_PersonAndOrganization) aAssignedPersonAndOrganization;
-#ifdef DEB
-       Standard_Boolean stat1 = 
-#endif
-         data->ReadEntity(num, 1,"assigned_person_and_organization", ach, STANDARD_TYPE(StepBasic_PersonAndOrganization), aAssignedPersonAndOrganization);
+    data->ReadEntity(num, 1,"assigned_person_and_organization", ach, STANDARD_TYPE(StepBasic_PersonAndOrganization), aAssignedPersonAndOrganization);
 
        // --- inherited field : role ---
 
        Handle(StepBasic_PersonAndOrganizationRole) aRole;
-#ifdef DEB
-       Standard_Boolean stat2 = 
-#endif
-         data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_PersonAndOrganizationRole), aRole);
+    data->ReadEntity(num, 2,"role", ach, STANDARD_TYPE(StepBasic_PersonAndOrganizationRole), aRole);
 
        // --- own field : items ---
 
index 9172bea..8c704d1 100755 (executable)
@@ -45,10 +45,7 @@ void RWStepAP214_RWAutoDesignSecurityClassificationAssignment::ReadStep
        // --- inherited field : assignedSecurityClassification ---
 
        Handle(StepBasic_SecurityClassification) aAssignedSecurityClassification;
-#ifdef DEB
-       Standard_Boolean stat1 = 
-#endif
-         data->ReadEntity(num, 1,"assigned_security_classification", ach, STANDARD_TYPE(StepBasic_SecurityClassification), aAssignedSecurityClassification);
+    data->ReadEntity(num, 1,"assigned_security_classification", ach, STANDARD_TYPE(StepBasic_SecurityClassification), aAssignedSecurityClassification);
 
        // --- own field : items ---
 
index 72f766a..bd184d4 100755 (executable)
@@ -49,10 +49,7 @@ void RWStepBasic_RWApplicationContextElement::ReadStep
        // --- own field : frameOfReference ---
 
        Handle(StepBasic_ApplicationContext) aFrameOfReference;
-#ifdef DEB
-       Standard_Boolean stat2 = 
-#endif
-         data->ReadEntity(num, 2,"frame_of_reference", ach, STANDARD_TYPE(StepBasic_ApplicationContext), aFrameOfReference);
+    data->ReadEntity(num, 2,"frame_of_reference", ach, STANDARD_TYPE(StepBasic_ApplicationContext), aFrameOfReference);
 
        //--- Initialisation of the read entity ---
 
index 5b16953..3431205 100755 (executable)
@@ -189,7 +189,8 @@ static Standard_Integer WhatKindOfLine(OSD_File& aFile,
   if (Debug)
     cout << "Key = '" << aToken1 << flush ;
 
-  if ((Pos = Line.FirstLocationNotInSet(WhiteSpace, Pos2+1, Line.Length()))) {
+  Pos = Line.FirstLocationNotInSet(WhiteSpace, Pos2+1, Line.Length());
+  if (Pos) {
     if (Line.Value(Pos) == '\\')
       switch(Line.Value(Pos+1)) {
       case '\\' :
index e1a4e05..6eb1906 100755 (executable)
@@ -2085,7 +2085,6 @@ void STEPCAFControl_Reader::ExpandSubShapes(const Handle(XCAFDoc_ShapeTool)& Sha
                                             const XCAFDoc_DataMapOfShapeLabel& ShapeLabelMap,
                                             const STEPCAFControl_DataMapOfShapePD& ShapePDMap) const
 {
-  const Handle(Interface_InterfaceModel)& Model = Reader().WS()->Model();
   const Handle(Transfer_TransientProcess)& TP = Reader().WS()->TransferReader()->TransientProcess();
   NCollection_DataMap<TopoDS_Shape, Handle(TCollection_HAsciiString)> ShapeNameMap;
   TColStd_MapOfTransient aRepItems;
index 7ac60e5..df6595f 100755 (executable)
@@ -91,7 +91,7 @@ Standard_Boolean Select3D_SensitiveEntity::Matches(const Standard_Real XMin,
 //=======================================================================
 
 Standard_Boolean Select3D_SensitiveEntity::Matches(const TColgp_Array1OfPnt2d& aPoly,
-                                                   const Bnd_Box2d& aBox,
+                                                   const Bnd_Box2d& /*aBox*/,
                                                    const Standard_Real aTol)
 {
   return Standard_False;
index 80997c1..4eb6417 100755 (executable)
@@ -366,7 +366,7 @@ static void SplitWire(const TopoDS_Wire& wire,
       ces.Append (i); statuses.SetValue (i, 1); //putting into CES
       Standard_Boolean SearchBackward = Standard_True;
 
-      while (Standard_True) {
+      for(;;) {
        Standard_Integer ei = ces.Last(); //ei-edge index, number of current edge analyzed for connection
        Standard_Boolean found;
        TopoDS_Edge edge;
index e2e2831..a0a41c9 100755 (executable)
@@ -2004,7 +2004,7 @@ void ShapeFix_ComposeShell::CollectWires (ShapeFix_SequenceOfWireSegment &wires,
   Standard_Integer iumin, iumax, ivmin, ivmax;
   Standard_Real dsu=0., dsv=0.;
   Standard_Boolean canBeClosed = Standard_False;
-  while ( 1 ) {
+  for(;;) {
     Standard_Integer index = 0;
     Standard_Boolean misoriented = Standard_True, samepatch = Standard_False;
     Standard_Boolean reverse = Standard_False, connected = Standard_False;
@@ -2731,7 +2731,7 @@ void ShapeFix_ComposeShell::DispatchWires (TopTools_SequenceOfShape &faces,
   // Collect wires in packets lying on same surface and dispatch them
   TColStd_Array1OfBoolean used ( 1, nb );
   used.Init ( Standard_False );
-  while ( 1 ) {
+  for(;;) {
     TopTools_SequenceOfShape loops;
 
     Handle(Geom_Surface) Surf;
index 4415733..babb72c 100755 (executable)
@@ -74,9 +74,10 @@ Standard_MMgrFactory::Standard_MMgrFactory()
 #endif*/
 
   char* aVar;
-  Standard_Integer anAllocId   = (aVar = getenv ("MMGT_OPT"      )) ?  atoi (aVar)       :
-    (OCCT_MMGT_OPT_DEFAULT);
-  Standard_Boolean toClear     = (aVar = getenv ("MMGT_CLEAR"    )) ? (atoi (aVar) != 0) : Standard_True;
+  aVar = getenv ("MMGT_OPT");
+  Standard_Integer anAllocId   = (aVar ?  atoi (aVar): OCCT_MMGT_OPT_DEFAULT);
+  aVar = getenv ("MMGT_CLEAR");
+  Standard_Boolean toClear     = (aVar ? (atoi (aVar) != 0) : Standard_True);
 
   // on Windows (actual for XP and 2000) activate low fragmentation heap
   // for CRT heap in order to get best performance.
@@ -95,10 +96,14 @@ Standard_MMgrFactory::Standard_MMgrFactory()
   {
     case 1:  // OCCT optimized memory allocator
     {
-      Standard_Boolean bMMap       = (aVar = getenv ("MMGT_MMAP"     )) ? (atoi (aVar) != 0) : Standard_True;
-      Standard_Integer aCellSize   = (aVar = getenv ("MMGT_CELLSIZE" )) ?  atoi (aVar) : 200;
-      Standard_Integer aNbPages    = (aVar = getenv ("MMGT_NBPAGES"  )) ?  atoi (aVar) : 1000;
-      Standard_Integer aThreshold  = (aVar = getenv ("MMGT_THRESHOLD")) ?  atoi (aVar) : 40000;
+      aVar = getenv ("MMGT_MMAP");
+      Standard_Boolean bMMap       = (aVar ? (atoi (aVar) != 0) : Standard_True);
+      aVar = getenv ("MMGT_CELLSIZE");
+      Standard_Integer aCellSize   = (aVar ?  atoi (aVar) : 200);
+      aVar = getenv ("MMGT_NBPAGES");
+      Standard_Integer aNbPages    = (aVar ?  atoi (aVar) : 1000);
+      aVar = getenv ("MMGT_THRESHOLD");
+      Standard_Integer aThreshold  = (aVar ?  atoi (aVar) : 40000);
       myFMMgr = new Standard_MMgrOpt (toClear, bMMap, aCellSize, aNbPages, aThreshold);
       break;
     }
index 332c9bf..bf34b1f 100755 (executable)
@@ -796,7 +796,7 @@ retry:
     HANDLE * aMBlock = NULL;
     // check for error and try allocating address space
     if ( ! hMap || GetLastError() == ERROR_ALREADY_EXISTS ||
-         ! (aMBlock = (HANDLE*)MapViewOfFile(hMap,FILE_MAP_WRITE,0,0,0)) ) 
+         ! ((aMBlock = (HANDLE*)MapViewOfFile(hMap,FILE_MAP_WRITE,0,0,0))) ) 
     {
       // close handle if allocated
       if ( hMap ) 
index 449b5dd..15d4bfd 100755 (executable)
@@ -251,7 +251,7 @@ void StdPrs_Curve::Add (const Handle (Prs3d_Presentation)& aPresentation,
 //==================================================================
 void StdPrs_Curve::Add (const Handle (Prs3d_Presentation)& aPresentation,
                         const Adaptor3d_Curve&                    aCurve,
-                        const Quantity_Length              aDeflection,
+                        const Quantity_Length              /*aDeflection*/,
                         const Handle(Prs3d_Drawer)&        aDrawer,
                         TColgp_SequenceOfPnt&           Points,
                         const Standard_Boolean drawCurve)
@@ -272,7 +272,7 @@ void StdPrs_Curve::Add (const Handle (Prs3d_Presentation)& aPresentation,
                         const Adaptor3d_Curve&                    aCurve,
                         const Standard_Real                U1,
                         const Standard_Real                U2,
-                        const Quantity_Length              aDeflection,
+                        const Quantity_Length              /*aDeflection*/,
                         TColgp_SequenceOfPnt&              Points,
                         const Standard_Integer             NbPoints,
                         const Standard_Boolean drawCurve)
index c653ab5..7fb355f 100644 (file)
@@ -49,7 +49,6 @@ void StdPrs_WFRestrictedFace::Add
                  const Handle(Prs3d_Drawer)&         theDrawer,
                  Prs3d_NListOfSequenceOfPnt&         theCurves)
 {
-  Standard_Real aLimit = theDrawer->MaximalParameterValue();
   Standard_Integer aNbPoints = theDrawer->Discretisation();
   StdPrs_ToolRFace aToolRst (theFace);
 
index 8276cc0..b6d459f 100755 (executable)
@@ -78,7 +78,7 @@ StepBasic_SiUnitName StepBasic_SiUnit::Name() const
 }
 
 
-void StepBasic_SiUnit::SetDimensions(const Handle(StepBasic_DimensionalExponents)& aDimensions)
+void StepBasic_SiUnit::SetDimensions(const Handle(StepBasic_DimensionalExponents)& /*aDimensions*/)
 {
   // WARNING : the field is redefined.
   // field set up forbidden.
index 6f05b5e..0031499 100755 (executable)
@@ -55,7 +55,7 @@ StepData_DefaultGeneral::StepData_DefaultGeneral ()
 void  StepData_DefaultGeneral::CheckCase(const Standard_Integer casenum,
                                          const Handle(Standard_Transient)& ent,
                                          const Interface_ShareTool& shares,
-                                         Handle(Interface_Check)& ach) const 
+                                         Handle(Interface_Check)& /*ach*/) const 
 {
 }  //  pas de Check sur une UndefinedEntity
 
index a1dee0a..5c689e4 100755 (executable)
@@ -37,7 +37,7 @@ StepData_DescrGeneral::StepData_DescrGeneral
 void StepData_DescrGeneral::CheckCase(const Standard_Integer CN,
                                       const Handle(Standard_Transient)& ent,
                                       const Interface_ShareTool& shares,
-                                      Handle(Interface_Check)& ach) const
+                                      Handle(Interface_Check)& /*ach*/) const
 {
 }    // pour l instant
 
index 3f25e3c..b2fd152 100755 (executable)
@@ -224,7 +224,7 @@ StepData_PDescr::StepData_PDescr  ()
 
 
 void StepData_PDescr::Check(const StepData_Field& afild,
-                            Handle(Interface_Check)& ach) const
+                            Handle(Interface_Check)& /*ach*/) const
 {
 //  pour l instant ...
 }
index 788a9f1..b2d28fa 100755 (executable)
@@ -78,7 +78,7 @@ StepData_Simple::StepData_Simple (const Handle(StepData_ESDescr)& descr)
       {  return thefields;  }
 
 
-void StepData_Simple::Check(Handle(Interface_Check)& ach) const
+void StepData_Simple::Check(Handle(Interface_Check)& /*ach*/) const
 {
 }  // qq chose ? cf la description
 
index f644b78..dd3f808 100755 (executable)
@@ -1606,7 +1606,7 @@ void StepData_StepReaderData::SetEntityNumbers(const Standard_Boolean withmap)
            Standard_Integer fromscope  = nr;
            Standard_Integer toscope    = indm(indmap);
            if (toscope < 0) toscope = -toscope;
-           while (1) {
+           for(;;) {
              fromscope --;    // iteration de base
              if (fromscope <= toscope) {
                errorscope = Standard_True;  // BANG, on est dessus
index e441c47..c7cd6c3 100755 (executable)
@@ -63,7 +63,7 @@ Standard_Boolean StepShape_OrientedClosedShell::Orientation() const
        return orientation;
 }
 
-void StepShape_OrientedClosedShell::SetCfsFaces(const Handle(StepShape_HArray1OfFace)& aCfsFaces)
+void StepShape_OrientedClosedShell::SetCfsFaces(const Handle(StepShape_HArray1OfFace)& /*aCfsFaces*/)
 {
        // WARNING : the field is redefined.
        // field set up forbidden.
index 414833a..cd49a4d 100755 (executable)
@@ -63,7 +63,7 @@ Standard_Boolean StepShape_OrientedFace::Orientation() const
        return orientation;
 }
 
-void StepShape_OrientedFace::SetBounds(const Handle(StepShape_HArray1OfFaceBound)& aBounds)
+void StepShape_OrientedFace::SetBounds(const Handle(StepShape_HArray1OfFaceBound)& /*aBounds*/)
 {
        // WARNING : the field is redefined.
        // field set up forbidden.
index 75117ee..7618189 100755 (executable)
@@ -63,7 +63,7 @@ Standard_Boolean StepShape_OrientedOpenShell::Orientation() const
        return orientation;
 }
 
-void StepShape_OrientedOpenShell::SetCfsFaces(const Handle(StepShape_HArray1OfFace)& aCfsFaces)
+void StepShape_OrientedOpenShell::SetCfsFaces(const Handle(StepShape_HArray1OfFace)& /*aCfsFaces*/)
 {
        // WARNING : the field is redefined.
        // field set up forbidden.
index 7436a45..16b9022 100755 (executable)
@@ -585,7 +585,7 @@ static Standard_Integer             i;
     }
 
     if (result) {
-      Standard_Integer otype,oref;
+      Standard_Integer otype, oref = 0;
 
       errorCode = f.BeginReadRefSection();
 
@@ -1364,7 +1364,7 @@ Standard_Boolean Storage_Schema::CheckTypeMigration(
        if(aFile.IsOpen() && aFile.IsReadable()) {
          TCollection_AsciiString aLine;
          Standard_Integer aNbReaded(0);
-         while (1) {
+         for (;;) {
            aFile.ReadLine(aLine, 80, aNbReaded);
            if(aFile.IsAtEnd() || !aNbReaded) {
              aFile.Close();
index 159c0c5..dc5ec63 100755 (executable)
@@ -35,7 +35,7 @@ extern Standard_Integer TColStd_intMapNode_findNext (const TColStd_intMapNode *,
 TColStd_MapIteratorOfPackedMapOfInteger::TColStd_MapIteratorOfPackedMapOfInteger
                                      (const TColStd_PackedMapOfInteger& theMap)
   : TCollection_BasicMapIterator (theMap),
-    myIntMask                    (~0)
+    myIntMask                    (~0U)
 {
   if (myNode) {
     const TColStd_intMapNode * aNode =
@@ -54,7 +54,7 @@ void TColStd_MapIteratorOfPackedMapOfInteger::Initialize
                                      (const TColStd_PackedMapOfInteger& theMap)
 {
   TCollection_BasicMapIterator::Initialize (theMap);
-  myIntMask = ~0;
+  myIntMask = ~0U;
   if (myNode) {
     const TColStd_intMapNode * aNode =
       reinterpret_cast <const TColStd_intMapNode *>(myNode);
@@ -70,7 +70,7 @@ void TColStd_MapIteratorOfPackedMapOfInteger::Initialize
 void TColStd_MapIteratorOfPackedMapOfInteger::Reset ()
 {
   TCollection_BasicMapIterator::Reset();
-  myIntMask = ~0;
+  myIntMask = ~0U;
   if (myNode) {
     const TColStd_intMapNode * aNode =
       reinterpret_cast <const TColStd_intMapNode *>(myNode);
index 544422d..fae5fe9 100755 (executable)
@@ -193,9 +193,9 @@ Standard_Integer TColStd_intMapNode_findNext (const TColStd_intMapNode* theNode,
   unsigned int val = theNode->myData & theMask;
   int nZeros (0);
   if (val == 0)
-    theMask = ~0;   // void, nothing to do
+    theMask = ~0U;   // void, nothing to do
   else{
-    unsigned int aMask = ~0;
+    unsigned int aMask = ~0U;
     if ((val & 0x0000ffff) == 0) {
       aMask = 0xffff0000;
       nZeros = 16;
@@ -237,9 +237,9 @@ Standard_Integer TColStd_intMapNode_findPrev (const TColStd_intMapNode* theNode,
   unsigned int val = theNode->myData & theMask;
   int nZeros (0);
   if (val == 0)
-    theMask = ~0;   // void, nothing to do
+    theMask = ~0U;   // void, nothing to do
   else {
-    unsigned int aMask = ~0;
+    unsigned int aMask = ~0U;
     if ((val & 0xffff0000) == 0) {
       aMask = 0x0000ffff;
       nZeros = 16;
@@ -647,7 +647,7 @@ Standard_Boolean TColStd_PackedMapOfInteger::Unite(const TColStd_PackedMapOfInte
         while (p1) {
           if (p1->IsEqual(aKeyInt)) {
             const size_t anOldPop = p1->NbValues();
-            Standard_Integer newData = p1->Data() | p2->Data();
+            unsigned int newData = p1->Data() | p2->Data();
             if ( newData != p1->Data() ) {
               p1->ChangeData() = newData;
               aNewExtent = aNewExtent - anOldPop +
index 8504eb5..63d4225 100755 (executable)
@@ -424,8 +424,8 @@ void TCollection_AsciiString::AssignCat(const TCollection_AsciiString& other)
 void TCollection_AsciiString::Capitalize()
 {
   for (int i=0; i < mylength; i++) {
-    if  ( i==0 ) mystring[i] = toupper(mystring[i]);
-    else         mystring[i] = tolower(mystring[i]);    
+    if  ( i==0 ) mystring[i] = ::UpperCase(mystring[i]);
+    else         mystring[i] = ::LowerCase(mystring[i]);    
   }
 }
 
@@ -457,9 +457,9 @@ void TCollection_AsciiString::ChangeAll(const Standard_Character aChar,
       if (mystring[i] == aChar) mystring[i] = NewChar;
   }
   else{
-    Standard_Character anUpperChar = toupper(aChar);
+    Standard_Character anUpperChar = ::UpperCase(aChar);
     for (int i=0; i < mylength; i++)
-      if (toupper(mystring[i]) == anUpperChar) mystring[i] = NewChar;
+      if (::UpperCase(mystring[i]) == anUpperChar) mystring[i] = NewChar;
   }
 }
 
@@ -1138,7 +1138,7 @@ Standard_Integer TCollection_AsciiString::Location
 void TCollection_AsciiString::LowerCase()
 {
   for (int i=0; i < mylength; i++)
-    mystring[i] = tolower(mystring[i]);
+    mystring[i] = ::LowerCase(mystring[i]);
 }
 
 //------------------------------------------------------------------------
@@ -1240,9 +1240,9 @@ void TCollection_AsciiString::RemoveAll(const Standard_Character what,
       if (mystring[i] != what) mystring[c++] = mystring[i];
   }
   else {
-    Standard_Character upperwhat = toupper(what);
+    Standard_Character upperwhat = ::UpperCase(what);
     for (int i=0; i < mylength; i++) { 
-      if (toupper(mystring[i]) != upperwhat) mystring[c++] = mystring[i];
+      if (::UpperCase(mystring[i]) != upperwhat) mystring[c++] = mystring[i];
     }
   }
   mylength = c;
@@ -1831,7 +1831,7 @@ void TCollection_AsciiString::Trunc(const Standard_Integer ahowmany)
 void TCollection_AsciiString::UpperCase()
 {
   for (int i=0; i < mylength; i++)
-    mystring[i] = toupper(mystring[i]);
+    mystring[i] = ::UpperCase(mystring[i]);
 }
 
 //------------------------------------------------------------------------
index 9a34559..1410eb9 100755 (executable)
@@ -312,7 +312,7 @@ void TDF_Attribute::RemoveBackup()
 //purpose  : Adds the referenced attributes or labels.
 //=======================================================================
 
-void TDF_Attribute::References(const Handle(TDF_DataSet)& aDataSet) const
+void TDF_Attribute::References(const Handle(TDF_DataSet)& /*aDataSet*/) const
 {}
 
 
index 6e7df22..dcb745a 100755 (executable)
@@ -37,7 +37,7 @@ Standard_Boolean TDataStd_Directory::Find (const TDF_Label& current,
   Handle(TDataStd_Directory) dir;
   if (L.IsNull()) return Standard_False; 
 
-  while (1) {
+  for(;;) {
     if(L.FindAttribute(TDataStd_Directory::GetID(), dir)) break; 
     L = L.Father();
     if (L.IsNull()) break; 
index 9c90472..826dc9b 100755 (executable)
@@ -39,7 +39,7 @@ Standard_Boolean TDataStd_NoteBook::Find (const TDF_Label& current,
   Handle(TDataStd_NoteBook) NB;
   if (L.IsNull()) return Standard_False; 
 
-  while (1) {
+  for(;;) {
     if(L.FindAttribute(TDataStd_NoteBook::GetID(), NB)) break; 
     L = L.Father();
     if (L.IsNull()) break; 
index e247f33..92714e2 100755 (executable)
@@ -42,7 +42,7 @@ Standard_Boolean TDataXtd_Shape::Find (const TDF_Label& current,
   TDF_Label L = current;
   Handle(TDataXtd_Shape) SA;
   if (L.IsNull()) return Standard_False; 
-  while (1) {
+  for(;;) {
     if(L.FindAttribute(TDataXtd_Shape::GetID(), SA))  break;
     L = L.Father();
     if (L.IsNull()) break; 
index 101719e..7669d37 100755 (executable)
@@ -172,7 +172,7 @@ Handle(TDF_Attribute) TFunction_Function::NewEmpty() const
 //purpose  : Collects the references
 //=======================================================================
 
-void TFunction_Function::References(const Handle(TDF_DataSet)& aDataSet) const
+void TFunction_Function::References(const Handle(TDF_DataSet)& /*aDataSet*/) const
 {
 }
 
index 69e851c..ea7c1b3 100755 (executable)
@@ -184,6 +184,6 @@ Standard_OStream& TNaming_UsedShapes::Dump(Standard_OStream& anOS) const
 //purpose  : 
 //=======================================================================
 
-void TNaming_UsedShapes::References(const Handle(TDF_DataSet)& aDataSet) const
+void TNaming_UsedShapes::References(const Handle(TDF_DataSet)& /*aDataSet*/) const
 {
 }
index c8c67eb..8b44afd 100755 (executable)
@@ -1965,7 +1965,7 @@ void TPrsStd_ConstraintTools::ComputePlacement
 //purpose  : 
 //=======================================================================
 void TPrsStd_ConstraintTools::ComputeOthers
-(const Handle(TDataXtd_Constraint)& aConst,
+(const Handle(TDataXtd_Constraint)& /*aConst*/,
  Handle(AIS_InteractiveObject)& anAIS) 
 {
 }
index 4b87abb..f1fb095 100755 (executable)
@@ -12,6 +12,5 @@ TestTopOpe_MesureCommands.cxx
 TestTopOpe_HDSDisplayer.hxx
 TestTopOpe_HDSDisplayer.cxx
 TestTopOpe_HDSCommands.cxx
-TestTopOpe_DSACommands.cxx
 TestTopOpe_CORCommands.cxx
 TestTopOpe_OtherCommands.cxx
index c9058a6..a4199b0 100755 (executable)
@@ -49,7 +49,7 @@ void  TestTopOpe::AllCommands(Draw_Interpretor& theCommands)
   TestTopOpe::HDSCommands(theCommands);
   TestTopOpe::MesureCommands(theCommands);
   TestTopOpe::CORCommands(theCommands);
-  TestTopOpe::DSACommands(theCommands);
+  //TestTopOpe::DSACommands(theCommands);
   TestTopOpe::OtherCommands(theCommands);
   TestTopOpeTools::AllCommands(theCommands);
   TestTopOpeDraw::AllCommands(theCommands);
index 694ea03..d1dd627 100755 (executable)
@@ -827,9 +827,7 @@ static Standard_Integer normal(Draw_Interpretor& di, Standard_Integer n, const c
 //  TopoDS_Face f = TopoDS::Face(DBRep::Get(a[1]));
   if (f.IsNull()) {di<<"null shape"<<"\n";return 1;}
   gp_Pnt p; DrawTrSurf::GetPoint(a[2], p);
-#ifdef DEB
-  Standard_Real length = Draw::Atof(a[3]);
-#endif
+
   Standard_Real dist=0.; gp_Pnt2d uv; Standard_Boolean ok = FUN_tool_projPonF(p,f,uv,dist);
   if (!ok) {di<<"projection failed"<<"\n"; return 1;}
 #ifdef DEB
diff --git a/src/TestTopOpe/TestTopOpe_DSACommands.cxx b/src/TestTopOpe/TestTopOpe_DSACommands.cxx
deleted file mode 100755 (executable)
index 11da71f..0000000
+++ /dev/null
@@ -1,167 +0,0 @@
-// Created on: 1997-12-01
-// Created by: Prestataire Mary FABIEN
-// Copyright (c) 1997-1999 Matra Datavision
-// Copyright (c) 1999-2012 OPEN CASCADE SAS
-//
-// The content of this file is subject to the Open CASCADE Technology Public
-// License Version 6.5 (the "License"). You may not use the content of this file
-// except in compliance with the License. Please obtain a copy of the License
-// at http://www.opencascade.org and read it completely before using this file.
-//
-// The Initial Developer of the Original Code is Open CASCADE S.A.S., having its
-// main offices at: 1, place des Freres Montgolfier, 78280 Guyancourt, France.
-//
-// The Original Code and all software distributed under the License is
-// distributed on an "AS IS" basis, without warranty of any kind, and the
-// Initial Developer hereby disclaims all such warranties, including without
-// limitation, any warranties of merchantability, fitness for a particular
-// purpose or non-infringement. Please see the License for the specific terms
-// and conditions governing the rights and limitations under the License.
-
-
-
-#include <TestTopOpe.ixx>
-#include <BRepAlgo_DSAccess.hxx>
-#include <DBRep.hxx>
-#include <TopOpeBRepDS_define.hxx>
-
-Standard_Integer DSACCESSCOMMANDS(Draw_Interpretor& interpretor,Standard_Integer na, const char** a)
-{ 
-  static BRepAlgo_DSAccess DSA;
-//  Standard_Integer i;
-
-  if (!strcmp(a[0],"dsload"))
-    if (na == 3) {
-      TopoDS_Shape SS1 = DBRep::Get(a[1]);
-      TopoDS_Shape& S1 = SS1;
-      TopoDS_Shape SS2 = DBRep::Get(a[2]);
-      TopoDS_Shape& S2 = SS2;
-      TestTopOpe::CurrentDS(DSA.DS());
-      DSA.Load(S1, S2);
-      return 0;
-    }
-    else 
-      return 0;
-
-  else if(!strcmp(a[0],"dsinit")) {
-    DSA.Init();
-  }
-  else if(!strcmp(a[0],"dsinters"))
-    if (na == 4)
-      if(!strcmp(a[1],"-sd")) {
-       const TopoDS_Shape& S1 = DBRep::Get(a[2]);
-       const TopoDS_Shape& S2 = DBRep::Get(a[3]);
-       DSA.SameDomain(S1, S2);
-      }
-      else 
-       return 0;
-    else {
-      if(na == 3) {
-       const TopoDS_Shape& S1 = DBRep::Get(a[1]);
-       const TopoDS_Shape& S2 = DBRep::Get(a[2]);
-       DSA.Intersect(S1, S2);
-      }
-      else
-       if(na == 1) 
-         DSA.Intersect();
-    }
-  else if(!strcmp(a[0],"dsgetsect")) {
-    if((na == 2) || (na == 4)) {
-      TopoDS_Shape EmptyShape;
-      const TopoDS_Shape& S1 = ((na == 2) ? EmptyShape : DBRep::Get(a[2]));
-      const TopoDS_Shape& S2 = ((na == 2) ? EmptyShape : DBRep::Get(a[3]));
-      const TopTools_ListOfShape& list = 
-       ((na == 2) ? DSA.GetSectionEdgeSet() : 
-        DSA.GetSectionEdgeSet(S1, S2));
-      Standard_Integer index = 1;
-      if(list.Extent() > 1) {
-       TopTools_ListIteratorOfListOfShape it(list);
-       for(; it.More(); it.Next(), index++) {
-         TCollection_AsciiString enam(a[1]);
-         enam=enam + "_" + TCollection_AsciiString(index);
-         DBRep::Set(enam.ToCString(),it.Value());
-       }
-      }
-      else
-       if(list.Extent() == 1) {
-         TCollection_AsciiString enam(a[1]);
-         DBRep::Set(enam.ToCString(),list.First());
-       }       
-       else
-         return 0;
-    }
-  }
-  
-  else if(!strcmp(a[0],"dssuppsect")) {
-    if(na == 2) {
-      const TopoDS_Shape& S = DBRep::Get(a[1]);
-      DSA.SuppressEdgeSet(S);
-    }
-    else
-      return 0;
-  }
-  
-  else if(!strcmp(a[0],"dsperform")) {
-    if(na == 3) {
-      TopoDS_Shape S;
-      TCollection_AsciiString enam(a[1]);
-      if(!strcmp(enam.ToCString(),"fus")) 
-       S = DSA.Merge(TopAbs_OUT, TopAbs_OUT);
-      else if(!strcmp(enam.ToCString(),"c12")) 
-       S = DSA.Merge(TopAbs_OUT, TopAbs_IN);
-      else if(!strcmp(enam.ToCString(),"c21")) 
-       S = DSA.Merge(TopAbs_IN, TopAbs_OUT);
-      else if(!strcmp(enam.ToCString(),"com")) 
-       S = DSA.Merge(TopAbs_IN, TopAbs_IN);
-      DBRep::Set(a[2], S);
-      TopOpeBRepDS_DataStructure& DS = DSA.DS()->ChangeDS();
-      Standard_Integer i,j,nbsh = DS.NbShapes();
-      for(i = 1;i<=nbsh;i++) {
-       TopTools_ListOfShape& los = DS.ChangeShapeSameDomain(i);
-       j = los.Extent();
-
-       TopTools_ListIteratorOfListOfShape li(los);
-       for(; li.More(); li.Next()) {
-#ifdef DEB
-         const TopoDS_Shape& Shap =
-#endif
-                                     li.Value();
-       }
-      }
-    }
-    else
-      return 0;
-  }
-  
-  else if(!strcmp(a[0],"dshelp")) {
-
-    interpretor<<"\n";
-    interpretor<<"  dsinit"<<"\n";
-    interpretor<<"  dsload shape1 shape2"<<"\n";
-    interpretor<<"  dsinters [-sd] [f1 f2]"<<"\n";
-    interpretor<<"    if -sd, the faces are samedomain"<<"\n";
-    interpretor<<"    no argument = intersect two input Shapes"<<"\n";
-    interpretor<<"  dsgetsect result [f1 f2] (result = compound)"<<"\n";
-    interpretor<<"  dssuppsect comp"<<"\n";
-    interpretor<<"  dsperform fus/c12/c21/com resultShape "<<"\n";
-    interpretor<<"\n";
-
-    return 0;
-  }
-  return 0;
-}
-//=======================================================================
-//function : BOOPCommands
-//purpose  : 
-//=======================================================================
-void TestTopOpe::DSACommands(Draw_Interpretor& theCommands)
-{
-  const char* g = "TestTopOpe DSACommands";
-  theCommands.Add("dsinit","Taper dshelp",__FILE__,DSACCESSCOMMANDS,g);
-  theCommands.Add("dsload","Taper dshelp",__FILE__,DSACCESSCOMMANDS,g);
-  theCommands.Add("dsinters","Taper dshelp",__FILE__,DSACCESSCOMMANDS,g);
-  theCommands.Add("dsgetsect","Taper dshelp",__FILE__,DSACCESSCOMMANDS,g);
-  theCommands.Add("dssuppsect","Taper dshelp",__FILE__,DSACCESSCOMMANDS,g);
-  theCommands.Add("dsperform","Taper dshelp",__FILE__,DSACCESSCOMMANDS,g);
-  theCommands.Add("dshelp","Taper dshelp",__FILE__,DSACCESSCOMMANDS,g);
-}
index ecf06fa..f80ccd2 100755 (executable)
@@ -227,7 +227,7 @@ static void BoopReadInitFile(Draw_Interpretor& di, const char* filename)
   di.EvalFile(filename);
 }
 
-static Standard_Integer ttab(Draw_Interpretor& di, Standard_Integer narg, const char** a) {
+static Standard_Integer ttab(Draw_Interpretor& di, Standard_Integer narg, const char** /*a*/) {
   char s[2000]; strcpy(s,"");
   Sprintf(s,"%s%s",s,"proc addt {at args} {upvar $at x;set L \"\";addl L [join $args];");
   Sprintf(s,"%s%s",s,"foreach l $L {set x([array size x]) $l}};");
index 849bf1f..0f4e16c 100755 (executable)
@@ -73,7 +73,7 @@ void TopOpeBRep_EdgesFiller::Insert(const TopoDS_Shape& E1,const TopoDS_Shape& E
   myLI1.Clear();
   myLI2.Clear();
   myHDS = HDS;
-
 #ifdef DEB
   Standard_Boolean trc = TopOpeBRepDS_GettraceDSF();
   trc = trc || TopOpeBRepDS_GettraceEDSF();
@@ -84,20 +84,11 @@ void TopOpeBRep_EdgesFiller::Insert(const TopoDS_Shape& E1,const TopoDS_Shape& E
     myPEI->Dump(str,myPDS->Shape(myE1),myPDS->Shape(myE2));
   }
 #endif
-  
-#ifdef DEB
-  Standard_Boolean hs =
-#endif
-           myPEI->HasSegment();
   Standard_Boolean esd = myPEI->SameDomain();
   if (esd) myPDS->FillShapesSameDomain(E1,E2);
   
   // exit if no point.
   myPEI->InitPoint(); if ( !myPEI->MorePoint() ) return;
-  
-#ifdef DEB
-  Standard_Boolean reducesegment = (hs && !esd);
-#endif
 
   // --- Add <E1,E2> in BDS
   Standard_Integer E1index = myPDS->AddShape(E1,1);
@@ -119,9 +110,8 @@ void TopOpeBRep_EdgesFiller::Insert(const TopoDS_Shape& E1,const TopoDS_Shape& E
     Standard_Integer if2 = 0; if ( ! myF2.IsNull() ) if2 = myPDS->AddShape(myF2,2);
 
 #ifdef DEB
-    Standard_Boolean pointofsegment =
+    Standard_Boolean pointofsegment = P2D.IsPointOfSegment();
 #endif
-                         P2D.IsPointOfSegment();
 
 #ifdef DEB
     if (trc) {
@@ -129,7 +119,7 @@ void TopOpeBRep_EdgesFiller::Insert(const TopoDS_Shape& E1,const TopoDS_Shape& E
       else if (pointofsegment && !esd) debposnesd();
     }
 #endif
-    
+
     TopOpeBRepDS_Transition T1 = P2D.Transition(1);
     TopOpeBRepDS_Transition T2 = P2D.Transition(2);
     
@@ -503,10 +493,6 @@ void TopOpeBRep_EdgesFiller::RecomputeInterferences(const TopoDS_Edge& E,TopOpeB
     Standard_Integer ifb = TU.IndexBefore();
     Standard_Integer ifa = TU.IndexAfter();
     const TopoDS_Face& fb = TopoDS::Face(myPDS->Shape(ifb));
-#ifdef DEB
-    const TopoDS_Face& fa =
-#endif
-                    TopoDS::Face(myPDS->Shape(ifa));
 
 #ifdef DEB
     if (ifb != ifa) {cout<<"TopOpeBRep_EdgesFiller : ifb != ifa on E"<<EIX<<" NYI"<<endl;}
@@ -515,10 +501,11 @@ void TopOpeBRep_EdgesFiller::RecomputeInterferences(const TopoDS_Edge& E,TopOpeB
     Standard_Real pE = FDS_Parameter(iloi); TopOpeBRepDS_Transition TN;
     TN.ShapeBefore(TU.ShapeBefore());TN.IndexBefore(TU.IndexBefore());
     TN.ShapeAfter(TU.ShapeAfter());TN.IndexAfter(TU.IndexAfter());
+
 #ifdef DEB
     Standard_Boolean ok =
 #endif
-             FDS_stateEwithF2d(*myPDS,E,pE,K,G,fb,TN);
+    FDS_stateEwithF2d(*myPDS,E,pE,K,G,fb,TN);
 
 #ifdef DEB
     if (TRC) {
index 61f9ea5..e5c0572 100755 (executable)
@@ -371,15 +371,9 @@ Standard_Boolean EdgesIntersector_checkT1D(const TopoDS_Edge& E1,const TopoDS_Ed
        TopoDS_Vertex vf,vl; TopExp::Vertices(myEdge1,vf,vl);
        gp_Pnt ptf = BRep_Tool::Pnt(vf); Standard_Real df = pt2.Distance(ptf); 
        gp_Pnt ptl = BRep_Tool::Pnt(vl);
-#ifdef DEB
-        Standard_Real dl =
-#endif
-                 pt2.Distance(ptl); 
+
        Standard_Real tolf = BRep_Tool::Tolerance(vf);
-#ifdef DEB
-        Standard_Real toll =
-#endif
-                   BRep_Tool::Tolerance(vl);
+
        Standard_Boolean onf = (df < tolf);
        TopoDS_Vertex v1 = onf ? vf : vl;
        TopTools_IndexedDataMapOfShapeListOfShape mapVE; TopExp::MapShapesAndAncestors(myFace1,TopAbs_VERTEX,TopAbs_EDGE,mapVE);
@@ -557,9 +551,7 @@ Standard_Boolean EdgesIntersector_checkT1D(const TopoDS_Edge& E1,const TopoDS_Ed
       Standard_Boolean isvertex11 = isvertex1 && !isvertex12;
 
       Standard_Boolean T1INT = (T1.Orientation(TopAbs_IN) == TopAbs_INTERNAL);
-#ifdef DEB
-      Standard_Boolean T1EXT = (T1.Orientation(TopAbs_IN) == TopAbs_EXTERNAL);
-#endif
+
       if (T1INT && isvertex2 && !isvertex1) {
        const TopoDS_Vertex& V2 = P2D.Vertex(2);        
        TopOpeBRepDS_Transition newT; Standard_Boolean computed = ::EdgesIntersector_checkT1D(myEdge1,myEdge2,V2,newT);
index 48f9200..381a433 100755 (executable)
@@ -79,9 +79,6 @@ void TopOpeBRep_FaceEdgeFiller::Insert
 
   // --- Add <FF,EE> in BDS
   Standard_Integer FFindex = BDS.AddShape(FF,1);
-#ifdef DEB
-  Standard_Integer EEindex =
-#endif
                              BDS.AddShape(EE,2);
 
   // --- get list of interferences connected to edges <FF>,<EE>
index 63446a6..eb6b403 100755 (executable)
@@ -248,10 +248,6 @@ static Standard_Boolean FUN_findTF(const TopOpeBRepDS_PDataStructure pDS,
     if (!OOdone) return Standard_False;
 
     const TopoDS_Edge& E   = TopoDS::Edge(pDS->Shape(iE));
-#ifdef DEB
-    const TopoDS_Face& F   =
-#endif
-                     TopoDS::Face(pDS->Shape(iF));
     const TopoDS_Face& OOF = TopoDS::Face(pDS->Shape(iOOF));
 
     Standard_Real f,l; FUN_tool_bounds(E,f,l);
@@ -943,7 +939,6 @@ void TopOpeBRep_FacesFiller::FillLineVPonR()
   Standard_Boolean tDSF = TopOpeBRepDS_GettraceDSF();
   Standard_Boolean trline = Standard_False;
 #endif
-
   // if a VP is on degenerated edge, adds the triplet
   // (vertex, closing edge, degenerated edge) to the
   // map as vertex for key.
@@ -953,10 +948,10 @@ void TopOpeBRep_FacesFiller::FillLineVPonR()
   
   mykeptVPnbr = 0; 
   
-  if (myLine->TypeLineCurve() == TopOpeBRep_RESTRICTION) {
+  if (myLine->TypeLineCurve() == TopOpeBRep_RESTRICTION) { 
 #ifdef DEB
     if (trline) FUN_traceRLine(*myLine);
-#endif    
+#endif
     ProcessRLine();
     return;
   }
@@ -966,11 +961,10 @@ void TopOpeBRep_FacesFiller::FillLineVPonR()
   if ( nINON == 0 ) {
     return; 
   }
-  
+   
 #ifdef DEB
   if (trline) FUN_traceGLine(*myLine);
 #endif
-  
   myLineIsonEdge = LSameDomainERL(*myLine, myERL);
   
   // walking (case mouch1a 1 1) : line (vpfirst on 3,vplast on 0,nvpkept = 2) => kept
@@ -982,9 +976,7 @@ void TopOpeBRep_FacesFiller::FillLineVPonR()
     else cout <<" geometric line not on edge"<<endl;
   }
 #endif
-  
-  //----------------------------------------------------------------------
-  // IMPORTANT : 
+  //----------------------------------------------------------------------  // IMPORTANT : 
   // Some of Curve/Point transitions for vpoints keep on RESTRICTION lines
   // sharing same domain with the current geometric line are computed here
   //----------------------------------------------------------------------
@@ -1004,9 +996,6 @@ void TopOpeBRep_FacesFiller::FillLineVPonR()
   }
   
   if ( myLineIsonEdge && (!myDSCIL.IsEmpty()) ) {
-#ifdef DEB
-    if (tDSF) cout<<"myLineIsonEdge && (!myDSCIL.IsEmpty())"<<endl;
-#endif
     myDSCIL.Clear();
   }
 }
index 980edbd..f570e18 100755 (executable)
@@ -52,15 +52,12 @@ TopOpeBRep_Hctxee2d::TopOpeBRep_Hctxee2d()
 //purpose  : 
 //=======================================================================
 void TopOpeBRep_Hctxee2d::SetEdges(const TopoDS_Edge& E1,const TopoDS_Edge& E2,
-                                  const BRepAdaptor_Surface& BAS1,
-                                  const BRepAdaptor_Surface& BAS2)
+                                   const BRepAdaptor_Surface& BAS1,
+                                   const BRepAdaptor_Surface& BAS2)
 {
   const TopoDS_Face& F1 = BAS1.Face();
   GeomAbs_SurfaceType ST1 = BAS1.GetType();
   const TopoDS_Face& F2 = BAS2.Face();
-#ifdef DEb
-  GeomAbs_SurfaceType ST2 = BAS2.GetType();
-#endif
   
   myEdge1 = TopoDS::Edge(E1);
   myEdge2 = TopoDS::Edge(E2);
@@ -75,14 +72,14 @@ void TopOpeBRep_Hctxee2d::SetEdges(const TopoDS_Edge& E1,const TopoDS_Edge& E2,
   BRep_Tool::UVPoints(myEdge1,F1,pfirst,plast);
   tole = BRep_Tool::Tolerance(myEdge1);
   myDomain1.SetValues(pfirst,first,tole,plast,last,tole);
-  
+
 #ifdef DEB
   Standard_Boolean trc = Standard_False;
   if (trc) {
     cout<<"ed1 on fa1 : {pfirst=("<<pfirst.X()<<" "<<pfirst.Y()<<"),first="<<first<<"\n";
     cout<<"              plast =("<<plast.X()<<" "<<plast.Y()<<"),last="<<last<<"}"<<endl;}
-#endif  
-  
+#endif
+
   Standard_Boolean memesfaces = F1.IsSame(F2);
   Standard_Boolean memesupport = Standard_False;
   TopLoc_Location L1,L2;
@@ -96,19 +93,20 @@ void TopOpeBRep_Hctxee2d::SetEdges(const TopoDS_Edge& E1,const TopoDS_Edge& E2,
     BRep_Tool::UVPoints(myEdge2,F1,pfirst,plast);
     tole = BRep_Tool::Tolerance(myEdge2);
     myDomain2.SetValues(pfirst,first,tole,plast,last,tole);
-    
+
 #ifdef DEB
     if (trc) {
       cout<<"ed2 on fa1 : {pfirst=("<<pfirst.X()<<" "<<pfirst.Y()<<"),first="<<first<<"\n";
-      cout<<"              plast =("<<plast.X()<<" "<<plast.Y()<<"),last="<<last<<"}"<<endl;}
+      cout<<"              plast =("<<plast.X()<<" "<<plast.Y()<<"),last="<<last<<"}"<<endl;
+    }
 #endif
-    
   }
   else {
 
     Handle(Geom2d_Curve) PC2on1; Handle(Geom_Curve) NC;
     Standard_Boolean dgE2 = BRep_Tool::Degenerated(myEdge2);
-    if (dgE2) { //xpu210998 : cto900Q3
+    if (dgE2)
+    { //xpu210998 : cto900Q3
       TopExp_Explorer exv(myEdge2,TopAbs_VERTEX);
       const TopoDS_Vertex& v2 = TopoDS::Vertex(exv.Current());
       gp_Pnt pt2 = BRep_Tool::Pnt(v2);
@@ -117,30 +115,21 @@ void TopOpeBRep_Hctxee2d::SetEdges(const TopoDS_Edge& E1,const TopoDS_Edge& E2,
       Handle(Geom_Surface) S1 = BRep_Tool::Surface(F1);
       Standard_Boolean apex = FUN_tool_onapex(uv2,S1);
       if (apex) {
-       TopoDS_Vertex vf,vl; TopExp::Vertices(myEdge1,vf,vl);
-       gp_Pnt ptf = BRep_Tool::Pnt(vf); Standard_Real df = pt2.Distance(ptf); 
-       gp_Pnt ptl = BRep_Tool::Pnt(vl);
-#ifdef DEB
-        Standard_Real dl =
-#endif
-                 pt2.Distance(ptl); 
-       Standard_Real tolf = BRep_Tool::Tolerance(vf);
-#ifdef DEB
-        Standard_Real toll =
-#endif
-                   BRep_Tool::Tolerance(vl);
-       Standard_Boolean onf = (df < tolf);
-       TopoDS_Vertex v1 = onf ? vf : vl;
-       TopTools_IndexedDataMapOfShapeListOfShape mapVE; TopExp::MapShapesAndAncestors(F1,TopAbs_VERTEX,TopAbs_EDGE,mapVE);
-       const TopTools_ListOfShape& Edsanc = mapVE.FindFromKey(v1);
-       TopTools_ListIteratorOfListOfShape it(Edsanc);
-       for (; it.More(); it.Next()){
-         const TopoDS_Edge& ee = TopoDS::Edge(it.Value());
-         Standard_Boolean dgee = BRep_Tool::Degenerated(ee);
-         if (!dgee) continue;
-//       Standard_Real f,l;
-          PC2on1 = BRep_Tool::CurveOnSurface(ee,F1,first,last);          
-       }
+        TopoDS_Vertex vf,vl; TopExp::Vertices(myEdge1,vf,vl);
+        gp_Pnt ptf = BRep_Tool::Pnt(vf); Standard_Real df = pt2.Distance(ptf); 
+        gp_Pnt ptl = BRep_Tool::Pnt(vl);
+        Standard_Real tolf = BRep_Tool::Tolerance(vf);
+        Standard_Boolean onf = (df < tolf);
+        TopoDS_Vertex v1 = onf ? vf : vl;
+        TopTools_IndexedDataMapOfShapeListOfShape mapVE; TopExp::MapShapesAndAncestors(F1,TopAbs_VERTEX,TopAbs_EDGE,mapVE);
+        const TopTools_ListOfShape& Edsanc = mapVE.FindFromKey(v1);
+        TopTools_ListIteratorOfListOfShape it(Edsanc);
+        for (; it.More(); it.Next()){
+          const TopoDS_Edge& ee = TopoDS::Edge(it.Value());
+          Standard_Boolean dgee = BRep_Tool::Degenerated(ee);
+          if (!dgee) continue;
+          PC2on1 = BRep_Tool::CurveOnSurface(ee,F1,first,last);
+        }
       }
       else {} // NYIxpu210998
     } //dgE2
@@ -160,12 +149,12 @@ void TopOpeBRep_Hctxee2d::SetEdges(const TopoDS_Edge& E1,const TopoDS_Edge& E2,
       myDomain2.SetValues(pfirst,first,tole,plast,last,tole);
 #ifdef DEB
       if ( TopOpeBRep_GettracePROEDG() ) {
-       cout<<"------------ projection de curve"<<endl;
-       cout<<"--- Curve : "<<endl;GeomTools_CurveSet::PrintCurve(NC,cout);
-       cout<<"--- nouvelle PCurve : "<<endl;GeomTools_Curve2dSet::PrintCurve2d(PC2on1,cout);
-       Handle(Geom_Surface) S1 = BRep_Tool::Surface(F1);
-       cout<<"--- sur surface : "<<endl;GeomTools_SurfaceSet::PrintSurface(S1,cout);
-       cout<<endl;
+        cout<<"------------ projection de curve"<<endl;
+        cout<<"--- Curve : "<<endl;GeomTools_CurveSet::PrintCurve(NC,cout);
+        cout<<"--- nouvelle PCurve : "<<endl;GeomTools_Curve2dSet::PrintCurve2d(PC2on1,cout);
+        Handle(Geom_Surface) S1 = BRep_Tool::Surface(F1);
+        cout<<"--- sur surface : "<<endl;GeomTools_SurfaceSet::PrintSurface(S1,cout);
+        cout<<endl;
       }
 #endif
     }
index 9adff7d..9d051a2 100755 (executable)
@@ -99,8 +99,7 @@ void TopOpeBRep_Point2d::Dump(const Standard_Integer E1index,const Standard_Inte
   Standard_Boolean keep = Keep();
   Standard_Integer sts = Status();
   Standard_Boolean pos = IsPointOfSegment();
-  Standard_Real tol = Tolerance();
-  
+
   const TopOpeBRepDS_Transition& T1 = Transition(1);
   const TopOpeBRepDS_Transition& T2 = Transition(2);
   
index af1696c..229a0c0 100755 (executable)
@@ -164,14 +164,13 @@ TopAbs_State TopOpeBRep_FacesFiller::StBipVPonF
   BRepAdaptor_Curve BC( arc );     
   Standard_Real x = 0.789; Standard_Real parmil = (1-x)*uf + x*ul; //xpu170898
   gp_Pnt pmil = BC.Value(parmil);
-  
+
 #ifdef DEB
   Standard_Boolean trc = TopOpeBRep_GettraceBIPS();
-#ifdef DRAW
-  if (trc) {TCollection_AsciiString aa("pmil"); FUN_brep_draw(aa,pmil);}
+    #ifdef DRAW
+      if (trc) {TCollection_AsciiString aa("pmil"); FUN_brep_draw(aa,pmil);}
+    #endif
 #endif
-#endif
   TopAbs_State st = FSC_StatePonFace (pmil,F,*myPShapeClassifier);
   return st;
 }
@@ -218,10 +217,6 @@ void TopOpeBRep_FacesFiller::Lminmax(const TopOpeBRep_LineInter& L,
   VPI.Init(L,Standard_False);
   for (; VPI.More(); VPI.Next()) {
     const TopOpeBRep_VPointInter& VP = VPI.CurrentVP();
-#ifdef DEB
-    Standard_Integer iVP =
-#endif
-              VPI.CurrentVPIndex();
     Standard_Real p = VP.ParameterOnLine();
     pmin = Min(pmin,p);
     pmax = Max(pmax,p);
index 3522dda..02cf39b 100755 (executable)
@@ -68,10 +68,6 @@ void TopOpeBRep_VPointInterIterator::Init()
   if ( mycheckkeep ) {
     while ( More() ) { 
       const TopOpeBRep_VPointInter& VP = CurrentVP();
-#ifdef DEB
-      Standard_Integer iVP = 
-#endif
-       CurrentVPIndex();
       if (VP.Keep()) break;
       else myVPointIndex++;
     }
@@ -99,10 +95,6 @@ void  TopOpeBRep_VPointInterIterator::Next()
   if ( mycheckkeep ) {
     while ( More() ) { 
       const TopOpeBRep_VPointInter& VP = CurrentVP();
-#ifdef DEB
-      Standard_Integer iVP = 
-#endif
-       CurrentVPIndex();
       if (VP.Keep()) break;
       else myVPointIndex++;
     }
index 414ea57..674c7f4 100755 (executable)
@@ -156,10 +156,11 @@ Standard_Boolean TopOpeBRepTool_CORRISO::Init(const TopoDS_Shape& S)
     const TopoDS_Edge& E = TopoDS::Edge(ex.Current());
 #ifdef DEB
     Standard_Integer iE = STATIC_PURGE_mapeds.Add(E);
-#ifdef DRAW
-    if (trc) {TCollection_AsciiString aa = TCollection_AsciiString("e"); FUN_tool_draw(aa,E,iE);}
-#endif
+    #ifdef DRAW
+        if (trc) {TCollection_AsciiString aa = TCollection_AsciiString("e"); FUN_tool_draw(aa,E,iE);}
+    #endif
 #endif
+
     // myEds :
     myEds.Append(E);
 
@@ -176,12 +177,12 @@ Standard_Boolean TopOpeBRepTool_CORRISO::Init(const TopoDS_Shape& S)
     // myVEds :
     TopExp_Explorer exv(E, TopAbs_VERTEX);
     for (; exv.More(); exv.Next()){
-      const TopoDS_Vertex& v = TopoDS::Vertex(exv.Current());   
+      const TopoDS_Vertex& v = TopoDS::Vertex(exv.Current());
 #ifdef DEB
-      Standard_Integer iv = STATIC_PURGE_mapv.Add(v);
-#ifdef DRAW
-      if (trc) {TCollection_AsciiString bb = TCollection_AsciiString("v"); FUN_tool_draw(bb,v,iv);}
-#endif      
+      Standard_Integer iE = STATIC_PURGE_mapeds.Add(E);
+      #ifdef DRAW
+        if (trc) {TCollection_AsciiString bb = TCollection_AsciiString("v"); FUN_tool_draw(bb,v,iv);}
+      #endif
 #endif
       Standard_Boolean isb = myVEds.IsBound(v);
       if (isb) myVEds.ChangeFind(v).Append(E);
@@ -344,15 +345,8 @@ static Standard_Boolean FUN_isonOcE(const TopOpeBRepTool_CORRISO CO, const TopoD
     TopOpeBRepTool_C2DF OcE2d; Standard_Boolean isOb = CO.UVRep(OcE,OcE2d);
     if (!isOb) return Standard_False; // NYIRAISE
     
-#ifdef DEB
-    const TopoDS_Vertex& vce1 =
-#endif
-                      TopoDS::Vertex(vcE(1)); 
     Standard_Real parvce1 = TopOpeBRepTool_TOOL::ParE(1,cE);   gp_Pnt2d UVvce1 = TopOpeBRepTool_TOOL::UVF(parvce1,cE2d);
-#ifdef DEB
-    const TopoDS_Vertex& vOce2 =
-#endif
-                       TopoDS::Vertex(vOcE(2)); 
+
     Standard_Real parvOcE2 = TopOpeBRepTool_TOOL::ParE(2,OcE); gp_Pnt2d UVvOcE2 = TopOpeBRepTool_TOOL::UVF(parvOcE2,OcE2d);
     Standard_Real tol = Max(tttuvcE,tttuvOcE);
     isonOcE2d = (UVvce1.Distance(UVvOcE2) < tol);
@@ -419,20 +413,13 @@ Standard_Boolean TopOpeBRepTool_CORRISO::PurgeFyClosingE(const TopTools_ListOfSh
       TopOpeBRepTool_C2DF OcE2d; Standard_Boolean isOb = UVRep(OcE,OcE2d);
       if (!isOb) return Standard_False; // NYIRAISE
       
-#ifdef DEB
-      const TopoDS_Vertex& vce1 =
-#endif
-                         TopoDS::Vertex(vcE(1)); 
       Standard_Real parvce1 = TopOpeBRepTool_TOOL::ParE(1,cE);   gp_Pnt2d UVvce1 = TopOpeBRepTool_TOOL::UVF(parvce1,cE2d);
-#ifdef DEB
-      const TopoDS_Vertex& vOce2 =
-#endif
-                         TopoDS::Vertex(vOcE(2)); 
+
       Standard_Real parvOcE2 = TopOpeBRepTool_TOOL::ParE(2,OcE); gp_Pnt2d UVvOcE2 = TopOpeBRepTool_TOOL::UVF(parvOcE2,OcE2d);
       Standard_Real tol = Max(tttuvcE,tttuvOcE);
       isoncE = (UVvce1.Distance(UVvOcE2) < tol);
       if (isoncE && (nfy != 1)) {// cto009L2
-       return Standard_False; 
+           return Standard_False; 
       }
     }
 
@@ -531,14 +518,9 @@ Standard_Boolean TopOpeBRepTool_CORRISO::PurgeFyClosingE(const TopTools_ListOfSh
       Standard_Real tttuvOcE = Max(Tol(1,tttolOcE),Tol(2,tttolOcE));
       TopOpeBRepTool_C2DF OcE2d; Standard_Boolean isOb = UVRep(OcE,OcE2d);
       if (!isOb) return Standard_False; // NYIRAISE
-#ifdef DEB 
-      const TopoDS_Vertex& vce1 = TopoDS::Vertex(vcE(1));
-#endif 
+
       Standard_Real parvce1 = TopOpeBRepTool_TOOL::ParE(1,cE);   gp_Pnt2d UVvce1 = TopOpeBRepTool_TOOL::UVF(parvce1,cE2d);
-#ifdef DEB
-      const TopoDS_Vertex& vOce2 =
-#endif
-                         TopoDS::Vertex(vOcE(2)); 
+
       Standard_Real parvOcE2 = TopOpeBRepTool_TOOL::ParE(2,OcE); gp_Pnt2d UVvOcE2 = TopOpeBRepTool_TOOL::UVF(parvOcE2,OcE2d);
       Standard_Real tol = Max(tttuvcE,tttuvOcE);
       isonOcE2d = (UVvce1.Distance(UVvOcE2) < tol);
@@ -840,17 +822,14 @@ Standard_Boolean TopOpeBRepTool_CORRISO::EdgeWithFaultyUV(const TopoDS_Edge& E,
     for (TopTools_ListIteratorOfListOfShape ite(loe); ite.More(); ite.Next()) {
       const TopoDS_Edge& e = TopoDS::Edge(ite.Value());
       TopAbs_Orientation oe = e.Orientation();
+
 #ifdef DEB
-      Standard_Integer ie = STATIC_PURGE_mapeds.Add(e);
-      if (trc) {cout<<"    : on e"<<ie<<endl;}
+    Standard_Integer ie = STATIC_PURGE_mapeds.Add(e);
+    if (trc) {cout<<"    : on e"<<ie<<endl;}
 #endif
+
       if (e.IsSame(E)) continue;      
       if (M_INTERNAL(oe) || M_EXTERNAL(oe)) continue;
-
-#ifdef DEB
-      Standard_Real tttole =
-#endif
-                   BRep_Tool::Tolerance(e);
       
       Standard_Boolean isb = myERep2d.IsBound(e);
       if (!isb) {FUN_RaiseError(); return Standard_False;}
index e8acdc8..2f0bc35 100755 (executable)
@@ -1047,14 +1047,7 @@ Standard_EXPORT Standard_Boolean FUN_tool_pcurveonF(const TopoDS_Face& fF,TopoDS
       
       Standard_Real tole = BRep_Tool::Tolerance(e);
       TopoDS_Vertex vf,vl; TopExp::Vertices(e,vf,vl);
-#ifdef DEB
-      Standard_Real parf  =
-#endif
-                  BRep_Tool::Parameter(vf,e);
-#ifdef DEB
-      Standard_Real parl  =
-#endif
-                  BRep_Tool::Parameter(vl,e);
+
       TopoDS_Edge newe = faultyE;
 //      TopoDS_Edge newe; FUN_ds_CopyEdge(e,newe); newe.Orientation(TopAbs_FORWARD);
 //      vf.Orientation(TopAbs_FORWARD);  BB.Add(newe,vf); FUN_ds_Parameter(newe,vf,parf); 
index 8fe70b6..dbbbb01 100755 (executable)
@@ -111,10 +111,6 @@ Standard_EXPORT void FTOL_FaceTolerances
   Standard_Real tolef= tolef1;
   if (tolef2>tolef) tolef=tolef2;
  //jmb le 30 juillet 99. on ne multiplie pas la tolerance par la dimension de la piece
- // Standard_Real tolbox = tolef*dx;
-  Standard_Real tolbox = tolef;
-
-  Standard_Real tolmin = Precision::Confusion();
 
   Deflection=0.01;
   MaxUV=0.01;
index 8f2848d..4bd8b49 100755 (executable)
@@ -1516,7 +1516,6 @@ void V3d_View::SetZSize(const Standard_Real Size)
 
   Standard_Real Front = MyViewContext.ZClippingFrontPlane() ;
   Standard_Real Back  = MyViewContext.ZClippingBackPlane() ;
-  Standard_Real focale= Focale();
 
   MyViewMapping.SetFrontPlaneDistance(Zmax) ;
   MyViewMapping.SetBackPlaneDistance(-Zmax) ;
index 11b63f2..f48bd7c 100755 (executable)
@@ -72,10 +72,7 @@ void V3d_View::Move(const Standard_Real Dx, const Standard_Real Dy, const Standa
   Zeye = Zrp*Zpn + Dx*XZ + Dy*YZ + Dz*ZZ ;
   Zrp = sqrt( Xeye*Xeye + Yeye*Yeye + Zeye*Zeye ) ;
   V3d_BadValue_Raise_if( Zrp <= 0. ,"V3d_View::Move:: Eye,At are Confused");
-#ifdef DEB
-  Standard_Real focale = 
-#endif
-    Focale();
+
   Prp.SetCoord(Xrp,Yrp,Zrp) ;
   MyViewMapping.SetProjectionReferencePoint(Prp) ;
   Xpn = Xeye / Zrp ; Ypn = Yeye / Zrp ; Zpn = Zeye / Zrp ;
@@ -84,24 +81,8 @@ void V3d_View::Move(const Standard_Real Dx, const Standard_Real Dy, const Standa
   MyView->SetViewOrientation(MyViewOrientation) ; 
   
   // Check ZClipping planes
-#ifdef IMP020300
   MyView->SetViewMapping(MyViewMapping) ; 
   SetZSize(0.);
-#else
-  Standard_Real Zmax,Xat,Yat,Zat ;
-  MyViewReferencePoint.Coord(Xat,Yat,Zat) ;
-  Xeye += Xat ; Yeye += Yat ; Zeye += Zat ;
-  Zmax = sqrt( Xeye*Xeye + Yeye*Yeye + Zeye*Zeye ) ;
-  if( Zmax > MyViewMapping.FrontPlaneDistance() &&
-       MyProjModel == V3d_TPM_SCREEN ) {
-    SetZSize(2.*Zmax+Zmax*Zmargin) ;
-  } else {
-    if( MyType == V3d_PERSPECTIVE ) {
-      SetFocale(focale) ;
-    }
-    MyView->SetViewMapping(MyViewMapping) ; 
-  }
-#endif
   ImmediateUpdate();
 }
 
@@ -124,10 +105,6 @@ void V3d_View::Move(const Standard_Real Length, const Standard_Boolean Start) {
   Zrp = sqrt( Xeye*Xeye + Yeye*Yeye + Zeye*Zeye ) ;
   V3d_BadValue_Raise_if( Zrp <= 0. ,"V3d_View::Move:: Eye,At are Confused");
   
-#ifdef DEB
-  Standard_Real focale = 
-#endif
-    Focale();
   Prp.SetCoord(Xrp,Yrp,Zrp) ;
   MyViewMapping.SetProjectionReferencePoint(Prp) ;
   Xpn = Xeye / Zrp ; Ypn = Yeye / Zrp ; Zpn = Zeye / Zrp ;
index 567a9d8..0084718 100644 (file)
@@ -170,7 +170,6 @@ void VUserDrawObj::Render(const Handle(OpenGl_Workspace)& theWorkspace) const
 {
   // To test linking against OpenGl_Workspace and all aspect classes
   const OpenGl_AspectLine* aLA = theWorkspace->AspectLine(0);
-  const OpenGl_AspectFace* aFA = theWorkspace->AspectFace(0);
   const OpenGl_AspectMarker* aMA = theWorkspace->AspectMarker(0);
   aMA->Type();
   const OpenGl_AspectText* aTA = theWorkspace->AspectText(0);
index 19b7de9..271371e 100755 (executable)
@@ -1806,7 +1806,6 @@ static LRESULT WINAPI AdvViewerWindowProc( HWND hwnd,
       if( !DragFirst )
       {
         HDC hdc = GetDC( hwnd );
-        HGDIOBJ anObj = SelectObject( hdc, GetStockObject( WHITE_PEN ) );
         SelectObject( hdc, GetStockObject( HOLLOW_BRUSH ) );
         SetROP2( hdc, R2_NOT );
         Rectangle( hdc, X_ButtonPress, Y_ButtonPress, X_Motion, Y_Motion );
index 32c9eba..7c10282 100755 (executable)
@@ -208,7 +208,7 @@ Standard_Boolean Voxel_FastConverter::Convert(Standard_Integer&      progress,
 
   // Convert
   TopLoc_Location L;
-  Standard_Integer iprogress = 0, prev_progress = 0;
+  Standard_Integer iprogress = 0;
   Standard_Integer n1, n2, n3;
   Standard_Integer ixmin, iymin, izmin, ixmax, iymax, izmax;
   Standard_Real xmin, ymin, zmin, xmax, ymax, zmax;
@@ -346,7 +346,7 @@ Standard_Boolean Voxel_FastConverter::ConvertUsingSAT(Standard_Integer&      pro
 
   // Convert
   TopLoc_Location L;
-  Standard_Integer iprogress = 0, prev_progress = 0;
+  Standard_Integer iprogress = 0;
   Standard_Integer n1, n2, n3;
   Standard_Integer ixmin, iymin, izmin, ixmax, iymax, izmax;
   Standard_Real xmin, ymin, zmin, xmax, ymax, zmax;
index 421f469..fb83e14 100755 (executable)
@@ -568,7 +568,7 @@ VrmlData_ErrorStatus VrmlData_TextureCoordinate::Read
           aStatus = VrmlData_VrmlFormatError;
         else {
           theBuffer.LinePtr++;
-          while (~0) {
+          for(;;) {
             gp_XY anXY;
             if (!OK(aStatus, VrmlData_Scene::ReadLine(theBuffer)))
               break;
@@ -657,7 +657,7 @@ VrmlData_ErrorStatus VrmlData_ArrayVec3d::ReadArray
         aStatus = VrmlData_VrmlFormatError;
       else {
         theBuffer.LinePtr++;
-        while (~0) {
+        for(;;) {
           gp_XYZ anXYZ;
           if (!OK(aStatus, VrmlData_Scene::ReadLine(theBuffer)))
             break;
index b82cf5f..ed91a58 100755 (executable)
@@ -305,7 +305,7 @@ VrmlData_Scene& VrmlData_Scene::operator << (Standard_IStream& theInput)
 //   if (myStatus == StatusOK)
 //     myStatus = ReadLine (aBuffer);
   // Read VRML data by nodes
-  while (~0) {
+  for(;;) {
     if (!VrmlData_Node::OK(myStatus, ReadLine(aBuffer))) {
       if (myStatus == VrmlData_EndOfFile)
         myStatus = VrmlData_StatusOK;
index 68f02b2..22ffec8 100755 (executable)
@@ -235,7 +235,7 @@ void VrmlData_ShapeConvert::Convert (const Standard_Boolean theExtractFaces,
                 TopLoc_Location aL;
 
                 Standard_Boolean found = Standard_False;
-                for(i = 1; Standard_True; i++) {
+                for(i = 1; ; i++) {
                   
                   BRep_Tool::PolygonOnTriangulation(aEdge, aPT, aT, aL, i);
 
index f46a4fe..28bca44 100755 (executable)
@@ -1020,7 +1020,7 @@ BOOL EHDC :: _DrawToEx ( PPOINT p ) {
  myIDDA = min( k, myIDDA );
  HPEN hpo = SelectPen(  myHDC, GetStockObject ( NULL_PEN )  );
 
- while ( TRUE ) {
+ for(;;) {
  
   if ( myPRlen == 0 ) {
  
@@ -1090,7 +1090,7 @@ void EHDC :: _LineTo ( PPOINT p ) {
  ddap.pp = myDDA1;
  LineDDA (  myStart.x, myStart.y, p -> x, p -> y, &___auxDDAF, ( LPARAM )&ddap  );
 
- while ( TRUE ) {
+ for(;;) {
 
   if ( myPRlen == 0 ) {
  
index eb00953..11cab24 100755 (executable)
@@ -287,45 +287,6 @@ Standard_Boolean WNT_Window :: DoMapping () const {
   return Standard_True;
 }
 
-static Standard_Boolean ConvertBitmap (HBITMAP       theHBitmap,
-                                       Image_PixMap& thePixMap)
-{
-  // Get informations about the bitmap
-  BITMAP aBitmap;
-  if (GetObject (theHBitmap, sizeof(BITMAP), (LPSTR )&aBitmap) == 0)
-  {
-    return Standard_False;
-  }
-
-  const Standard_Size aSizeRowBytes = Standard_Size(aBitmap.bmWidth) * 4;
-  if (!thePixMap.InitTrash (Image_PixMap::ImgBGR32, Standard_Size(aBitmap.bmWidth), Standard_Size(aBitmap.bmHeight), aSizeRowBytes))
-  {
-    return Standard_False;
-  }
-  thePixMap.SetTopDown (false);
-
-  // Setup image data
-  BITMAPINFOHEADER aBitmapInfo;
-  memset (&aBitmapInfo, 0, sizeof(BITMAPINFOHEADER));
-  aBitmapInfo.biSize        = sizeof(BITMAPINFOHEADER);
-  aBitmapInfo.biWidth       = aBitmap.bmWidth;
-  aBitmapInfo.biHeight      = aBitmap.bmHeight; // positive means bottom-up!
-  aBitmapInfo.biPlanes      = 1;
-  aBitmapInfo.biBitCount    = 32; // use 32bit for automatic word-alignment per row
-  aBitmapInfo.biCompression = BI_RGB;
-
-  // Copy the pixels
-  HDC aDC = GetDC (NULL);
-  Standard_Boolean isSuccess = GetDIBits (aDC, theHBitmap,
-                                          0,                           // first scan line to set
-                                          aBitmap.bmHeight,            // number of scan lines to copy
-                                          thePixMap.ChangeData(),      // array for bitmap bits
-                                          (LPBITMAPINFO )&aBitmapInfo, // bitmap data info
-                                          DIB_RGB_COLORS) != 0;
-  ReleaseDC (NULL, aDC);
-  return isSuccess;
-}
-
 //***//
 //******************************* Ratio **********************************//
 //***//
index ab02f3d..7f7d48d 100755 (executable)
@@ -142,10 +142,7 @@ void XSDRAW::LoadDraw (Draw_Interpretor& theCommands)
 {
   char mess[100];
   Sprintf (mess,command,varname);
-#ifdef DEB
-  IFSelect_ReturnStatus stat = 
-#endif
-    thepilot->Execute (mess);
+  thepilot->Execute (mess);
   return 1;  // stat ?
 }
 
index 7a828e7..8a2cdcb 100755 (executable)
@@ -571,7 +571,7 @@ static Standard_Integer stepfileunits (Draw_Interpretor& di, Standard_Integer ar
   TColStd_SequenceOfAsciiString anUnitSolidAngleNames;
   aStepReader.FileUnits( anUnitLengthNames,anUnitAngleNames,anUnitSolidAngleNames);
                                    
-  Standard_Integer i =1, nb = anUnitLengthNames.Length();
+  Standard_Integer i =1;
   di<<"=====================================================\n";
   di<<"LENTH Unit"<<"\n";
   for( ; i <= anUnitLengthNames.Length() ; i++)
index 4222d88..9b97be4 100755 (executable)
@@ -172,10 +172,9 @@ void XmlMDataStd_IntegerArrayDriver::Paste
     str.Allocate(12 * intArray.Length() + 1);
 
   Standard_Integer i = aL;
-  while (1) 
+  for (;;) 
   {
-    const Standard_Integer& intValue = intArray.Value(i);
-    iChar += Sprintf(&(str[iChar]), "%d ", intValue);
+    iChar += Sprintf(&(str[iChar]), "%d ", intArray.Value(i));
     if (i >= anU)
       break;
     ++i;
index eb4cfef..e4c5d10 100755 (executable)
@@ -733,7 +733,7 @@ void XmlMDataStd_NamedDataDriver::Paste(const Handle(TDF_Attribute)& theSource,
       //Value = Array
       TCollection_AsciiString aValueStr2;
       Standard_Integer j = anArr1.Lower();
-       while (1) {
+       for(;;) {
         aValueStr2 += TCollection_AsciiString(anArr1.Value(j));
         if (j >= anArr1.Upper()) break;
         aValueStr2 += ' ';
@@ -762,7 +762,7 @@ void XmlMDataStd_NamedDataDriver::Paste(const Handle(TDF_Attribute)& theSource,
       //Value = Array
       TCollection_AsciiString aValueStr2;
       Standard_Integer j = anArr1.Lower();
-      while (1) {
+      for(;;) {
 #ifndef OCC6010
        aValueStr2 += TCollection_AsciiString(anArr1.Value(j));
 #else
index 926b161..46eacc9 100755 (executable)
@@ -174,10 +174,9 @@ void XmlMDataStd_RealArrayDriver::Paste (const Handle(TDF_Attribute)& theSource,
     str.Allocate(25 * realArray.Length() + 1);
 
   Standard_Integer i = aL;
-  while (1) 
+  for (;;) 
   {
-    const Standard_Real& dblValue = realArray.Value(i);
-    iChar += Sprintf(&(str[iChar]), "%.17g ", dblValue);
+    iChar += Sprintf(&(str[iChar]), "%.17g ", realArray.Value(i));
     if (i >= anU)
       break;
     ++i;
index 86c8ed2..802c8b0 100755 (executable)
@@ -239,7 +239,7 @@ void XmlObjMgt::SetTagEntryString (XmlObjMgt_DOMString&           theTarget,
   memcpy (aTarget, aRefPrefix, sizeof (aRefPrefix) - 1);
   char * aTargetPtr = aTarget + (sizeof (aRefPrefix) - 1);
 
-  while (1) {
+  for(;;) {
     //  Check for the end-of-string; find the delimeter ':'
     aPtr = strchr (aTagEntry, ':');
     if (aPtr == NULL) break;
index 4d770f2..86b5467 100755 (executable)
@@ -81,7 +81,7 @@ void gp_GTrsf::Power (const Standard_Integer N)
       gp_XYZ Temploc = loc;
 //      Standard_Real Tempscale = scale;
       gp_Mat Tempmatrix (matrix);
-      while (1) {
+      for(;;) {
        if (IsOdd(Npower)) {
          loc.Add (Temploc.Multiplied (matrix));
          matrix.Multiply (Tempmatrix);
index f206b07..019af7e 100755 (executable)
@@ -98,7 +98,7 @@ void gp_GTrsf2d::Power (const Standard_Integer N)
       gp_XY Temploc = loc;
 //      Standard_Real Tempscale = scale;
       gp_Mat2d Tempmatrix (matrix);
-      while (1) {
+      for(;;) {
        if (IsOdd(Npower)) {
          loc.Add (Temploc.Multiplied (matrix));
          matrix.Multiply (Tempmatrix);
index 7809aa2..d0af6c4 100755 (executable)
@@ -259,7 +259,7 @@ void gp_Mat::Power (const Standard_Integer N)
     if (Npower < 0) Npower = - Npower;
     Npower--;
     gp_Mat Temp = *this;
-    while (1) {
+    for(;;) {
       if (IsOdd(Npower)) Multiply (Temp);
       if (Npower == 1)   break; 
       Temp.Multiply (Temp);
index 63b9e10..8417079 100755 (executable)
@@ -141,7 +141,7 @@ void gp_Mat2d::Power (const Standard_Integer N)
     if (Npower < 0) Npower = - Npower;
     Npower--;
     gp_Mat2d Temp = *this;
-    while (1) {
+    for(;;) {
       if (IsOdd(Npower)) Multiply (Temp);
       if (Npower == 1)   break;
       Temp.Multiply (Temp);
index 32a3d7b..f0fbccd 100755 (executable)
@@ -610,7 +610,7 @@ void gp_Trsf::Power (const Standard_Integer N)
         if (Npower < 0) Npower = - Npower;
         Npower--;
         gp_XYZ Temploc = loc;
-        while (1) {
+        for(;;) {
           if (IsOdd(Npower))  loc.Add (Temploc);
           if (Npower == 1) break;
           Temploc.Add (Temploc);
@@ -623,7 +623,7 @@ void gp_Trsf::Power (const Standard_Integer N)
         Npower--;
         gp_XYZ Temploc = loc;
         Standard_Real Tempscale = scale;
-        while (1) {
+        for(;;) {
           if (IsOdd(Npower)) {
             loc.Add (Temploc.Multiplied(scale));
             scale = scale * Tempscale;
@@ -640,7 +640,7 @@ void gp_Trsf::Power (const Standard_Integer N)
         Npower--;
         gp_Mat Tempmatrix (matrix);
         if (loc.X() == 0.0 && loc.Y() == 0.0 && loc.Z() == 0.0) {
-          while (1) {
+          for(;;) {
             if (IsOdd(Npower)) matrix.Multiply (Tempmatrix);
             if (Npower == 1)   break;
             Tempmatrix.Multiply (Tempmatrix);
@@ -649,7 +649,7 @@ void gp_Trsf::Power (const Standard_Integer N)
         }
         else {
           gp_XYZ Temploc = loc;
-          while (1) {
+          for(;;) {
             if (IsOdd(Npower)) {
               loc.Add (Temploc.Multiplied (matrix));
               matrix.Multiply (Tempmatrix);
@@ -680,7 +680,7 @@ void gp_Trsf::Power (const Standard_Integer N)
         gp_XYZ Temploc = loc;
         Standard_Real Tempscale = scale;
         gp_Mat Tempmatrix (matrix);
-        while (1) {
+        for(;;) {
           if (IsOdd(Npower)) {
             loc.Add ((Temploc.Multiplied (matrix)).Multiplied (scale));
             scale = scale * Tempscale;
index 16770f3..74eaeb9 100755 (executable)
@@ -310,7 +310,7 @@ void gp_Trsf2d::Power (const Standard_Integer N)
        if (Npower < 0) Npower = - Npower;
        Npower--;
        gp_XY Temploc = loc;
-       while (1) {
+       for(;;) {
          if (IsOdd(Npower))  loc.Add (Temploc);
          if (Npower == 1) break;
          Temploc.Add (Temploc);
@@ -323,7 +323,7 @@ void gp_Trsf2d::Power (const Standard_Integer N)
        Npower--;
        gp_XY Temploc = loc;
        Standard_Real Tempscale = scale;
-       while (1) {
+       for(;;) {
          if (IsOdd(Npower)) {
            loc.Add (Temploc.Multiplied(scale));
            scale = scale * Tempscale;
@@ -340,7 +340,7 @@ void gp_Trsf2d::Power (const Standard_Integer N)
        Npower--;
        gp_Mat2d Tempmatrix (matrix);
        if (loc.X() == 0.0 && loc.Y() == 0.0) {
-         while (1) {
+         for(;;) {
            if (IsOdd(Npower))  matrix.Multiply (Tempmatrix);
            if (Npower == 1) break;
            Tempmatrix.Multiply (Tempmatrix);
@@ -349,7 +349,7 @@ void gp_Trsf2d::Power (const Standard_Integer N)
        }
        else {
          gp_XY Temploc = loc;
-         while (1) {
+         for(;;) {
            if (IsOdd(Npower)) {
              loc.Add (Temploc.Multiplied (matrix));
              matrix.Multiply (Tempmatrix);
@@ -379,7 +379,7 @@ void gp_Trsf2d::Power (const Standard_Integer N)
        gp_XY Temploc = loc;
        Standard_Real Tempscale = scale;
        gp_Mat2d Tempmatrix (matrix);
-       while (1) {
+       for(;;) {
          if (IsOdd(Npower)) {
            loc.Add ((Temploc.Multiplied (matrix)).Multiplied (scale));
            scale = scale * Tempscale;
index b105f30..84216ad 100755 (executable)
@@ -62,7 +62,7 @@ const Standard_Real ZERO = 1.0e-30;
 
     static Standard_Real Improve(const Standard_Integer N, Standard_Real *Poly, const Standard_Real IniSol) {
 
-        Standard_Real Val, Der, Delta;
+        Standard_Real Val = 0., Der, Delta;
         Standard_Real Sol = IniSol;
         Standard_Real IniVal = Value(N, Poly, IniSol);
         Standard_Integer Index;
index f6008ef..9dad5fa 100755 (executable)
@@ -61,50 +61,50 @@ proc checkpoint {coord_x coord_y rd_ch gr_ch bl_ch} {
     set mistake 0
     set i 0
     while { $mistake != 1 && $i <= 5 } {
-       set j 0
-       while { $mistake != 1 && $j <= 5 } {
-           set position_x [expr ${x_start} + $j]
-           set position_y [expr ${y_start} + $i]
-           puts $position_x
-           puts $position_y
-           global color2d
-           if { [info exists color2d] } {
-               set color [ QAAISGetPixelColor2d ${position_x} ${position_y} ]
-           } else {
-               set color [ QAGetPixelColor ${position_x} ${position_y} ]
-           }
-           regexp {RED +: +([-0-9.+eE]+)} $color full rd
-           regexp {GREEN +: +([-0-9.+eE]+)} $color full gr
-           regexp {BLUE +: +([-0-9.+eE]+)} $color full bl
-           set rd_int [expr int($rd * 1.e+05)]
-           set gr_int [expr int($gr * 1.e+05)]
-           set bl_int [expr int($bl * 1.e+05)]
-           
-           if { $rd_ch != 0 } {
-               set tol_rd [expr abs($rd_ch - $rd_int)/$rd_ch]
-           } else {
-               set tol_rd $rd_int
-           }
-           if { $gr_ch != 0 } {
-               set tol_gr [expr abs($gr_ch - $gr_int)/$gr_ch]
-           } else {
-               set tol_gr $gr_int
-           }
-           if { $bl_ch != 0 } {
-               set tol_bl [expr abs($bl_ch - $bl_int)/$bl_ch]
-           } else {
-               set tol_bl $bl_int
-           }
-
-           if { $tol_rd > 0.2 || $tol_gr > 0.2 || $tol_bl > 0.2 } {
-               puts "Warning : Point with true color was not found near default coordinates"
-               set mistake 0
-           } else {
-               set mistake 1
-           }
-           incr j
-       }
-       incr i
+      set j 0
+      while { $mistake != 1 && $j <= 5 } {
+          set position_x [expr ${x_start} + $j]
+          set position_y [expr ${y_start} + $i]
+          puts $position_x
+          puts $position_y
+          global color2d
+          if { [info exists color2d] } {
+            set color [ QAAISGetPixelColor2d ${position_x} ${position_y} ]
+          } else {
+            set color [ QAGetPixelColor ${position_x} ${position_y} ]
+          }
+          regexp {RED +: +([-0-9.+eE]+)} $color full rd
+          regexp {GREEN +: +([-0-9.+eE]+)} $color full gr
+          regexp {BLUE +: +([-0-9.+eE]+)} $color full bl
+          set rd_int [expr int($rd * 1.e+05)]
+          set gr_int [expr int($gr * 1.e+05)]
+          set bl_int [expr int($bl * 1.e+05)]
+            
+          if { $rd_ch != 0 } {
+            set tol_rd [expr abs($rd_ch - $rd_int)/$rd_ch]
+          } else {
+            set tol_rd $rd_int
+          }
+          if { $gr_ch != 0 } {
+            set tol_gr [expr abs($gr_ch - $gr_int)/$gr_ch]
+          } else {
+            set tol_gr $gr_int
+          }
+          if { $bl_ch != 0 } {
+            set tol_bl [expr abs($bl_ch - $bl_int)/$bl_ch]
+          } else {
+            set tol_bl $bl_int
+          }
+
+          if { $tol_rd > 0.2 || $tol_gr > 0.2 || $tol_bl > 0.2 } {
+            puts "Warning : Point with true color was not found near default coordinates"
+            set mistake 0
+          } else {
+            set mistake 1
+          }
+          incr j
+      }
+      incr i
     }
     return $mistake
 }
@@ -118,15 +118,15 @@ proc checkcolor { coord_x coord_y rd_get gr_get bl_get } {
     puts "BLUE color of RGB is $bl_get"
     
     if { $coord_x <= 1 || $coord_y <= 1 } {
-       puts "Error : minimal coordinate is x = 2, y = 2. But we have x = $coord_x y = $coord_y"
-       return -1
+      puts "Error : minimal coordinate is x = 2, y = 2. But we have x = $coord_x y = $coord_y"
+      return -1
     }
     global color2d
-       if { [info exists color2d] } {
-           set color [ QAAISGetPixelColor2d ${coord_x} ${coord_y} ]
-       } else {
-           set color [ QAGetPixelColor ${coord_x} ${coord_y} ]
-       }
+    if { [info exists color2d] } {
+      set color [ QAAISGetPixelColor2d ${coord_x} ${coord_y} ]
+    } else {
+      set color [ QAGetPixelColor ${coord_x} ${coord_y} ]
+    }
 
     regexp {RED +: +([-0-9.+eE]+)} $color full rd
     regexp {GREEN +: +([-0-9.+eE]+)} $color full gr
@@ -139,51 +139,51 @@ proc checkcolor { coord_x coord_y rd_get gr_get bl_get } {
     set bl_ch [expr int($bl_get * 1.e+05)]
     
     if { $rd_ch != 0 } {
-       set tol_rd [expr abs($rd_ch - $rd_int)/$rd_ch]
+      set tol_rd [expr abs($rd_ch - $rd_int)/$rd_ch]
     } else {
-       set tol_rd $rd_int
+      set tol_rd $rd_int
     }
     if { $gr_ch != 0 } {
-       set tol_gr [expr abs($gr_ch - $gr_int)/$gr_ch]
+      set tol_gr [expr abs($gr_ch - $gr_int)/$gr_ch]
     } else {
-       set tol_gr $gr_int
+      set tol_gr $gr_int
     }
     if { $bl_ch != 0 } {
-       set tol_bl [expr abs($bl_ch - $bl_int)/$bl_ch]
+      set tol_bl [expr abs($bl_ch - $bl_int)/$bl_ch]
     } else {
-       set tol_bl $bl_int
+      set tol_bl $bl_int
     }
     set status 0
     if { $tol_rd > 0.2 } {
-       puts "Warning : RED light of additive color model RGB is invalid"
-       set status 1
+      puts "Warning : RED light of additive color model RGB is invalid"
+      set status 1
     }
     if { $tol_gr > 0.2 } {
-       puts "Warning : GREEN light of additive color model RGB is invalid"
-       set status 1
+      puts "Warning : GREEN light of additive color model RGB is invalid"
+      set status 1
     }
     if { $tol_bl > 0.2 } {
-       puts "Warning : BLUE light of additive color model RGB is invalid"
-       set status 1
+      puts "Warning : BLUE light of additive color model RGB is invalid"
+      set status 1
     }
 
     if { $status != 0 } {
-       puts "Warning : Colors of default coordinate are not equal"
+      puts "Warning : Colors of default coordinate are not equal"
     }
 
     global stat
     if { $tol_rd > 0.2 || $tol_gr > 0.2 || $tol_bl > 0.2 } {
-       set info [checkpoint $coord_x $coord_y $rd_ch $gr_ch $bl_ch]
-       set stat [lindex $info end]
-       if { ${stat} != 1 } {
-           puts "Error : Colors are not equal in default coordinate and in the near coordinates too"
-           return $stat
-       } else {
-           puts "Point with valid color was found"
-           return $stat
-       }
+      set info [checkpoint $coord_x $coord_y $rd_ch $gr_ch $bl_ch]
+      set stat [lindex $info end]
+      if { ${stat} != 1 } {
+          puts "Error : Colors are not equal in default coordinate and in the near coordinates too"
+          return $stat
+      } else {
+          puts "Point with valid color was found"
+          return $stat
+      }
     } else {
-       set stat 1
+      set stat 1
     }
 }
 
index 8e8bf0c..595dcc8 100755 (executable)
@@ -2,9 +2,9 @@ if { [info exists square] } {
     set prop "square"
     set mass $square
     if { [info exists tol_square] } {
-       regexp {Mass +: +([-0-9.+eE]+)} [sprops result $tol_square] full m
+      regexp {Mass +: +([-0-9.+eE]+)} [sprops result $tol_square] full m
     } else {
-       regexp {Mass +: +([-0-9.+eE]+)} [sprops result] full m
+      regexp {Mass +: +([-0-9.+eE]+)} [sprops result] full m
     }
 }
 if { [info exists length] } {
@@ -19,34 +19,34 @@ if { [info exists length] } {
 #if mass (length or square) is empty in test case then result should be an empty shape.
 if { [info exists mass] } {
     if { [string compare $subgroup "moddata"] != 0 } {
-       puts "checkshape"
-       checkshape result
+      puts "checkshape"
+      checkshape result
     }
       
     if { [string compare "$mass" "empty"] != 0 } {
-       if { $m == 0 } {
-                   puts "Error : The command is not valid. The $prop is 0."
-       }
-       if { $mass > 0 } {
-                   puts "The expected $prop is $mass"
-       }
-       #check of change of square is < 1%
-       if { ($mass != 0 && [expr 1.*abs($mass - $m)/$mass] > 0.01) || ($mass == 0 && $m != 0) } {
-           puts "Error : The $prop of result shape is $m"
-       }
+      if { $m == 0 } {
+        puts "Error : The command is not valid. The $prop is 0."
+      }
+      if { $mass > 0 } {
+        puts "The expected $prop is $mass"
+      }
+      #check of change of square is < 1%
+      if { ($mass != 0 && [expr 1.*abs($mass - $m)/$mass] > 0.01) || ($mass == 0 && $m != 0) } {
+        puts "Error : The $prop of result shape is $m"
+      }
     } else {
-       if { $m != 0 } {
-                   puts "Error : The command is not valid. The $prop is $m"
-       }
+      if { $m != 0 } {
+        puts "Error : The command is not valid. The $prop is $m"
+      }
     }
 }
 
 set glob_inf [info global nb_*_good]
 if { [regexp "nb_.*_good" $glob_inf] == 1 } {
     if { [info exists nbsh_t] } {
-       set nb_info [nbshapes result -t]
+      set nb_info [nbshapes result -t]
     } else {
-        set nb_info [nbshapes result]
+      set nb_info [nbshapes result]
     }
 }
 
@@ -146,18 +146,18 @@ if { [info exists rel_tol] } {
       puts "Error   : area by triangles differs from the actual area by $rel_err %"
    } else {
       if { $rel_tol > 1 && $rel_tol < 100 } {
-        puts "Error: Improvement: The current area difference is $rel_err instead of $rel_tol"
+        puts "Error: Improvement: The current area difference is $rel_err instead of $rel_tol"
       }
    }
 }
 
 if { [isdraw result] } {
   if { [info exists 2dviewer] } {
-       clear
-       smallview
-       donly result
-       fit
-       xwd $imagedir/${test_image}.png
+    clear
+    smallview
+    donly result
+    fit
+    xwd $imagedir/${test_image}.png
   }
   if { [info exists 3dviewer] } {
     #for multiview support: dump result from all opened views
index a58e92f..0795dac 100755 (executable)
@@ -1,7 +1,4 @@
-puts "TODO OCC12345 ALL: Faulty shapes in variables faulty_1 to faulty_"
-puts "TODO OCC12345 ALL: Faulty : mistakes are found in checked shape by checkshape command"
-puts "TODO OCC12345 ALL: Error : The square of result shape is"
-
+puts "TODO OCC12345 windows: Faulty shapes in variables faulty_1 to faulty_"
 
 puts "=========="
 puts "BUC60462"
@@ -15,12 +12,7 @@ puts "Fuse begin"
 bfuse result a b
 puts "Fuse end"
 
-set che [checkshape result r]
-if { [regexp {OK} $che] != 1 } {
-    puts "Faulty : mistakes are found in checked shape by checkshape command"
-} else {
-    puts "Checking by checkshape - OK"
-}
+checkshape result r
 
-set square 0
+set square 89562.1
 set 2dviewer 0