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 <mich...@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
signature.asc
Description: PGP signature