Fixed most clang -Wextra-semi warnings

diff --git a/modules/calib3d/src/chessboard.hpp b/modules/calib3d/src/chessboard.hpp
index 4c862ec..f49b835 100644 (file)
--- a/modules/calib3d/src/chessboard.hpp
+++ b/modules/calib3d/src/chessboard.hpp
@@ -43,7 +43,7 @@ class FastX : public cv::Feature2D
 
     public:
         FastX(const Parameters &config = Parameters());
-        virtual ~FastX(){};
+        virtual ~FastX(){}
 
         void reconfigure(const Parameters &para);
 
@@ -74,8 +74,8 @@ class FastX : public cv::Feature2D
         std::vector<std::vector<float> > calcAngles(const std::vector<cv::Mat> &rotated_images,
                                                             std::vector<cv::KeyPoint> &keypoints)const;
         // define pure virtual methods
-        virtual int descriptorSize()const override{return 0;};
-        virtual int descriptorType()const override{return 0;};
+        virtual int descriptorSize()const override{return 0;}
+        virtual int descriptorType()const override{return 0;}
         virtual void operator()( cv::InputArray image, cv::InputArray mask, std::vector<cv::KeyPoint>& keypoints, cv::OutputArray descriptors, bool useProvidedKeypoints=false )const
         {
             descriptors.clear();
@@ -620,10 +620,10 @@ class Chessboard: public cv::Feature2D
                  */
                 void swap(Chessboard::Board &other);
 
-                bool operator==(const Chessboard::Board& other) const {return rows*cols == other.rows*other.cols;};
-                bool operator< (const Chessboard::Board& other) const {return rows*cols < other.rows*other.cols;};
-                bool operator> (const Chessboard::Board& other) const {return rows*cols > other.rows*other.cols;};
-                bool operator>= (const cv::Size& size)const { return rows*cols >= size.width*size.height; };
+                bool operator==(const Chessboard::Board& other) const {return rows*cols == other.rows*other.cols;}
+                bool operator< (const Chessboard::Board& other) const {return rows*cols < other.rows*other.cols;}
+                bool operator> (const Chessboard::Board& other) const {return rows*cols > other.rows*other.cols;}
+                bool operator>= (const cv::Size& size)const { return rows*cols >= size.width*size.height; }
 
                 /**
                  * \brief Returns a specific corner
@@ -824,8 +824,8 @@ class Chessboard: public cv::Feature2D
         Chessboard::Board detectImpl(const cv::Mat& image,std::vector<cv::Mat> &feature_maps,const cv::Mat& mask)const;
 
         // define pure virtual methods
-        virtual int descriptorSize()const override{return 0;};
-        virtual int descriptorType()const override{return 0;};
+        virtual int descriptorSize()const override{return 0;}
+        virtual int descriptorType()const override{return 0;}
         virtual void operator()( cv::InputArray image, cv::InputArray mask, std::vector<cv::KeyPoint>& keypoints, cv::OutputArray descriptors, bool useProvidedKeypoints=false )const
         {
             descriptors.clear();

diff --git a/modules/calib3d/src/usac/estimator.cpp b/modules/calib3d/src/usac/estimator.cpp
index 75bc3cf..c5b783b 100644 (file)
--- a/modules/calib3d/src/usac/estimator.cpp
+++ b/modules/calib3d/src/usac/estimator.cpp
@@ -23,7 +23,7 @@ public:
     int estimateModelNonMinimalSample(const std::vector<int> &sample, int sample_size,
             std::vector<Mat> &models, const std::vector<double> &weights) const override {
         return non_min_solver->estimate (sample, sample_size, models, weights);
-    };
+    }
     int getMaxNumSolutions () const override {
         return min_solver->getMaxNumberOfSolutions();
     }
@@ -118,7 +118,7 @@ public:
     int estimateModelNonMinimalSample(const std::vector<int> &sample, int sample_size,
             std::vector<Mat> &models, const std::vector<double> &weights) const override {
         return non_min_solver->estimate(sample, sample_size, models, weights);
-    };
+    }
     int getMaxNumSolutions () const override {
         return min_solver->getMaxNumberOfSolutions();
     }

diff --git a/modules/core/include/opencv2/core/utils/trace.private.hpp b/modules/core/include/opencv2/core/utils/trace.private.hpp
index afc4115..5a826b4 100644 (file)
--- a/modules/core/include/opencv2/core/utils/trace.private.hpp
+++ b/modules/core/include/opencv2/core/utils/trace.private.hpp
@@ -63,7 +63,7 @@ class TraceMessage;
 class TraceStorage {
 public:
     TraceStorage() {}
-    virtual ~TraceStorage() {};
+    virtual ~TraceStorage() {}
 
     virtual bool put(const TraceMessage& msg) const = 0;
 };

diff --git a/modules/core/src/parallel/registry_parallel.impl.hpp b/modules/core/src/parallel/registry_parallel.impl.hpp
index c8b57e7..2208748 100644 (file)
--- a/modules/core/src/parallel/registry_parallel.impl.hpp
+++ b/modules/core/src/parallel/registry_parallel.impl.hpp
@@ -43,7 +43,7 @@ std::vector<ParallelBackendInfo>& getBuiltinParallelBackendsInfo()
 #endif
     };
     return g_backends;
-};
+}
 
 static
 bool sortByPriority(const ParallelBackendInfo &lhs, const ParallelBackendInfo &rhs)

diff --git a/modules/features2d/src/fast.avx2.cpp b/modules/features2d/src/fast.avx2.cpp
index ca0bcb9..72e7d66 100644 (file)
--- a/modules/features2d/src/fast.avx2.cpp
+++ b/modules/features2d/src/fast.avx2.cpp
@@ -165,7 +165,7 @@ public:
         _mm256_zeroupper();
     }
 
-    virtual ~FAST_t_patternSize16_AVX2_Impl() CV_OVERRIDE {};
+    virtual ~FAST_t_patternSize16_AVX2_Impl() CV_OVERRIDE {}
 
 private:
     int cols;

diff --git a/modules/features2d/src/fast.hpp b/modules/features2d/src/fast.hpp
index 6f750fe..1e221cb 100644 (file)
--- a/modules/features2d/src/fast.hpp
+++ b/modules/features2d/src/fast.hpp
@@ -54,7 +54,7 @@ class FAST_t_patternSize16_AVX2
 public:
     static Ptr<FAST_t_patternSize16_AVX2> getImpl(int _cols, int _threshold, bool _nonmax_suppression, const int* _pixel);
     virtual void process(int &j, const uchar* &ptr, uchar* curr, int* cornerpos, int &ncorners) = 0;
-    virtual ~FAST_t_patternSize16_AVX2() {};
+    virtual ~FAST_t_patternSize16_AVX2() {}
 };
 #endif
 }

diff --git a/modules/imgcodecs/include/opencv2/imgcodecs.hpp b/modules/imgcodecs/include/opencv2/imgcodecs.hpp
index cb4a170..18b04d5 100644 (file)
--- a/modules/imgcodecs/include/opencv2/imgcodecs.hpp
+++ b/modules/imgcodecs/include/opencv2/imgcodecs.hpp
@@ -354,8 +354,8 @@ public:
         Mat* operator->();
         iterator& operator++();
         iterator operator++(int);
-        friend bool operator== (const iterator& a, const iterator& b) { return a.m_curr == b.m_curr; };
-        friend bool operator!= (const iterator& a, const iterator& b) { return a.m_curr != b.m_curr; };
+        friend bool operator== (const iterator& a, const iterator& b) { return a.m_curr == b.m_curr; }
+        friend bool operator!= (const iterator& a, const iterator& b) { return a.m_curr != b.m_curr; }
 
     private:
         ImageCollection* m_pCollection;

diff --git a/modules/imgproc/src/filter.dispatch.cpp b/modules/imgproc/src/filter.dispatch.cpp
index 097dd32..53b8bb4 100644 (file)
--- a/modules/imgproc/src/filter.dispatch.cpp
+++ b/modules/imgproc/src/filter.dispatch.cpp
@@ -1401,7 +1401,7 @@ static void ocvSepFilter(int stype, int dtype, int ktype,
     Mat src(Size(width, height), stype, src_data, src_step);
     Mat dst(Size(width, height), dtype, dst_data, dst_step);
     f->apply(src, dst, Size(full_width, full_height), Point(offset_x, offset_y));
-};
+}
 
 //===================================================================
 //       HAL functions

diff --git a/modules/imgproc/src/imgwarp.hpp b/modules/imgproc/src/imgwarp.hpp
index 4b81b5e..35e9f7b 100644 (file)
--- a/modules/imgproc/src/imgwarp.hpp
+++ b/modules/imgproc/src/imgwarp.hpp
@@ -82,7 +82,7 @@ public:
     static Ptr<WarpPerspectiveLine_SSE4> getImpl(const double *M);
     virtual void processNN(const double *M, short* xy, double X0, double Y0, double W0, int bw) = 0;
     virtual void process(const double *M, short* xy, short* alpha, double X0, double Y0, double W0, int bw) = 0;
-    virtual ~WarpPerspectiveLine_SSE4() {};
+    virtual ~WarpPerspectiveLine_SSE4() {}
 };
 #endif
 }

diff --git a/modules/imgproc/src/imgwarp.sse4_1.cpp b/modules/imgproc/src/imgwarp.sse4_1.cpp
index 5625b45..a2ec939 100644 (file)
--- a/modules/imgproc/src/imgwarp.sse4_1.cpp
+++ b/modules/imgproc/src/imgwarp.sse4_1.cpp
@@ -492,7 +492,7 @@ public:
                 (X & (INTER_TAB_SIZE - 1)));
         }
     }
-    virtual ~WarpPerspectiveLine_SSE4_Impl() CV_OVERRIDE {};
+    virtual ~WarpPerspectiveLine_SSE4_Impl() CV_OVERRIDE {}
 };
 
 Ptr<WarpPerspectiveLine_SSE4> WarpPerspectiveLine_SSE4::getImpl(const double *M)

diff --git a/modules/imgproc/src/resize.cpp b/modules/imgproc/src/resize.cpp
index 4ad6453..f388587 100644 (file)
--- a/modules/imgproc/src/resize.cpp
+++ b/modules/imgproc/src/resize.cpp
@@ -339,7 +339,7 @@ template <typename ET, typename FT, int n, bool mulall, int cncnt>
 static void hlineResizeCn(ET* src, int cn, int *ofst, FT* m, FT* dst, int dst_min, int dst_max, int dst_width)
 {
     hline<ET, FT, n, mulall, cncnt>::ResizeCn(src, cn, ofst, m, dst, dst_min, dst_max, dst_width);
-};
+}
 
 template <>
 void hlineResizeCn<uint8_t, ufixedpoint16, 2, true, 1>(uint8_t* src, int, int *ofst, ufixedpoint16* m, ufixedpoint16* dst, int dst_min, int dst_max, int dst_width)

diff --git a/modules/ml/src/inner_functions.cpp b/modules/ml/src/inner_functions.cpp
index b823c5b..6b3affc 100644 (file)
--- a/modules/ml/src/inner_functions.cpp
+++ b/modules/ml/src/inner_functions.cpp
@@ -126,10 +126,10 @@ public:
 
         errStrip[idxErr]=err ;
 
-    };
+    }
     ParallelCalcError& operator=(const ParallelCalcError &) {
         return *this;
-    };
+    }
 };
 
 

diff --git a/modules/python/src2/cv2_numpy.hpp b/modules/python/src2/cv2_numpy.hpp
index 7c386c7..9343339 100644 (file)
--- a/modules/python/src2/cv2_numpy.hpp
+++ b/modules/python/src2/cv2_numpy.hpp
@@ -51,13 +51,13 @@ NPY_TYPES asNumpyType<bool>()
         return NPY_U##dst;                                 \
     }
 
-CV_GENERATE_INTEGRAL_TYPE_NPY_CONVERSION(int8_t, INT8);
+CV_GENERATE_INTEGRAL_TYPE_NPY_CONVERSION(int8_t, INT8)
 
-CV_GENERATE_INTEGRAL_TYPE_NPY_CONVERSION(int16_t, INT16);
+CV_GENERATE_INTEGRAL_TYPE_NPY_CONVERSION(int16_t, INT16)
 
-CV_GENERATE_INTEGRAL_TYPE_NPY_CONVERSION(int32_t, INT32);
+CV_GENERATE_INTEGRAL_TYPE_NPY_CONVERSION(int32_t, INT32)
 
-CV_GENERATE_INTEGRAL_TYPE_NPY_CONVERSION(int64_t, INT64);
+CV_GENERATE_INTEGRAL_TYPE_NPY_CONVERSION(int64_t, INT64)
 
 #undef CV_GENERATE_INTEGRAL_TYPE_NPY_CONVERSION
 

diff --git a/modules/python/src2/cv2_util.hpp b/modules/python/src2/cv2_util.hpp
index 3662ffc..0d27e98 100644 (file)
--- a/modules/python/src2/cv2_util.hpp
+++ b/modules/python/src2/cv2_util.hpp
@@ -12,7 +12,7 @@
 bool isPythonBindingsDebugEnabled();
 void emit_failmsg(PyObject * exc, const char *msg);
 int failmsg(const char *fmt, ...);
-PyObject* failmsgp(const char *fmt, ...);;
+PyObject* failmsgp(const char *fmt, ...);
 
 //======================================================================================================================