bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH V2] localcharset: short-circuit the search for an alias on a


From: Bruno Haible
Subject: Re: [PATCH V2] localcharset: short-circuit the search for an alias on a Mac
Date: Sat, 21 Apr 2018 12:41:57 +0200
User-agent: KMail/5.1.3 (Linux/4.4.0-119-generic; KDE/5.18.0; x86_64; ; )

Hi Benno,

> * lib/localcharset.c (get_charset_aliases): Add a tautological
> UTF-8 entry to speed up the search for this case.
> 
> Most machines default to a UTF-8 locale nowadays, so begin the
> list of aliases with a dummy UTF-8 entry so it will be found
> immediately and a time-consuming search through the rest of
> the list is avoided.

Thanks, applied.

I added a comment.
Explanations about the code belong in the code, as comments, IMO.
Readers of the code should not be forced to constantly refer to
ChangeLog entries and commit messages in order to understand the code.

Bruno




reply via email to

[Prev in Thread] Current Thread [Next in Thread]