[Libvirt-cim] [PATCH 1 of 4] Replace printf() with CU_DEBUG in EAFP

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Tue Jan 8 15:17:17 UTC 2008


Dan Smith wrote:
> KR> # HG changeset patch
> KR> # User Kaitlin Rupert <karupert at us.ibm.com>
> KR> # Date 1199740240 28800
> KR> # Node ID 7281142059a1c419405addd2ba351c68fb719ff0
> KR> # Parent  abcb596636c9530009756b1aa1e8900a2f01d10e
> KR> Replace printf() with CU_DEBUG in EAFP.
> 
> I'm all for cleanups of this fashion.  However,
> 
> KR> -                printf("Device %hhi:%s", type, dev_id);
> KR> +                CU_DEBUG("Device %hhi:%s", type, dev_id);
> 
> This looks like crufty debug that uses a platform-dependent bit-width
> anyway.  This and the rest in this file should probably be excised
> instead of formalized.  What do you think?

This is a good point. I'll generate a new patch.

-- 
Kaitlin Rupert
IBM Linux Technology Center
karupert at us.ibm.com




More information about the Libvirt-cim mailing list