tdf#119388 only enqueue events we actually process

shaves 10% of the time off, deleting columns

Change-Id: Id992572419a3c9a5693d37d7c5e5fe2da955bb23
Reviewed-on: https://gerrit.libreoffice.org/78029
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 10 insertions, 3 deletions

diff --git a/svx/source/accessibility/AccessibleTextEventQueue.cxx b/svx/source/accessibility/AccessibleTextEventQueue.cxx
index 6a586117ddd3..177950aac8ce 100644
--- a/svx/source/accessibility/AccessibleTextEventQueue.cxx
+++ b/svx/source/accessibility/AccessibleTextEventQueue.cxx
@@ -49,7 +49,13 @@ namespace accessibility
 
     void AccessibleTextEventQueue::Append( const SdrHint& rHint )
     {
-        maEventQueue.push_back( new SdrHint( rHint ) );
+        // only enqueue the events we actually care about in
+        // AccessibleTextHelper_Impl::ProcessQueue(), because
+        // the cost of some events adds up.
+        auto eKind = rHint.GetKind();
+        if (eKind == SdrHintKind::BeginEdit
+            || eKind == SdrHintKind::EndEdit)
+            maEventQueue.push_back( new SdrHint( rHint ) );
     }
 
     void AccessibleTextEventQueue::Append( const TextHint& rHint )
@@ -82,8 +88,9 @@ namespace accessibility
     void AccessibleTextEventQueue::Clear()
     {
         // clear queue
-        while( !IsEmpty() )
-            PopFront();
+        for( auto p : maEventQueue)
+            delete p;
+        maEventQueue.clear();
     }
 
 } // end of namespace accessibility