From: Junio C Hamano Date: Tue, 22 Jan 2013 04:10:50 GMT Subject: Re: [PATCH 1/2] Update :/abc ambiguity check Message-ID: <7vlibldfhx.fsf@alter.siamese.dyndns.org> In-Reply-To: Duy Nguyen writes: >>> ... take ":/abc" as rev even it's ambiguous. This patch makes it: >>> >>> - ambiguous when "abc" exists on worktree >>> - a rev if abc does not exist on worktree >>> - a path if abc is not found in any commits (although better use >> >> The "any commits" above sounds very scary. Are you really going to >> check against all the commits? > > If I remember correctly :/ will search through commit chains until it > finds a commit that matches. So :/non-existent-string definitely > searches through all commits. That is the real work the user asked us to do, so it is not a wasted latency. The description looked as if you were doing extra work only for disambiguation, which triggered my "Huh?" meter.