[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: yy_state_t
From: |
Akim Demaille |
Subject: |
Re: yy_state_t |
Date: |
Mon, 21 Oct 2019 11:27:07 +0200 |
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.
- yy_state_t, Akim Demaille, 2019/10/12
- Re: yy_state_t, Paul Eggert, 2019/10/12
- Re: yy_state_t, Akim Demaille, 2019/10/13
- Re: yy_state_t, Akim Demaille, 2019/10/15
- Re: yy_state_t,
Akim Demaille <=
- Re: yy_state_t, Paul Eggert, 2019/10/23
- Re: yy_state_t, Akim Demaille, 2019/10/24
- Re: yy_state_t, Akim Demaille, 2019/10/25