sc/source/ui/app/inputhdl.cxx |    4 ++++
 1 file changed, 4 insertions(+)

New commits:
commit b48bcb45d7c503eeb5ec54085d3261ceba7393fd
Author:     Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Sat Nov 30 08:24:55 2019 +0200
Commit:     Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Sat Nov 30 11:31:56 2019 +0100

    Clarify the comment
    
    Change-Id: I610057d4226616cad928eba82ce0055b37fe4e16
    Reviewed-on: https://gerrit.libreoffice.org/84102
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>

diff --git a/sc/source/ui/app/inputhdl.cxx b/sc/source/ui/app/inputhdl.cxx
index 2ea4ba7a6c2d..2d03cdecb5df 100644
--- a/sc/source/ui/app/inputhdl.cxx
+++ b/sc/source/ui/app/inputhdl.cxx
@@ -1489,6 +1489,10 @@ void completeFunction( EditView* pView, const OUString& 
rInsert, bool& rParInser
             // Since transliteration service is used to test for match, the 
replaced string could be
             // longer than rInsert, so in order to find longest match before 
the cursor, test whole
             // string from start to current cursor position (don't limit to 
length of rInsert)
+            // Disclaimer: I really don't know if a match longer than rInsert 
is actually possible,
+            // so the above is based on assumptions how "transliteration" 
might possibly work. If
+            // it's in fact impossible, an optimization would be useful to 
limit aSel.nStartPos to
+            // std::max(sal_Int32(0), aSel.nEndPos - rInsert.getLength()).
             aSel.nStartPos = 0;
             pView->SetSelection(aSel);
             const OUString aAll = pView->GetSelected();
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to