From: Tay Ray Chuan Date: Mon, 09 Nov 2009 13:56:51 GMT Subject: Re: [PATCH v2] Show usage string for 'git http-push -h' Message-ID: In-Reply-To: <20091109104732.GA15519@progeny.tock> Hi, On Mon, Nov 9, 2009 at 6:47 PM, Jonathan Nieder wrote: > (I was making sure "git http-push -h" would work without a git > directory by putting the check for -h before setup_git_directory(). > But nothing in the argv loop requires a git repository, so it is > better and simpler to move the setup_git_directory() call to after the > loop.  Thanks for the catch.) Actually, I didn't notice the removal of the "is in a git directory for -h" requirement; I was only looking at the argument mix. Thanks. Acked-by: Tay Ray Chuan -- Cheers, Ray Chuan