[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 00/11] qapi: untie 'if' conditions from C preprocessor
|
From: |
Markus Armbruster |
|
Subject: |
Re: [PATCH v6 00/11] qapi: untie 'if' conditions from C preprocessor |
|
Date: |
Tue, 03 Aug 2021 15:44:00 +0200 |
|
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
marcandre.lureau@redhat.com writes:
> From: Marc-André Lureau <marcandre.lureau@redhat.com>
>
> Hi,
>
> This series makes the 'if' conditions less liberal, by formalizing a simple
> expression tree based on bare boolean logic of configure option identifiers.
>
> (this allows to express conditions in Rust in my QAPI-Rust PoC series)
>
> thanks
I like this overall.
The commit messages are rather terse in places. I have a few questions,
I asked for a few minor tweaks, and I also noted a few possible
improvements that can be done on top. I wonder whether we can drop
PATCH 04.
Let's discuss my findings, then decide whether we want a respin.
Thank you!
- Re: [PATCH v6 00/11] qapi: untie 'if' conditions from C preprocessor,
Markus Armbruster <=