[PATCH libreoffice-4-0] only reset marked area when using find/replace all, fdo#5310...

Markus Mohrhard (via Code Review) gerrit at gerrit.libreoffice.org
Thu Jun 6 21:49:11 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/4185

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/85/4185/1

only reset marked area when using find/replace all, fdo#53106

Change-Id: I151db5db7bcdf5295e1f67423de8926250ae8ea3
---
M sc/source/ui/view/viewfun2.cxx
1 file changed, 8 insertions(+), 5 deletions(-)



diff --git a/sc/source/ui/view/viewfun2.cxx b/sc/source/ui/view/viewfun2.cxx
index 590e134..2a02c94 100644
--- a/sc/source/ui/view/viewfun2.cxx
+++ b/sc/source/ui/view/viewfun2.cxx
@@ -1647,12 +1647,15 @@
                                         aUndoStr, pUndoDoc.release(), pSearchItem ) );
             }
 
-            rMark.ResetMark();
-            for (size_t i = 0, n = aMatchedRanges.size(); i < n; ++i)
+            if (nCommand == SVX_SEARCHCMD_FIND_ALL || nCommand == SVX_SEARCHCMD_REPLACE_ALL)
             {
-                const ScRange& r = *aMatchedRanges[i];
-                if (r.aStart.Tab() == nTab)
-                    rMark.SetMultiMarkArea(r);
+                rMark.ResetMark();
+                for (size_t i = 0, n = aMatchedRanges.size(); i < n; ++i)
+                {
+                    const ScRange& r = *aMatchedRanges[i];
+                    if (r.aStart.Tab() == nTab)
+                        rMark.SetMultiMarkArea(r);
+                }
             }
 
             break;                  // break 'while (TRUE)'

-- 
To view, visit https://gerrit.libreoffice.org/4185
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I151db5db7bcdf5295e1f67423de8926250ae8ea3
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Markus Mohrhard <markus.mohrhard at googlemail.com>



More information about the LibreOffice mailing list