canvas/source/tools/pagemanager.cxx |    5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

New commits:
commit be1093cce53a3c34a67233391922674c7c62f4e3
Author:     Julien Nabet <[email protected]>
AuthorDate: Thu Nov 16 18:32:30 2023 +0100
Commit:     Julien Nabet <[email protected]>
CommitDate: Fri Nov 17 08:29:04 2023 +0100

    c++20: use std::erase instead of std::removed followed by erase
    
    See https://en.cppreference.com/w/cpp/container/vector/erase2
    
    Change-Id: I26744e74b34d1efa1524a1f88aa3af9e153bc9ee
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159515
    Tested-by: Jenkins
    Reviewed-by: Julien Nabet <[email protected]>

diff --git a/canvas/source/tools/pagemanager.cxx 
b/canvas/source/tools/pagemanager.cxx
index 6ffd7ee01b58..4ee7df76ed7e 100644
--- a/canvas/source/tools/pagemanager.cxx
+++ b/canvas/source/tools/pagemanager.cxx
@@ -65,10 +65,7 @@ namespace canvas
     {
         // erase the reference to the given fragment from our
         // internal container.
-        FragmentContainer_t::iterator it(
-            std::remove(
-                maFragments.begin(),maFragments.end(),pFragment));
-        maFragments.erase(it,maFragments.end());
+        std::erase(maFragments, pFragment);
 
         // let the fragment itself know about it...
         // we need to pass 'this' as argument since the fragment

Reply via email to