diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2021-03-11 13:50:09 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2021-03-12 11:23:58 +0100 |
| commit | 9aa60d592925e2e9d5219799c26639eec3fb4a53 (patch) | |
| tree | d828ee9e678ccb59fe33f861ad90fb475d1723e2 /vcl/unx/gtk3/gtk3gtkinst.cxx | |
| parent | 1d0b2ae1423142f3688de00d9938917329e958af (diff) | |
Avoid modifying then restoring this const iterator, its copy is trivial
Change-Id: I354010fbc47087bf7587a9662b9218606ba37dd4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112344
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'vcl/unx/gtk3/gtk3gtkinst.cxx')
| -rw-r--r-- | vcl/unx/gtk3/gtk3gtkinst.cxx | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/vcl/unx/gtk3/gtk3gtkinst.cxx b/vcl/unx/gtk3/gtk3gtkinst.cxx index f1e99eccca07..bdebc7985ba4 100644 --- a/vcl/unx/gtk3/gtk3gtkinst.cxx +++ b/vcl/unx/gtk3/gtk3gtkinst.cxx @@ -11902,12 +11902,8 @@ public: virtual bool iter_has_child(const weld::TreeIter& rIter) const override { - weld::TreeIter& rNonConstIter = const_cast<weld::TreeIter&>(rIter); - GtkInstanceTreeIter& rGtkIter = static_cast<GtkInstanceTreeIter&>(rNonConstIter); - GtkTreeIter restore(rGtkIter.iter); - bool ret = iter_children(rNonConstIter); - rGtkIter.iter = restore; - return ret; + GtkInstanceTreeIter aTempCopy(static_cast<const GtkInstanceTreeIter*>(&rIter)); + return iter_children(aTempCopy); } virtual bool get_row_expanded(const weld::TreeIter& rIter) const override |