bison-patches
[Top][All Lists]
Advanced

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

Re: yy_state_t


From: Paul Eggert
Subject: Re: yy_state_t
Date: Sat, 12 Oct 2019 12:17:14 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0

On 10/12/19 1:58 AM, Akim Demaille wrote:
I was about to rename the types for states as

yy_state_num -> yy_small_state_t
int -> yy_state_t

But the latter is seldom used, while the former appears many times, so I'd 
prefer

yy_state_num -> yy_state_t
int -> yy_fast_state_t

WDYT?

yy_state_num values are typically narrower than int, so following Leibniz's notational principle that notation should reflect reality, the former type name should be narrower than the latter, which means your preference is better.

You might consider spelling that last name "yy_state_fast_t" for consistency with stdint.h type names. Plus, whatever name is used, I suppose it should be typedeffed to int_fastN_t (if available) instead of to plain int.



reply via email to

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