On Jul 28, 2011, at 10:19 AM, Ben Pfaff wrote:

> /* More convenience macros, for testing whether a given level is enabled in
>  * THIS_MODULE.  When constructing a log message is expensive, this enables it
>  * to be skipped. */
> -#define VLOG_IS_EMER_ENABLED() true
> #define VLOG_IS_ERR_ENABLED() vlog_is_enabled(THIS_MODULE, VLL_ERR)

Do you think not having a VLOG_IS_EMER_ENABLED is useful?  I guess those 
shouldn't be printed in much volume.

Otherwise, looks good to me.

--Justin



_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to