Skip to content

Commit

Permalink
Add missing override to ~Impl()
Browse files Browse the repository at this point in the history
The other virtual methods are marked with `override`, so there's no good reason for the destructor to not have this too.
  • Loading branch information
ben-clayton committed Dec 11, 2023
1 parent 869caba commit bbde7fb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/session.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ class Impl : public dap::Session {
return send(s.dump());
}

~Impl() {
~Impl() override {
inbox.close();
reader.close();
writer.close();
Expand Down

0 comments on commit bbde7fb

Please sign in to comment.