On 07/18/2012 05:52 AM, Daniel P. Berrange wrote: > From: "Daniel P. Berrange" <berrange@xxxxxxxxxx> > > Nearly every source file does something like > > #define VIR_FROM_THIS VIR_FROM_FOO > #define virFooReportErorr(code, ...) \ > virReportErrorHelper(VIR_FROM_THIS, code, __FILE__, \ > __FUNCTION__, __LINE__, \ > __VA_ARGS__) > > This creates needless duplication and inconsistent error > reporting function names in each file. It is trivial to > just have virterror_internal.h provide a virReportError > macro that is equivalent > > * src/util/virterror_internal.h: Define virReportError(code, ...) > > Signed-off-by: Daniel P. Berrange <berrange@xxxxxxxxxx> > --- > cfg.mk | 1 + > src/util/virterror_internal.h | 3 +++ > 2 files changed, 4 insertions(+) ACK. The real trick will be using it in all the followup patches :) -- Eric Blake eblake@xxxxxxxxxx +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list