On 05/19/2013 03:29 AM, Daniel Naber wrote:
> Am 19.05.2013 05:30, schrieb Andriy Rysin:
>
>> The patch is below. Basically it adds
>> ignoreToken(AnalyzedTokenReadings[] tokens, int idx) method that
>> provides context of the token (whole sentence) instead of just a word
>> text (defaulting to calling old ignoreWord() method).
> That makes sense. But as this is more powerful than the old
> ignoreWord(), we should maybe replace that? As users might rely on
> ignoreWord(), the right approach seems to be to deprecate ignoreWord(),
> and add ignoreToken() to SpellingCheckRule instead of
> MorfologikSpellerRule?
ok, one more question ignoreWord() in SpellingCheckRule has these two lines:
// TODO?: this is needed at least for German as Hunspell
tokenization includes the dot:
final String cleanWord = word.endsWith(".") ? word.substring(0,
word.length() - 1) : word;
it prevents me from checking words with dots against the ignore.txt
list. Is this line really needed in common code? If yes I'll have to
either expose ignoredWords set to subclass or do some other workaround.
Thanks
Andriy
------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_may
_______________________________________________
Languagetool-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/languagetool-devel
