[Libreoffice-bugs] [Bug 105355] Automatic Spell Checking highlights correctly spelled AutoInput word starting with capital letter

[bugzilla-daemon at bugs.documentfoundation.org]

https://bugs.documentfoundation.org/show_bug.cgi?id=105355

--- Comment #7 from Justin L <jluth at mail.com> ---
The code for this seems to be in sc/source/ui/app/inputhdl.cxx's
ScInputHandler::EnterHandler where mpEditEngine->CompleteOnlineSpelling()
checks spelling before bAutoComplete performs case-matching after that.

This is a very complicated function however. Reviewing some of the history,
these commits looked important/useful.

commit 891fd9e8a64d14d8c2b1593493ae04ccdfc2b1b7
Author: Rüdiger Timm on Tue Jul 3 14:53:54 2007 +0000
    2007/06/08 13:11:20 nn 1.68.54.1: #i47125# with AutoInput, do final
AutoCorrect at end of text

commit d8bb8c4fb850b9de762567ed7d337972467470d6
Author: Kohei Yoshida on Thu Sep 16 10:22:36 2010 +0200
    calc-autoinput-case-insensitive-matching.diff: no case matching on
autoinput


But MOST CONCERNING is that cellsh4.cxx calls ExecuteMove which again leads to
EnterHandler running with the wrong, lower-cased string!!

In fact, every time you move from one cell to the next (15 times if you move
left 15 times), you do all of that processing - NOT FOR THE CELL YOU ARE ON -
but for the last entry you typed.  a COMPLETE waste of CPU resources.

Obviously there are fundamental problems here. A calc guy needs to look into
this.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice-bugs/attachments/20200718/66231228/attachment.htm>