revert "vlookup - optimize SC_EQUAL and NOT_EQUAL." (tdf#139612)
commit8dec2a98ce29251936cd45ebf864a89ff767ee50
authorLuboš Luňák <l.lunak@collabora.com>
Tue, 16 Nov 2021 00:30:24 +0000 (16 01:30 +0100)
committerLuboš Luňák <l.lunak@collabora.com>
Wed, 17 Nov 2021 09:10:58 +0000 (17 10:10 +0100)
tree5b336bda4ba803f2859bc0bdc0a57e89ab6213d8
parentfcad2503ede92b515076f9bb3162855dcc2c575d
revert "vlookup - optimize SC_EQUAL and NOT_EQUAL." (tdf#139612)

That commit breaks lookup when the  "Search criteria = and <> must
apply to whole cells" option is disabled, as it enforces whole cell
checking regardless of the option. Given that the option is enabled
by default in LO ('SearchCriteria' in Calc.xcs) and it's what
MSOffice does as well, and this default gives good performance
regardless of the option, I don't understand the purpose
of the commit. Possibly it was based on a document where somebody
disabled the option and then indeed got worse performance.
Solution: Don't do that :).

This reverts the code parts of a953fa1c0f6a40a08859570516c511f3a841 .
The test I've kept but switched to ensure that partial matching
does work if the option to match whole cells is disabled. I've
also changed the tooltip for the option to mention performance
and not suggest that off is the default.

Change-Id: I56d7b6e7b8e9f0622f7ad6d447daf56c3b705a7e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/125267
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
sc/qa/unit/data/functions/spreadsheet/fods/vlookup2.fods
sc/source/core/data/table3.cxx
sc/uiconfig/scalc/ui/optcalculatepage.ui