[patch] Move method into TEST macros and check for verbose in _dbus_verbose_bytes

John (J5) Palmieri johnp at redhat.com
Fri Sep 8 13:47:25 PDT 2006


On Fri, 2006-09-08 at 16:21 -0400, Havoc Pennington wrote:
> John (J5) Palmieri wrote:
> > -  if (!verbose)
> > +  if (!_dbus_is_verbose_real())
> 
> Maybe leave this accessing the variable directly, just so this function 
> is as fast as possible with verbose disabled? (this is dbus_verbose_real 
> itself)

That is there because it calls _dbus_verbose_init() (I should inline
that).  I could go back to calling _dbus_verbose_init and then checking
verbose for speed.

> Patch looks fine.
> 
> Havoc
> 
-- 



More information about the dbus mailing list