]> OCCT Git - occt-copy.git/commitdiff
0027097: GCC and CLang compiler warnings and errors with -Wpedantic
authorabv <abv@opencascade.com>
Fri, 22 Jan 2016 08:42:32 +0000 (11:42 +0300)
committerabv <abv@opencascade.com>
Fri, 22 Jan 2016 08:42:51 +0000 (11:42 +0300)
Useless semicolons are removed after closing braces of namespaces and macros in code dependent on FreeImage and VTK.
Comments added at the closing braces of namespaces to help comprehension.
In OSD_File.cxx and OSD_Disk.cxx, unnecessary declarations of system functions are removed

27 files changed:
src/Aspect/Aspect_Convert.hxx
src/BRepMesh/BRepMesh.hxx
src/Draw/Draw_Interpretor.cxx
src/Font/Font_FontMgr.cxx
src/IVtkDraw/IVtkDraw_Interactor.cxx
src/IVtkDraw/IVtkDraw_Interactor.hxx
src/IVtkTools/IVtkTools.cxx
src/IVtkTools/IVtkTools.hxx
src/IVtkTools/IVtkTools_DisplayModeFilter.cxx
src/IVtkTools/IVtkTools_DisplayModeFilter.hxx
src/IVtkTools/IVtkTools_ShapeDataSource.cxx
src/IVtkTools/IVtkTools_ShapeDataSource.hxx
src/IVtkTools/IVtkTools_ShapeObject.cxx
src/IVtkTools/IVtkTools_ShapeObject.hxx
src/IVtkTools/IVtkTools_ShapePicker.cxx
src/IVtkTools/IVtkTools_ShapePicker.hxx
src/IVtkTools/IVtkTools_SubPolyDataFilter.cxx
src/IVtkTools/IVtkTools_SubPolyDataFilter.hxx
src/IVtkVTK/IVtkVTK_View.cxx
src/Image/Image_AlienPixMap.cxx
src/Image/Image_Diff.cxx
src/OSD/OSD_Disk.cxx
src/OSD/OSD_SharedLibrary.cxx
src/OpenGl/OpenGl_Text.cxx
src/StdPrs/StdPrs_ShadedShape.cxx
src/TopOpeBRep/TopOpeBRep_FacesFiller_1.cxx
src/ViewerTest/ViewerTest_ViewerCommands.cxx

index 9ba492443da737d51fed26d5160f177eae2536c9..d1c9f4a14bff5dee7723aa4a6a91282b3fccf6ff 100644 (file)
@@ -81,6 +81,6 @@ namespace Aspect_Convert
     }
   }
 
-}
+} // namespace Aspect_Convert
 
 #endif /* _Aspect_Convert_HeaderFile */
index dc9deb4ffa2c351b8d5e54a78e3212202f94b10d..c1022ed5390c9cae1befd38903d1282fe8e5405a 100644 (file)
@@ -132,6 +132,7 @@ namespace BRepMesh
   //! Other data structures
   typedef std::pair<HArray1OfSegments, HBndBox2dTree>                                               SegmentsTree;
   typedef NCollection_Array1<SegmentsTree>                                                          Array1OfSegmentsTree;
-}
+
+} // namespace BRepMesh
 
 #endif
index 776bcf366e1fb29669941586a28c5fb4a193af01..bffe2fbef9f2510a965bb6b863a3dbfe684709e5 100644 (file)
@@ -108,7 +108,7 @@ namespace {
       tmp_file->Remove();
   }
 
-}
+} // anonymous namespace
 
 // MKV 29.03.05
 #if ((TCL_MAJOR_VERSION > 8) || ((TCL_MAJOR_VERSION == 8) && (TCL_MINOR_VERSION >= 4))) && !defined(USE_NON_CONST)
index fc64f4bad1ea2e71434eb0b0aca87817340e83f2..39882ea6db36e0dae9e2b42350743a2c064bb5bb 100644 (file)
@@ -173,7 +173,7 @@ static const Font_FontMgr_FontAliasMapNode Font_FontMgr_MapOfFontsAliases[] =
       }
     }
 
-  }
+  } // anonymous namespace
 
 #endif
 
index 8b8524a1aa0d447e4bab8c7fb1921dcfb2fb8b3d..7fef1f9427ab21b5bef6f9e93140f08657496450 100644 (file)
@@ -76,7 +76,7 @@ static void ClearHighlightAndSelection (const Handle(ShapePipelineMap)& theMap,
   }
 }
 
-vtkStandardNewMacro(IVtkDraw_Interactor);
+vtkStandardNewMacro(IVtkDraw_Interactor)
 
 //===========================================================
 // Function : Constructor
index 38f931a9b13d86fdd4863bd4116dfc798b676131..f0d3da440037d4373ab90cabaf1b8279d39134bb 100644 (file)
@@ -43,7 +43,8 @@ class IVtkDraw_Interactor : public vtkRenderWindowInteractor
 {
 public:
   static IVtkDraw_Interactor *New();
-  vtkTypeMacro (IVtkDraw_Interactor, vtkRenderWindowInteractor);
+
+  vtkTypeMacro (IVtkDraw_Interactor, vtkRenderWindowInteractor)
 
   virtual void Initialize();
   virtual void Enable();
index b16de19203663155344aea40fd5c5a00e68e64e6..5de6bbf887d32995977fd4ad5b993aad3091a8ea 100644 (file)
@@ -121,4 +121,5 @@ void InitShapeMapper (vtkMapper* theMapper, vtkLookupTable* theColorTable)
   theMapper->SetLookupTable (theColorTable);
   theMapper->Update();
 }
-};
+
+} // namespace IVtkTools
index 3a580cfab36997c00b188206e4f661f2ebc7e364..2cd96483667bb5ea30fa3501a5c2809f24aa25e1 100644 (file)
@@ -83,6 +83,6 @@ namespace IVtkTools
   Standard_EXPORT void InitShapeMapper (vtkMapper* theMapper, 
                                         vtkLookupTable* theColorTable);
 
-};
+}
 
 #endif // IVtkTOOLS_H
index 1cf1013fe219eac76bdb65a5cdc1f7a202629f22..10bc6f258afcfec893bcc64443200528d7eebe9a 100644 (file)
@@ -19,7 +19,7 @@
 #include <vtkInformationVector.h>
 #include <vtkObjectFactory.h>
 
-vtkStandardNewMacro(IVtkTools_DisplayModeFilter);
+vtkStandardNewMacro(IVtkTools_DisplayModeFilter)
 
 //============================================================================
 // Method: Constructor
index d5d2bc8166e3719668eb10b29810e145a7cfbce7..b029eddbe6f2d3369328989a829b099f6aeb433d 100644 (file)
@@ -28,7 +28,8 @@
 class Standard_EXPORT IVtkTools_DisplayModeFilter : public IVtkTools_SubPolyDataFilter
 {
 public:
-  vtkTypeMacro(IVtkTools_DisplayModeFilter,IVtkTools_SubPolyDataFilter);
+  vtkTypeMacro(IVtkTools_DisplayModeFilter,IVtkTools_SubPolyDataFilter)
+
   static IVtkTools_DisplayModeFilter *New();
   void PrintSelf (std::ostream& os, vtkIndent indent);
 
index f783887b49d03f422a3142116e9364aa69ce1ed4..eba9873e4eb52e21ee9349504976e16002a3c1ce 100644 (file)
@@ -31,7 +31,7 @@
 #include <vtkTransform.h>
 #include <vtkTransformPolyDataFilter.h>
 
-vtkStandardNewMacro(IVtkTools_ShapeDataSource);
+vtkStandardNewMacro(IVtkTools_ShapeDataSource)
 
 //================================================================
 // Function : Constructor
index 02019276a4b7bfe57e9dd3173d4fb746b8df1d88..81401e3f8d2c8b93984091bffc7b3eb94a197067 100644 (file)
@@ -33,7 +33,8 @@ class IVtkTools_EXPORT IVtkTools_ShapeDataSource : public vtkPolyDataAlgorithm
 {
 public:
 
-  vtkTypeMacro(IVtkTools_ShapeDataSource, vtkPolyDataAlgorithm);
+  vtkTypeMacro(IVtkTools_ShapeDataSource, vtkPolyDataAlgorithm)
+
   static IVtkTools_ShapeDataSource* New();
 
 
index 314e8bec4c93bef30da19186215d0eda4bcf86dc..c45148ee64f666408483956b9c71d49cb4ab8c74 100644 (file)
@@ -121,7 +121,7 @@ void IVtkTools_ShapeObject::SetShapeSource (IVtkTools_ShapeDataSource* theDataSo
 
 //! @class IVtkTools_ShapeObject 
 //! @brief VTK holder class for OCC shapes to pass them through pipelines.
-vtkStandardNewMacro(IVtkTools_ShapeObject);
+vtkStandardNewMacro(IVtkTools_ShapeObject)
 
 //============================================================================
 //  Method: Constructor
index 84bcbe278ce2032893e132acf660de3e6ac1f6de..db46437ca6b2fb8240df5fb10fddab7b4efcc5eb 100644 (file)
@@ -37,7 +37,7 @@ class IVtkTools_ShapeDataSource;
 class IVtkTools_EXPORT IVtkTools_ShapeObject :  public vtkDataObject
 {
 public:
-  vtkTypeMacro (IVtkTools_ShapeObject, vtkObject);
+  vtkTypeMacro (IVtkTools_ShapeObject, vtkObject)
 
   static IVtkTools_ShapeObject* New(); 
 
index 2a0dd63558f9ae7b26e862d40ba40bf8f891bae2..6091f631823a4dcfc5c5b97ff6c6f8e562e72d95 100644 (file)
@@ -30,7 +30,7 @@
 //! before the user starts to select interactively, in order for the OCCT selection
 //! algorithm to prepare its internal selection data.
 
-vtkStandardNewMacro(IVtkTools_ShapePicker);
+vtkStandardNewMacro(IVtkTools_ShapePicker)
 
 //============================================================================
 //  Method: IVtkTools_ShapePicker
index 9c181a5c47f44302f4069f6e631457dae00923a7..67a69cd2c2cdf0faf001ec04e5ba8f1edd24fee9 100644 (file)
@@ -29,7 +29,8 @@ class vtkActorCollection;
 class IVtkTools_EXPORT IVtkTools_ShapePicker :  public vtkAbstractPropPicker
 {
 public:
-  vtkTypeMacro(IVtkTools_ShapePicker, vtkAbstractPropPicker);
+  vtkTypeMacro(IVtkTools_ShapePicker, vtkAbstractPropPicker)
+
   static IVtkTools_ShapePicker* New();
 
   //! Pick entities in the given point or area.
index 054b30421cc1dda14451d948b999a4292f14f6cb..1c75e6e0ec258e2e3b9aac2992cb555cdbfb3e81 100644 (file)
@@ -23,7 +23,7 @@
 #include <vtkIdTypeArray.h>
 
 
-vtkStandardNewMacro(IVtkTools_SubPolyDataFilter);
+vtkStandardNewMacro(IVtkTools_SubPolyDataFilter)
 
 //================================================================
 // Function : Constructor
index a78a73c6dd4a2c6c08646b6a386a206906313e32..d6072f2d85678f388bc17ec2e094af0a2ae3f79d 100644 (file)
@@ -26,7 +26,8 @@
 class IVtkTools_EXPORT IVtkTools_SubPolyDataFilter : public vtkPolyDataAlgorithm
 {
 public:
-  vtkTypeMacro(IVtkTools_SubPolyDataFilter,vtkPolyDataAlgorithm);
+  vtkTypeMacro(IVtkTools_SubPolyDataFilter,vtkPolyDataAlgorithm)
+
   static IVtkTools_SubPolyDataFilter *New();
   void PrintSelf (std::ostream& theOs, vtkIndent theIndent);
 
index 15a0d31c1b1a1118b36f783f39cbf8054976df2d..917b00e5b85d2c7e699fdfa53423dfe2a291ddb8 100644 (file)
@@ -27,8 +27,8 @@ IMPLEMENT_STANDARD_RTTIEXT(IVtkVTK_View,IVtk_IView)
 // Since VTK 6 the factory methods require "auto-initialization" depending on
 // what modules are enabled at VTK configure time.
 // Some defines are needed in order to make the factories work properly.
-VTK_MODULE_INIT(vtkRenderingOpenGL);
-VTK_MODULE_INIT(vtkInteractionStyle);
+VTK_MODULE_INIT(vtkRenderingOpenGL)
+VTK_MODULE_INIT(vtkInteractionStyle)
 
 // Handle implementation
 
index 68a525c48565b13973bc6c016c08426ccebaacdb..42c1c1d612536d854414d2832833cbbff5974e30 100644 (file)
@@ -101,7 +101,7 @@ namespace
         return FIT_UNKNOWN;
     }
   }
-};
+}
 #endif
 
 
index 85b4f2b7f784e1e8aa37c5918f1100edb8fd55ea..72a6252e5b78438e803724d685bf68ef5121cb67 100644 (file)
@@ -126,7 +126,8 @@ namespace
         || theFormat == Image_PixMap::ImgRGBA
         || theFormat == Image_PixMap::ImgBGRA;
   }
-}
+
+} // anonymous namespace
 
 // =======================================================================
 // function : Image_Diff
index 20a3bbad5d478c4e937a775989426dcdd161d77a..5823c1e70e77e9f899bd6878007a32de43c0e2b3 100644 (file)
@@ -32,11 +32,6 @@ extern "C" {
   #define fstatvfs fstatfs
 #else
   #include <sys/statvfs.h>
-  #ifdef __GNU_LIBRARY__
-    int statvfs(const char *, struct statvfs *) __THROW;
-  #else
-    int statvfs(const char *, struct statvfs *);
-  #endif
 #endif
 
 #ifdef __cplusplus
index 6d4258b013e77fa60197235103cbd80c5b575110..333bf9106a8c96f9320553f1a743ad27fe91a50c 100644 (file)
@@ -43,13 +43,6 @@ extern "C" {void    *dlerror (void);}
 
 #include <dlfcn.h>
 
-#ifdef __GNU_LIBRARY__
-extern "C" { size_t strlen(const char* s) __THROW; }
-#else
-extern "C" { size_t strlen(const char* s); }
-#endif
-
-
 #define BAD(X)  ((X) == NULL)
 
 // ----------------------------------------------------------------
index 7ffe273073a6d1f7c67cab924561c3c3fecbab0d..bf3b4d20bc56fa7a3f568956a89ad79f195e093f 100644 (file)
@@ -136,7 +136,7 @@ namespace
   }
 #endif
 
-}
+} // anonymous namespace
 
 // =======================================================================
 // function : OpenGl_Text
index 92f080c3ea9a8af18bd2f7b19a6631d03ceffb2a..98c6bcb52e587236eec255715740e5677641d277 100644 (file)
@@ -418,7 +418,8 @@ namespace
     aPrsGrp->SetGroupPrimitivesAspect (aBoundaryAspect);
     aPrsGrp->AddPrimitiveArray (aSegments);
   }
-}
+
+} // anonymous namespace
 
 // =======================================================================
 // function : ExploreSolids
index 3299fd38b8c00bd26b8e856bbdb3af22f37c6e05..13a68a7b614f754d296c30845833278a4f02d81d 100644 (file)
@@ -357,9 +357,9 @@ static Standard_Boolean FUN_findTOOF(const TopOpeBRepDS_PDataStructure pDS,
   return ok;
 } 
 
-Standard_EXPORT Standard_Boolean GLOBAL_btcx = Standard_False;void debtcx(){}
+Standard_EXPORT Standard_Boolean GLOBAL_btcx = Standard_False;
 Standard_EXPORT void debtcxmess(Standard_Integer f1,Standard_Integer f2,Standard_Integer il)
-{cout<<"f1,f2,il : "<<f1<<","<<f2<<","<<il<<endl;cout.flush();debtcx();}
+{cout<<"f1,f2,il : "<<f1<<","<<f2<<","<<il<<endl;cout.flush();}
 
 //=======================================================================
 //function : ProcessLine
index 51dbb9cde0fd1bfc07a1c3c52b806d07f254eaac..a82705dca1a1d6faa146c1437e42ab8e0cb10dbe 100644 (file)
@@ -6461,13 +6461,10 @@ public:
                             const Standard_ShortReal theXShift,
                             const Standard_ShortReal theYShift,
                             const Standard_ShortReal theAngle);
-  DEFINE_STANDARD_RTTI_INLINE(OCC_TextureEnv,Graphic3d_TextureEnv);
+  DEFINE_STANDARD_RTTI_INLINE(OCC_TextureEnv,Graphic3d_TextureEnv)
 };
 DEFINE_STANDARD_HANDLE(OCC_TextureEnv, Graphic3d_TextureEnv)
 
-
-
-
 OCC_TextureEnv::OCC_TextureEnv(const Standard_CString theFileName)
   : Graphic3d_TextureEnv(theFileName)
 {