diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-01-14 15:34:38 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-01-16 17:24:58 +0100 |
commit | 5d2f157b25305caaff914a756727b85451abb852 (patch) | |
tree | 2eff210e27f4032217cde2119245112872b56b3a /sc/source/ui/miscdlgs/acredlin.cxx | |
parent | 9796738e1149a99f8b3ff687b0f72264ba3a56ff (diff) |
simplify ScChangeTrack queues
no need to store such small struct on the heap
Change-Id: I8d9c12082a40e1d24cf7d25cca7369e1bb034c7a
Reviewed-on: https://gerrit.libreoffice.org/66323
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc/source/ui/miscdlgs/acredlin.cxx')
-rw-r--r-- | sc/source/ui/miscdlgs/acredlin.cxx | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/sc/source/ui/miscdlgs/acredlin.cxx b/sc/source/ui/miscdlgs/acredlin.cxx index ad4d2ba5e61c..25e95a525d34 100644 --- a/sc/source/ui/miscdlgs/acredlin.cxx +++ b/sc/source/ui/miscdlgs/acredlin.cxx @@ -1541,16 +1541,16 @@ IMPL_LINK( ScAcceptChgDlg, ChgTrackModHdl, ScChangeTrack&, rChgTrack, void) sal_uLong nStartAction; sal_uLong nEndAction; - for (const auto& pMsg : aMsgQueue) + for (const auto& rMsg : aMsgQueue) { - nStartAction = pMsg->nStartAction; - nEndAction = pMsg->nEndAction; + nStartAction = rMsg.nStartAction; + nEndAction = rMsg.nEndAction; if(!bIgnoreMsg) { bNoSelection=true; - switch(pMsg->eMsgType) + switch(rMsg.eMsgType) { case SC_CTM_APPEND: AppendChanges(&rChgTrack,nStartAction,nEndAction); break; @@ -1560,13 +1560,10 @@ IMPL_LINK( ScAcceptChgDlg, ChgTrackModHdl, ScChangeTrack&, rChgTrack, void) case SC_CTM_CHANGE: //bNeedsUpdate=true; UpdateEntrys(&rChgTrack,nStartAction,nEndAction); break; - default: - { + default: ; // added to avoid warnings - } } } - delete pMsg; } aMsgQueue.clear(); |