Re: [dev] [ST] [PATCH] Changed type for UTF-32 codepoints from long to uint_least32_t

From: Silvan Jegen <s.jegen_AT_gmail.com>
Date: Tue, 5 May 2015 21:37:11 +0200

On Tue, May 05, 2015 at 07:37:01PM +0100, Connor Lane Smith wrote:
> On 5 May 2015 at 19:31, suigin <suigin_AT_national.shitposting.agency> wrote:
> > Hi all, here's a patch that changes occurences of long to uint_least32_t
> > where it's being used to store UTF-32 codepoints, as was previously
> > discussed. Other cases where long is used are preserved as is.
>
> I get the feeling we should typedef uint_least32_t Rune, as in libutf,
> so we don't have to have this long-winded and somewhat cryptic type
> everywhere.

I think this is a good idea. It would save some bytes and would make
changing the type in the future (if needed) easier.
Received on Tue May 05 2015 - 21:37:11 CEST