diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-09-18 14:15:59 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-09-19 11:42:47 +0200 |
| commit | 82572caae4a282cdf79456b977508ca71507c584 (patch) | |
| tree | befc8478f8cd9bf7e7c9a87a776b063a9ed4b718 /sc | |
| parent | 71ef762f21ada8c25aad2183065478171e985e8c (diff) | |
improve and enable loplugin:fragiledestructor
Where the problem was benign and the class was not extended, I marked
the class as final.
Where the problem was benign and the class was extended, I marked the
relevant callee methods as final.
Other cases were excluded in the plugin.
Change-Id: Idb762fb2206af4e8b534aa35ff77f8368c7909bc
Reviewed-on: https://gerrit.libreoffice.org/79089
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/inc/chgtrack.hxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sc/inc/chgtrack.hxx b/sc/inc/chgtrack.hxx index 9c8cc6ab4b5c..502d463a6c3e 100644 --- a/sc/inc/chgtrack.hxx +++ b/sc/inc/chgtrack.hxx @@ -426,7 +426,7 @@ public: short GetCutOffTo() const { return nCutOffTo; } }; -class ScChangeActionDel : public ScChangeAction +class ScChangeActionDel final : public ScChangeAction { friend class ScChangeTrack; friend void ScChangeAction::Accept(); @@ -499,7 +499,7 @@ public: }; // ScChangeActionMove -class ScChangeActionMove : public ScChangeAction +class ScChangeActionMove final : public ScChangeAction { friend class ScChangeTrack; friend struct std::default_delete<ScChangeActionMove>; // for std::unique_ptr |