using STL and lambda expressions is better

Change-Id: I0224dccb130ead02da46f2376b3df5ec889f8e8c
Signed-off-by: Adrien Ollier <adr.ollier@hotmail.fr>
Reviewed-on: https://gerrit.libreoffice.org/71812
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

vcl/source/animate/Animation.cxx

@@ -229,16 +229,11 @@ bool Animation::Start(OutputDevice* pOut, const Point& rDestPt, const Size& rDes
 
 void Animation::Stop(OutputDevice* pOut, long nExtraData)
 {
-    for (size_t i = 0; i < maViewList.size();)
-    {
-        ImplAnimView* pView = maViewList[i].get();
-        if (pView->matches(pOut, nExtraData))
-        {
-            maViewList.erase(maViewList.begin() + i);
-        }
-        else
-            i++;
-    }
+    maViewList.erase(std::remove_if(maViewList.begin(), maViewList.end(),
+                                    [=](const std::unique_ptr<ImplAnimView>& pAnimView) -> bool {
+                                        return pAnimView->matches(pOut, nExtraData);
+                                    }),
+                     maViewList.end());
 
     if (maViewList.empty())
     {