2014-04-28 11:05 GMT+02:00 Michael Tokarev <m...@tls.msk.ru>: > > 28.04.2014 12:26, Marc Marí пишет: > > From: Marc Marí <5.markm...@gmail.com> > > > > Modify debug macros as explained in https://lists.gnu.org/archive/html/qemu-devel/2014-04/msg03642.html > > > > Signed-off-by: Marc Marí <5.markm...@gmail.com> > > --- > > hw/dma/i82374.c | 17 ++++++++++------- > > hw/dma/i8257.c | 24 +++++++++++++++++------- > > hw/dma/rc4030.c | 13 +++++++++---- > > 3 files changed, 36 insertions(+), 18 deletions(-) > > > > diff --git a/hw/dma/i82374.c b/hw/dma/i82374.c > > index dc7a767..fff4e6f 100644 > > --- a/hw/dma/i82374.c > > +++ b/hw/dma/i82374.c > > @@ -24,15 +24,18 @@ > > > > #include "hw/isa/isa.h" > > > > -//#define DEBUG_I82374 > > +//#define DEBUG_I82374 1 > > > > -#ifdef DEBUG_I82374 > > -#define DPRINTF(fmt, ...) \ > > -do { fprintf(stderr, "i82374: " fmt , ## __VA_ARGS__); } while (0) > > -#else > > -#define DPRINTF(fmt, ...) \ > > -do {} while (0) > > +#ifndef DEBUG_I82374 > > +#define DEBUG_I82374 0 > > #endif > > + > > +#define DPRINTF(fmt, ...) \ > > + do { \ > > + if(DEBUG_I82374) { \ > > + fprintf(stderr, "I82374: " fmt, ## __VA_ARGS__); \ > > + } \ > > + } while (0) > > Since this is the same pattern in all files touched, it can be generalized: > > qemu-common.h: > > #define QEMU_DPRINTF(cond,pfx,fmt,...) \ > do { \ > if (cond) { > fprintf(stderr, pfx # ": " fmt, ## __VA_ARGS__); \ > } \ > while(0) > > This file (and other): > > #ifndef DEBUG_I82374 > #define DEBUG_I82374 0 > #endif > #define DPRINTF(fmt, ...) QEMU_DPRINTF(DEBUG_I82374, "I82374", fmt, ## __VA_ARGS__) > > > which is just 4 lines per file. > > FWIW. > > Thanks, > > /mjt
Approximately 1/3 cannot be forced into the same pattern (do not use fprintf(stderr), mainly). I'll change the ones that are possible for v2.