48865 2004-04-24 03:15 0000 patch for aspell-0.50.3.ebuild 2004-07-30 14:09:14 0000 1 1 1 Unclassified Gentoo Linux Ebuilds unspecified All Linux RESOLVED FIXED P1 normal --- 1 powerman-asdf@yandex.ru app-dicts@gentoo.org powerman-asdf@yandex.ru 2004-04-24 03:15:01 0000 aspell incorrectly filter out text in --mode=email : it skip lines with "cyrillic small letter o" found in first 10 line characters. This happens because aspell incorrectly match this "small o" with ">" symbol, which is correct "email quotation mark" (in --mode=email aspell skip lines with ">" symbol within first 10 characters of line). As far as I understand, these two symbols match because aspell incorrectly work with Unicode (it cast <int> to <unsigned char>), look at symbol codes: 003E GREATER-THAN SIGN 043E CYRILLIC SMALL LETTER O I've created patch. It's worked, but I'm not sure is it doing right thing because I don't know C++ at all. :^) Reproducible: Always Steps to Reproduce: 1. 2. 3. powerman-asdf@yandex.ru 2004-04-24 03:16:47 0000 Created an attachment (id=29946) patch... :) Add to .ebuild something like this: src_unpack() { unpack ${A} cd ${S} epatch ${FILESDIR}/${P}-quotechar.patch } arj@gentoo.org 2004-07-15 12:28:35 0000 Does this still apply to aspell 0.50.5-r2? powerman-asdf@yandex.ru 2004-07-23 06:26:34 0000 Yeah. :-( This bug exists in 0.50.5-r2 too. Looks like my patch can be used for 0.50.5-r2 without changes and solve this issue again. I've reported this bug to aspell developers long time ago, but looks like they don't really do something with this issue. You can track my report here: https://sourceforge.net/tracker/?func=detail&atid=300245&aid=941238&group_id=245 arj@gentoo.org 2004-07-30 14:09:14 0000 Very good detective work. I've checked your patch and the source code and it does indeed look correct. Although it looks like from the source that they didn't plan for UTF-8 support at all :-) Anyway the patch has been included in aspell-0.50 29946 2004-04-24 03:16 0000 patch... :) aspell-0.50.3-quotechar.patch text/plain ZGlmZiAtdU5yIC4vbW9kdWxlcy9maWx0ZXIvZW1haWwuY3BwIC4vbW9kdWxlcy9maWx0ZXIvZW1h aWwuY3BwCi0tLSAuL21vZHVsZXMvZmlsdGVyL2VtYWlsLmNwcAkyMDA0LTA0LTI0IDEyOjM5OjU2 LjM3MTIxNjgwMCArMDMwMAorKysgLi9tb2R1bGVzL2ZpbHRlci9lbWFpbC5jcHAJMjAwNC0wNC0y NCAxMjo0MDowNS41MjE4MjU2OTYgKzAzMDAKQEAgLTMzLDcgKzMzLDggQEAKICAgICAgICAgbWVt c2V0KGRhdGEsIDAsIHNpemVvZihib29sKSoyNTYpOwogCXJldHVybiBub19lcnI7CiAgICAgICB9 Ci0gICAgICBib29sIGhhdmUoY2hhciBjKSB7CisgICAgICBib29sIGhhdmUoRmlsdGVyQ2hhcjo6 Q2hyIGMpIHsKKwlpZiAoYyA+IDI1NSkgcmV0dXJuIGZhbHNlOwogICAgICAgICByZXR1cm4gZGF0 YVtzdGF0aWNfY2FzdDx1bnNpZ25lZCBjaGFyPihjKV07CiAgICAgICB9CiAgICAgICBRdW90ZUNo YXJzKCkge2NsZWFyKCk7fQo=