Refactor routine to check for ASCII-only case
Hi all,
As mentioned in [1]/messages/by-id/X9lVLGRuG0hTHrVo@paquier.xyz -- Michael, there are three places where there is the same
routine to check if a string is made only of ASCII characters.
This makes for a small-ish but nice cleanup, as per the attached.
Thanks,
[1]: /messages/by-id/X9lVLGRuG0hTHrVo@paquier.xyz -- Michael
--
Michael
Attachments:
ascii-refactor-v1.patchtext/x-diff; charset=us-asciiDownload+26-52
On 18/12/2020 05:57, Michael Paquier wrote:
As mentioned in [1], there are three places where there is the same
routine to check if a string is made only of ASCII characters.This makes for a small-ish but nice cleanup, as per the attached.
+1
- Heikki
Greetings,
* Heikki Linnakangas (hlinnaka@iki.fi) wrote:
On 18/12/2020 05:57, Michael Paquier wrote:
As mentioned in [1], there are three places where there is the same
routine to check if a string is made only of ASCII characters.This makes for a small-ish but nice cleanup, as per the attached.
+1
Yeah, in a quick look, this looks like a good improvement.
Thanks,
Stephen