Re: [PATCH 5/5] Migrate git-am.sh to use git-rev-parse --parseopt
On Sat, Nov 03, 2007 at 11:54:45AM +0000, Pierre Habouzit wrote:
Show 24 quoted lines
> On Sat, Nov 03, 2007 at 09:55:56AM +0000, Alex Riesen wrote:
> > Pierre Habouzit, Fri, Nov 02, 2007 23:39:52 +0100:
> > > diff --git a/git-am.sh b/git-am.sh
> > > index 2514d07..e5ed6a7 100755
> > > --- a/git-am.sh
> > > +++ b/git-am.sh
> > ....
> > > - usage ;;
> > > - *)
> > > - break ;;
> > > + -i|--interactive)
> > > + interactive=t ;;
> > > + -b|--binary)
> > > + binary=t ;;
> >
> > Did you really have to change the indentation?
>
> Well, I'm unsure what the standard is for git, I gladly use any
> indentation, I don't really care. I assumed that it wasn't indented
> before becauuse instead of -i|--interactive you had:
>
> -i|--in|--int|....|--interactive which took a lot of space, as it
> seemed to me that the case ".." in foo) esac construction in git had the
> cases indented in most places. But I may be wrong.
Okay seems I was wrong, sorry then, that wasn't done on purpose at
all.
--
·O· Pierre Habouzit
··O madcoder@debian.org
OOO http://www.madism.org