Issue 7328 - ISO-8859-1 bug in myspell.cxx
Summary: ISO-8859-1 bug in myspell.cxx
Status: CLOSED FIXED
Alias: None
Product: Internationalization
Classification: Code
Component: code (show other issues)
Version: current
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: khendricks
QA Contact: issues@l10n
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-08-29 20:53 UTC by dan
Modified: 2006-12-15 15:28 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description dan 2002-08-29 20:53:50 UTC 
On line 60 of myspell.cxx is defined array of special characters. This is also
used for all languages/encoding, but is valid only for ISO-8859-1. For us
(ISO-8859-2) are that regular characters. This bug prevetns correct
spellchecking for Czech (and possibly for other languages too).

There shloud be call to locale to fetch array of correct/incorrect characters,
or there should be for any encoding another array switched by encodings variable.

I can also supply array od special characters for ISO-8859-2, if required.
Comment 1 khendricks 2002-10-15 13:12:57 UTC 
Hi,  
  
This special chars array was added as a hack to work around  
incorrect parsing of text done by breakiterator.  When the new  
breakiterator starts getting used, we should be able to remove this  
workaround completely, removing the problem.  
  
If you can supply a better special chars array that is valid for 
most ISO-2 based lanaguages (and not just one specific language), I 
will include it in lingucomponent for the next build. 
 
If some workaround still needs to be included, then I will add an  
optional REMOVE line to the .aff file format that will be used to  
replace the special chars array if present on a language/affix file 
specific basis.  
  
I am still waiting to hear reports of success /failure from the next  
build before proceeding further.  
  
Hope this helps,  
  
Kevin
Comment 2 khendricks 2002-10-31 19:30:47 UTC 
removed all nonscii chars for nect OO643C myspell to test the new 
breakiterator to see if the special chars things can be removed completely
Comment 3 khendricks 2003-03-24 18:39:49 UTC 
Hi, 
 
I am now resolving this as fixed with the new breakiterator, I no longer need the 
debatable special chars tests. 
 
Kevin
Comment 4 caolanm 2006-12-15 15:28:29 UTC 
closed