On 05/11/2015 01:24 AM, Markus Armbruster wrote: > Report to stderr, prefix with the program name. Also reject > extra arguments. > > Signed-off-by: Markus Armbruster <arm...@redhat.com> > Reviewed-by: Eric Blake <ebl...@redhat.com> > --- > scripts/qapi.py | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) >
> @@ -999,4 +999,8 @@ def parse_command_line(extra_options = "", > extra_long_options = []): > do_c = True > do_h = True > > + if len(args) != 0: > + print >>sys.stderr, "%s: too many arguments"% sys.argv[0] Spacing on both sides of %? Or even: print >>sys.stderr, sys.argv[0] + ": too many arguments" but trivial enough to keep R-b whatever you decide. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
