Re: [spice-common v3 3/6] test-region: Replace direct printf with g_debug() calls

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> On Wed, Jun 27, 2018 at 06:04:16AM -0400, Frediano Ziglio wrote:
> > > 
> > > Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> > > ---
> > >  common/region.c     |  28 ++++++------
> > >  tests/test-region.c | 106 ++++++++++++++++++++++----------------------
> > >  2 files changed, 67 insertions(+), 67 deletions(-)
> > > 
> > > diff --git a/common/region.c b/common/region.c
> > > index c950a9b..d690167 100644
> > > --- a/common/region.c
> > > +++ b/common/region.c
> > > @@ -496,29 +496,29 @@ void region_dump(const QRegion *rgn, const char
> > > *prefix)
> > >      pixman_box32_t *rects, *extents;
> > >      int n_rects, i;
> > >  
> > > -    printf("%sREGION: %p, ", prefix, rgn);
> > > +    g_debug("%sREGION: %p, ", prefix, rgn);
> > >  
> > >      if (!pixman_region32_not_empty((pixman_region32_t *)rgn)) {
> > > -        printf("EMPTY\n");
> > > +        g_debug("EMPTY");
> > >          return;
> > >      }
> > >  
> > >      extents = pixman_region32_extents((pixman_region32_t *)rgn);
> > >      rects = pixman_region32_rectangles((pixman_region32_t *)rgn,
> > >      &n_rects);
> > > -    printf("num %u bounds (%d, %d, %d, %d)\n",
> > > -           n_rects,
> > > -           extents->x1,
> > > -           extents->y1,
> > > -           extents->x2,
> > > -           extents->y2);
> > > +    g_debug("num %u bounds (%d, %d, %d, %d)",
> > > +            n_rects,
> > > +            extents->x1,
> > > +            extents->y1,
> > > +            extents->x2,
> > > +            extents->y2);
> > >  
> > >  
> > >      for (i = 0; i < n_rects; i++) {
> > > -        printf("%*s  %12d %12d %12d %12d\n",
> > > -               (int)strlen(prefix), "",
> > > -               rects[i].x1,
> > > -               rects[i].y1,
> > > -               rects[i].x2,
> > > -               rects[i].y2);
> > > +        g_debug("%*s  %12d %12d %12d %12d",
> > > +                (int)strlen(prefix), "",
> > > +                rects[i].x1,
> > > +                rects[i].y1,
> > > +                rects[i].x2,
> > > +                rects[i].y2);
> > >      }
> > >  }
> > 
> > Sorry, why are you changing this function?
> > This is not part only of the tests.
> > Can I remove this hunk?
> 
> I don't think it's that great to have code directly calling printf. I
> can remove the calls to region_dump() in the test case instead..
> 
> Christophe
> 
> 

Sounds good.

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]