Abdelrazak Younes <[EMAIL PROTECTED]> writes:

| Georg Baum wrote:
| > Am Sonntag, 3. September 2006 12:28 schrieb Abdelrazak Younes:
| >> Georg Baum wrote:
| >>> Am Sonntag, 3. September 2006 12:11 schrieb Abdelrazak Younes:
| >>>> This patch fixes the isDigit() MSVC warning in ControlSpellchecker.C.
| >>> If that is really the correct isDigit() check for unciode
| >> My understanding is that ascii is a subset of ucs4 so it must work as is.
| > That is true, but the question is rather: For what purpose is
| > isDigits used? Shall it be true for digits in other scripts, or are
| > ASCII digits enough? I don't know.
| 
| For one, isDigit() is used in LyXText::insertChar() and it works with
| my change.
| isLineSeparatorChar() is also used there and it works as expected.

Yes, but it is not unicode correct.

We need to lookup the unicode tables to discover if a codepoint is a
number or not.

-- 
        Lgb

Reply via email to