Re: yy_state_t

[Akim Demaille]

Hi Paul,

I'd like your input on this.  Cheers!

> Le 15 oct. 2019 à 08:45, Akim Demaille <address@hidden> a écrit :
> 
>> Le 13 oct. 2019 à 09:54, Akim Demaille <address@hidden> a écrit :
>> 
>> Here's what it gives.   Cheers!
>> 
>> commit 3c9656ec6ea39c4f1bad4ab2c153267f61015438
>> Author: Akim Demaille <address@hidden>
>> Date:   Sat Oct 12 11:00:05 2019 +0200
>> 
>>   yacc: rename types for states
>> 
>>   * data/skeletons/yacc.c (yy_state_num): Rename as...
>>   (yy_state_t): this.
>>   (yy_state_fast_t): New.
>>   Use it.
> 
> Installed.
> 
> I'm ambivalent for glr.c and lalr1.cc, since they already use int everywhere 
> for states, and are unlikely to benefit from the width reduction.  Except for 
> the auxiliary state stack for LAC in lalr1.cc.
> 
> I do like the fact that the skeletons are more consistent between them (I 
> will rename the types for states afterwards), but still...  I'd appreciate 
> other opinions.