On 30.11.2017 14:41, Eric Blake wrote: > For a couple of macros in pcnet.c, we have to provide a new scope > to avoid compiler warnings about declarations in the middle of a > switch statement that aren't in a sub-scope. But use of > 'do { ... } while (0);' merely to provide that new scope is arcane > overkill, compared to just using '{ ... }'. > > Signed-off-by: Eric Blake <ebl...@redhat.com> > --- > hw/net/pcnet.c | 20 ++++++++++---------- > 1 file changed, 10 insertions(+), 10 deletions(-)
Reviewed-by: Thomas Huth <th...@redhat.com>