Merge pull request #20954 from shengyu7697:remove-redundant-semicolon

modules/gapi/include/opencv2/gapi/fluid/gfluidbuffer.hpp

@@ -25,7 +25,7 @@ namespace fluid {
 struct Border
 {
     // This constructor is required to support existing kernels which are part of G-API
-    Border(int _type, cv::Scalar _val) : type(_type), value(_val) {};
+    Border(int _type, cv::Scalar _val) : type(_type), value(_val) {}
 
     int type;
     cv::Scalar value;

modules/gapi/include/opencv2/gapi/gtransform.hpp

@@ -31,7 +31,7 @@ struct GAPI_EXPORTS GTransform
     F pattern;
     F substitute;
 
-    GTransform(const std::string& d, const F &p, const F &s) : description(d), pattern(p), substitute(s){};
+    GTransform(const std::string& d, const F &p, const F &s) : description(d), pattern(p), substitute(s) {}
 };
 
 namespace detail

modules/gapi/include/opencv2/gapi/plaidml/gplaidmlkernel.hpp

@@ -59,7 +59,7 @@ public:
     using F = std::function<void(GPlaidMLContext &)>;
 
     GPlaidMLKernel() = default;
-    explicit GPlaidMLKernel(const F& f) : m_f(f) {};
+    explicit GPlaidMLKernel(const F& f) : m_f(f) {}
 
     void apply(GPlaidMLContext &ctx) const
     {

modules/gapi/include/opencv2/gapi/streaming/onevpl/data_provider_interface.hpp

@@ -17,7 +17,7 @@ namespace wip {
 namespace onevpl {
 
 struct GAPI_EXPORTS DataProviderException : public std::exception {
-    virtual ~DataProviderException() {};
+    virtual ~DataProviderException() {}
 };
 
 struct GAPI_EXPORTS DataProviderSystemErrorException : public DataProviderException {
@@ -42,7 +42,7 @@ private:
 struct GAPI_EXPORTS IDataProvider {
     using Ptr = std::shared_ptr<IDataProvider>;
 
-    virtual ~IDataProvider() {};
+    virtual ~IDataProvider() {}
 
     /**
      * The function is used by onevpl::GSource to extract binary data stream from @ref IDataProvider

modules/gapi/include/opencv2/gapi/util/optional.hpp

@@ -33,7 +33,7 @@ namespace util
         // Constructors
         // NB.: there were issues with Clang 3.8 when =default() was used
         // instead {}
-        optional() {};
+        optional() {}
         optional(const optional&) = default;
         explicit optional(T&&) noexcept;
         explicit optional(const T&) noexcept;

modules/gapi/src/compiler/gislandmodel.hpp

@@ -140,7 +140,7 @@ public:
     // FIXME: This thing will likely break stuff once we introduce
     // "multi-source streaming", a better design needs to be proposed
     // at that stage.
-    virtual void handleNewStream() {}; // do nothing here by default
+    virtual void handleNewStream() {} // do nothing here by default
 
     // This method is called for every IslandExecutable when
     // the stream-based execution is stopped.

modules/gapi/src/compiler/transactions.hpp

@@ -69,8 +69,8 @@ struct ChangeT
 {
     struct Base
     {
-        virtual void commit  (ade::Graph & ) {};
-        virtual void rollback(ade::Graph & ) {};
+        virtual void commit  (ade::Graph & ) {}
+        virtual void rollback(ade::Graph & ) {}
         virtual ~Base() = default;
     };
 

modules/gapi/src/executor/gtbbexecutor.hpp

@@ -83,8 +83,8 @@ struct tile_node {
 
     std::vector<tile_node*>                         dependants;
 
-    tile_node(decltype(sync_task_body::body)&& f) : task_body(sync_task_body{std::move(f)}) {};
-    tile_node(async_tag, decltype(async_task_body::body)&& f) : task_body(async_task_body{std::move(f)}) {};
+    tile_node(decltype(sync_task_body::body)&& f) : task_body(sync_task_body{std::move(f)}) {}
+    tile_node(async_tag, decltype(async_task_body::body)&& f) : task_body(async_task_body{std::move(f)}) {}
 };
 
 std::ostream& operator<<(std::ostream& o, tile_node const& n);