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

--- Comment #8 from Mike Kaganski <[email protected]> ---
(In reply to Dave McKellar from comment #7)

And you even may remove one check that is already built into the inline
toAsciiUpperCase [1]:

> if (!rtl::isAsciiAlpha(cChar))
>     break;
> cChar = rtl::toAsciiUpperCase(cChar) - 'A';

That would be a nice cleanup, and I see how it would improve the code; but I
don't see how would that *change* current behavior.

[1]
https://opengrok.libreoffice.org/xref/core/include/rtl/character.hxx?r=b1fb6338#290

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to