OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-1205-g70624fd
Gerrit Code Review
gerrit@openafs.org
Mon, 23 May 2011 19:54:24 -0700 (PDT)
The following commit has been merged in the master branch:
commit 70624fd2ab0e580b7a34e0e9f3a83fc0d59b53c4
Author: Andrew Deason <adeason@sinenomine.net>
Date: Mon May 23 17:11:28 2011 -0500
cmd: Fix parsing positional args
If the first parameter of a libcmd syntax is a flag, cmd_Parse was
skipping over positional arguments, since j will be 0 at this point
(the j variable is only used if we're processing an explicit switch).
Effectively revert this area to what it was before
a2f1ca5fd52ac2fb7e68b101bbe3da9878c10474 so such positional parameters
work again.
Also move the j variable to inside the only block in which it is used,
to try and avoid such mistakes in the future.
Change-Id: Ifa52cecf50a3f561c70de351a9ebd026e90eeeab
Reviewed-on: http://gerrit.openafs.org/4711
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
src/cmd/cmd.c | 15 +++++++++++----
1 files changed, 11 insertions(+), 4 deletions(-)
--
OpenAFS Master Repository