From: Ramkumar Ramachandra Date: Wed, 16 Nov 2011 06:17:43 GMT Subject: Re: [PATCH 3/5] sequencer: sequencer state is useless without todo Message-ID: In-Reply-To: <7v7h31wduv.fsf@alter.siamese.dyndns.org> Hi, Jonathan Nieder wrote: > Just use a sane idiom.  Which means: as few git_path() values in > flight at a time as possible. Makes sense, thanks. Junio C Hamano wrote: > Or perhaps http://thread.gmane.org/gmane.comp.version-control.git/184963/focus=185436 I noticed that sha1_to_hex() also operates like this. The resolve_ref() function is really important, but using the same technique for these tiny functions is probably an overkill; something in `Documentation/technical` perhaps? Thanks. -- Ram