Re: [PATCH] mbrtowc: remove a redundant condition

2021-03-22 Thread Bruno Haible
Hi Benno, > By staring at the code and trying to understand whether I am missing > something, why that c == 0xf4 is there. I see. For consistency with the other occurrences of this idiom lib/unistr/u8-check.c:66: && (c < 0xf4 || (c == 0xf4 && s[1] < 0x90))) lib/unistr/u8-mblen.

Re: [PATCH] mbrtowc: remove a redundant condition

2021-03-22 Thread Benno Schulenberg
Op 21-03-2021 om 19:22 schreef Bruno Haible: > Hi Benno, > >> * lib/mbrtowc-impl-utf8.h: There is no need to check for c == 0xf4 >> when !(c < 0xf4), as ten lines earlier c <= 0xf4 was established. > > Your patch is correct. But we generally don't need to do micro- > optimizations that a compile

Re: [PATCH] mbrtowc: remove a redundant condition

2021-03-21 Thread Bruno Haible
Hi Benno, > * lib/mbrtowc-impl-utf8.h: There is no need to check for c == 0xf4 > when !(c < 0xf4), as ten lines earlier c <= 0xf4 was established. Your patch is correct. But we generally don't need to do micro- optimizations that a compiler easily can do for us. Just curious: Did you get a compil

[PATCH] mbrtowc: remove a redundant condition

2021-03-21 Thread Benno Schulenberg
* lib/mbrtowc-impl-utf8.h: There is no need to check for c == 0xf4 when !(c < 0xf4), as ten lines earlier c <= 0xf4 was established. --- ChangeLog | 6 ++ lib/mbrtowc-impl-utf8.h | 2 +- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 76e