Skip to content

Commit

Permalink
[codemod][lowrisk] Remove extra semi colon from caffe2/caffe2/core/ob…
Browse files Browse the repository at this point in the history
…server.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: #126976
Approved by: https://github.com/Skylion007
  • Loading branch information
r-barnes authored and pytorchmergebot committed May 23, 2024
1 parent a63310e commit bbe68a1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions caffe2/core/observer.h
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ class ObserverBase {
return "Not implemented.";
}

virtual ~ObserverBase() noexcept {};
virtual ~ObserverBase() noexcept {}

T* subject() const {
return subject_;
Expand All @@ -32,7 +32,7 @@ class ObserverBase {
virtual std::unique_ptr<ObserverBase<T>> rnnCopy(T* subject, int rnn_order)
const {
return nullptr;
};
}

protected:
T* subject_;
Expand Down

0 comments on commit bbe68a1

Please sign in to comment.