virsh: Enforce proper ordering of options
virsh: Enforce proper ordering of options

Even though vshCmddefOptParse() tried returning -1 if there was an
optional option specification that preceded a required one, it failed to
check that for boolean type options and options with VSH_OFLAG_REQ_OPT
flag set.  On the other hand, it makes sense that VSH_OT_ARGV is
specified at the end of the option list.

Returning -1 enforces the proper ordering thanks to virsh-synopsis test
in 'make check'.

Signed-off-by: Martin Kletzander <e82f23e74d43760279ebcc505dac4ae70d5627d0@redhat.com>

zippy2/libvirt,crobinso/libvirt,rlaager/libvirt,crobinso/libvirt,elmarco/libvirt,taget/libvirt,eskultety/libvirt,agx/libvirt,datto/libvirt,crobinso/libvirt,rlaager/libvirt,fabianfreyer/libvirt,fabianfreyer/libvirt,shugaoye/libvirt,jfehlig/libvirt,fabianfreyer/libvirt,libvirt/libvirt,zippy2/libvirt,VenkatDatta/libvirt,olafhering/libvirt,rlaager/libvirt,jfehlig/libvirt,olafhering/libvirt,datto/libvirt,andreabolognani/libvirt,andreabolognani/libvirt,libvirt/libvirt,datto/libvirt,crobinso/libvirt,datto/libvirt,VenkatDatta/libvirt,libvirt/libvirt,shugaoye/libvirt,shugaoye/libvirt,jardasgit/libvirt,andreabolognani/libvirt,elmarco/libvirt,zippy2/libvirt,nertpinx/libvirt,eskultety/libvirt,fabianfreyer/libvirt,jfehlig/libvirt,libvirt/libvirt,agx/libvirt,taget/libvirt,shugaoye/libvirt,fabianfreyer/libvirt,elmarco/libvirt,nertpinx/libvirt,agx/libvirt,datto/libvirt,jardasgit/libvirt,jfehlig/libvirt,rlaager/libvirt,nertpinx/libvirt,eskultety/libvirt,zippy2/libvirt,VenkatDatta/libvirt,agx/libvirt,olafhering/libvirt,rlaager/libvirt,andreabolognani/libvirt,VenkatDatta/libvirt,nertpinx/libvirt,eskultety/libvirt,taget/libvirt,elmarco/libvirt,jardasgit/libvirt,shugaoye/libvirt,elmarco/libvirt,taget/libvirt,jardasgit/libvirt,jardasgit/libvirt,eskultety/libvirt,andreabolognani/libvirt,taget/libvirt,agx/libvirt,nertpinx/libvirt,VenkatDatta/libvirt,olafhering/libvirt