From: Junio C Hamano Date: Fri, 09 Dec 2011 21:27:00 GMT Subject: Re: [PATCH 1/2] index_pack: indent find_unresolved_detals one level Message-ID: <7vpqfxcu6z.fsf@alter.siamese.dyndns.org> In-Reply-To: <4ee0be67.05c1e70a.1956.ffff800b@mx.google.com> pclouds@gmail.com writes: > From: Nguyễn Thái Ngọc Duy > > The next patch puts most of the code in one level deeper. By indenting > separately, it'd be easier to see the actual changes. Yuck. Isn't it a sign that "the next patch" should perhaps be helped by a small helper function that does whatever the part you are indenting here?