bison-patches
[Top][All Lists]
Advanced

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

Re: [PATCH for Dlang support] Push parser


From: Adela Vais
Subject: Re: [PATCH for Dlang support] Push parser
Date: Sun, 18 Apr 2021 13:39:47 +0300

Hello Akim,

I updated the 'calc' example to use token constructors.

Adela

În lun., 12 apr. 2021 la 07:46, Akim Demaille <akim@lrde.epita.fr> a scris:

> Hi Adela,
>
> > Le 2 avr. 2021 à 01:23, Adela Vais <adela.vais99@gmail.com> a écrit :
> >
> > În sâm., 27 mar. 2021 la 08:35, Akim Demaille <akim@lrde.epita.fr> a
> scris:
> >
> >> I see that you have keep the possibility for the regular pull parser
> >> to use local variables.  Have you benchmarked if this is worth the
> >> effort?  Because it there's no measurable difference, you may just
> >> as well always use attributes of the parser object.  That would simplify
> >> the skeleton.
> >>
> >
> > I benchmarked it, and there is no measurable difference. I put the
> > variables as class attributes.
>
> Good!
>
> >> I don't think we need too many examples.  So I would go with keeping
> >> a super simple one, and have the other one grow with all the bells
> >> and whistles, a bit like what the bistromathic example does in C.
> >>
> >
> > I added the push parser to the 'calc' example.
>
> Perfect!  I would suggest that this example also use the token
> constructor functions.
>
> Installed.
>
> Cheers!

Attachment: 0001-d-demonstrate-the-token-constructors.patch
Description: Binary data


reply via email to

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