From bbe68a16b958d5aa3292eef4e1e447fe0ed3b4d3 Mon Sep 17 00:00:00 2001 From: Richard Barnes Date: Thu, 23 May 2024 17:31:17 +0000 Subject: [PATCH] [codemod][lowrisk] Remove extra semi colon from caffe2/caffe2/core/observer.h (#126976) Summary: `-Wextra-semi` or `-Wextra-semi-stmt` If the code compiles, this is safe to land. Test Plan: Sandcastle Reviewed By: palmje Differential Revision: D57632765 Pull Request resolved: https://github.com/pytorch/pytorch/pull/126976 Approved by: https://github.com/Skylion007 --- caffe2/core/observer.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/caffe2/core/observer.h b/caffe2/core/observer.h index 378a7569d37bb..3897bb76b52a2 100644 --- a/caffe2/core/observer.h +++ b/caffe2/core/observer.h @@ -23,7 +23,7 @@ class ObserverBase { return "Not implemented."; } - virtual ~ObserverBase() noexcept {}; + virtual ~ObserverBase() noexcept {} T* subject() const { return subject_; @@ -32,7 +32,7 @@ class ObserverBase { virtual std::unique_ptr> rnnCopy(T* subject, int rnn_order) const { return nullptr; - }; + } protected: T* subject_;