On Mon, Sep 25, 2023 at 07:40:00PM +0200, Matthias van de Meent wrote:
> On Wed, 20 Sept 2023 at 07:06, Michael Paquier <michael@paquier.xyz> wrote:
>> #define COPY_HEADER_FIELD(_dst, _size) \
>> do { \
>> - if (remaining < _size) \
>> + if (remaining < (_size)) \
>> goto shortdata_err; \
>>
>> There are a couple of stylistic changes like this one, that I guess
>> could just use their own patch to make these macros easier to use.
>
> They actually fix complaints of my IDE, but are otherwise indeed stylistic.
Oh, OK. I just use an old-school terminal, but no objections in
changing these if they make life easier for some hackers. Still, that
feels independant of what you are proposing here.
--
Michael