On 06/13/2015 08:20 AM, Markus Armbruster wrote: > Error classes other than ERROR_CLASS_GENERIC_ERROR should not be used > in new code. Hiding them in QERR_ macros makes new uses hard to spot. > Fortunately, there's just one such macro left. Eliminate it with this > coccinelle semantic patch: > > @@ > expression EP, E; > @@ > -error_set(EP, QERR_DEVICE_NOT_FOUND, E) > +error_set(EP, ERROR_CLASS_DEVICE_NOT_FOUND, "Device '%s' not found", E) > > Signed-off-by: Markus Armbruster <arm...@redhat.com> > --- > backends/rng-egd.c | 3 ++- > blockdev-nbd.c | 3 ++- > blockdev.c | 33 ++++++++++++++++++++++----------- > hmp.c | 6 ++++-- > include/qapi/qmp/qerror.h | 3 --- > net/net.c | 6 ++++-- > qdev-monitor.c | 6 ++++-- > qmp.c | 12 ++++++++---- > qom/object.c | 6 ++++-- > ui/input.c | 3 ++- > 10 files changed, 52 insertions(+), 29 deletions(-)
Plain transformation would be closer to a 1:1 insertion/deletion count. The larger insertion count is due to reflowing long lines after the transformation. Does coccinelle do that for you, or do you have to touch things up manually? But I'm okay with the result. Reviewed-by: Eric Blake <ebl...@redhat.com> -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
