From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff King Subject: Re: [PATCH] add -p: make the beginning of the hunk easier to see Date: Sat, 13 Feb 2010 06:23:43 -0500 Message-ID: <20100213112342.GA1299@coredump.intra.peff.net> References: <4B768807.3030003@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: git@vger.kernel.org, Junio C Hamano To: =?utf-8?B?QmrDtnJu?= Gustavsson X-From: git-owner@vger.kernel.org Sat Feb 13 12:23:53 2010 Return-path: Envelope-to: gcvg-git-2@lo.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1NgG6F-00042U-Db for gcvg-git-2@lo.gmane.org; Sat, 13 Feb 2010 12:23:47 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752794Ab0BMLXm convert rfc822-to-quoted-printable (ORCPT ); Sat, 13 Feb 2010 06:23:42 -0500 Received: from peff.net ([208.65.91.99]:56670 "EHLO peff.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750779Ab0BMLXl (ORCPT ); Sat, 13 Feb 2010 06:23:41 -0500 Received: (qmail 13394 invoked by uid 107); 13 Feb 2010 11:23:50 -0000 Received: from coredump.intra.peff.net (HELO coredump.intra.peff.net) (10.0.0.2) by peff.net (qpsmtpd/0.40) with (AES128-SHA encrypted) SMTP; Sat, 13 Feb 2010 06:23:50 -0500 Received: by coredump.intra.peff.net (sSMTP sendmail emulation); Sat, 13 Feb 2010 06:23:43 -0500 Content-Disposition: inline In-Reply-To: <4B768807.3030003@gmail.com> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: On Sat, Feb 13, 2010 at 12:07:51PM +0100, Bj=C3=B6rn Gustavsson wrote: > diff --git a/git-add--interactive.perl b/git-add--interactive.perl > index cd43c34..4eb7c7c 100755 > --- a/git-add--interactive.perl > +++ b/git-add--interactive.perl > @@ -1439,6 +1439,8 @@ sub patch_update_file { > !defined $hunk[$ix]{USE}); > } > } > + } continue { > + print "\n" x 4, '=3D' x 72, "\n"; > } > =20 > @hunk =3D coalesce_overlapping_hunks(@hunk); Wow, I find this terrifically ugly. I guess you don't use color.interactive, which will output a colored hunk header? Or is it that small hunks tend to confused with the previous hunk? -Peff