Hi Michael, Michael Schroeder wrote: > On Thu, Feb 11, 2021 at 11:39:09PM +0100, Axel Beckert wrote: > > The only thing it didn't fix so far is > > https://savannah.gnu.org/bugs/?31336 aka > > https://bugs.debian.org/600246 although I really had some hope that > > this might be fixed as a side-effect of your patch. :-) > > Oh, that happens because the U+3099 combining char is also matched > in utf8_isdouble(). The code in ansi.c does not expect this. Thus > this is another fallout from that commit. ;) > > A simple fix (other than removing the entries from the isdouble > table) is to move the curr->w_mbcs = 0xff setting after the > combining character handling:
Can confirm! This fixes the above mentioned bug without reopening the regression (https://bugs.debian.org/677512) which the initially proposed patch in Debian had. Thanks once again! Regards, Axel -- ,''`. | Axel Beckert <a...@debian.org>, https://people.debian.org/~abe/ : :' : | Debian Developer, ftp.ch.debian.org Admin `. `' | 4096R: 2517 B724 C5F6 CA99 5329 6E61 2FF9 CD59 6126 16B5 `- | 1024D: F067 EA27 26B9 C3FC 1486 202E C09E 1D89 9593 0EDE
signature.asc
Description: PGP signature