Re: [patch 1/2] speakup: Fix hang on spelling words with non-latin1 characters

2017-03-13 Thread Samuel Thibault
Dan Carpenter, on lun. 13 mars 2017 15:46:34 +0300, wrote: > Run your patches through scripts/checkpatch.pl D'oh. For such a small patch, I didn't bother doing it indeed. Anyway, that's superseded by version 2 I have submitted (speakup: Support spelling unicode characters) Samuel

Re: [patch 1/2] speakup: Fix hang on spelling words with non-latin1 characters

2017-03-13 Thread Dan Carpenter
Run your patches through scripts/checkpatch.pl regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[patch 1/2] speakup: Fix hang on spelling words with non-latin1 characters

2017-03-11 Thread Samuel Thibault
9831013cbdbd3d06430a1db01d8c32d50c7d1c04 ('speakup: convert screen reading to 16bit characters') made spell_word() ignore non-latin1 characters now read from the VC, but it missed actually skipping them, leading to an infinite loop. This fixes it by just advancing the unicode character pointer. R