On Mon, Sep 14, 2020 at 03:28:17PM +0200, Jasper Lievisse Adriaanse wrote:
> Hi,
>
> This diff adds support for the '&' and '|' operators, along with
> a new testcase.
>
> OK?
make sens to me. ok semarie@
> Index: usr.sbin/btrace/bt_parse.y
> ===
On Mon, Sep 14, 2020 at 03:39:04PM +0200, Otto Moerbeek wrote:
> On Mon, Sep 14, 2020 at 03:28:17PM +0200, Jasper Lievisse Adriaanse wrote:
>
> > Hi,
> >
> > This diff adds support for the '&' and '|' operators, along with
> > a new testcase.
> >
> > OK?
>
> The precedence looks funny
>
> I'd
On Mon, Sep 14, 2020 at 03:28:17PM +0200, Jasper Lievisse Adriaanse wrote:
> Hi,
>
> This diff adds support for the '&' and '|' operators, along with
> a new testcase.
>
> OK?
The precedence looks funny
I'd guess you want
%left '|'
%left '&'
%left '+' '-'
%left '/' '*'
To avoid suprises.
Hi,
This diff adds support for the '&' and '|' operators, along with
a new testcase.
OK?
Index: usr.sbin/btrace/bt_parse.y
===
RCS file: /cvs/src/usr.sbin/btrace/bt_parse.y,v
retrieving revision 1.16
diff -u -p -r1.16 bt_parse.y
---