Re: [TopGit PATCH] Check for help invocation before setup

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



also sprach Petr Baudis <pasky@xxxxxxx> [2008.11.19.1924 +0100]:
> > +args_saved="$@"
> > +while [ -n "$1" ]; do
> > +	case "$1" in
> > +	help|--help|-h)
> > +		shift
> > +		do_help "$1"
> > +		exit 1;;
> > +	esac
> > +	shift
> > +done
> > +set -- $args_saved
> > +unset args_saved
> >  
> >  ## Initial setup
> 
> Huh, why do you actually need $args_saved at all? :-) This is bound to
> do horrible things with space-containing arguments etc., I think. You
> don't need to do the outer shift and then drop $args_saved altogether,
> no?

I figured I need to restore $@ for others to consume, e.g. when you
check for -r later.

The challenge is to identify help|--help|-h anywhere on the command
line. Thus, you need to iterate, or do some weird matching against
$*.

When you iterate in a for loop, it's not easy to get at the next
argument, except if you use a state machine. I wanted to avoid that.

But you are right, I am wreaking havoc with space-containing
arguments. I will have to go back and rework this.

I take it you agree with the general principle though?

-- 
 .''`.   martin f. krafft <madduck@xxxxxxxxxx>
: :'  :  proud Debian developer, author, administrator, and user
`. `'`   http://people.debian.org/~madduck - http://debiansystem.info
  `-  Debian - when you have better things to do than fixing systems
 
"it is easier to be a lover than a husband for the simple reason
 that it is more difficult to be witty every day
 than to say pretty things from time to time."
                                                   -- honoré de balzac

Attachment: digital_signature_gpg.asc
Description: Digital signature (see http://martin-krafft.net/gpg/)


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux