From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bert Wesarg Subject: Re: [PATCH 1/1] git-gui: fix hunk parsing for corner case changes Date: Sat, 14 Jan 2012 06:25:28 +0100 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: git@vger.kernel.org, Bert Wesarg To: Pat Thoyts X-From: git-owner@vger.kernel.org Sat Jan 14 06:25:36 2012 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 1Rlw7T-0008JU-5d for gcvg-git-2@lo.gmane.org; Sat, 14 Jan 2012 06:25:35 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750817Ab2ANFZa convert rfc822-to-quoted-printable (ORCPT ); Sat, 14 Jan 2012 00:25:30 -0500 Received: from mail-wi0-f174.google.com ([209.85.212.174]:44956 "EHLO mail-wi0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750707Ab2ANFZ3 convert rfc822-to-8bit (ORCPT ); Sat, 14 Jan 2012 00:25:29 -0500 Received: by wibhm14 with SMTP id hm14so866436wib.19 for ; Fri, 13 Jan 2012 21:25:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlemail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; bh=lQTjOLhaVH7LFrJy291faOEh6Q4fz7IpJwfrumafIWw=; b=WUX2WtttaMgSgI3V+cc31pe9uOIRXa96fq0nGNW6SW7ZEhqXJ2NCb8P2eOQNRGzxl3 BBhSytGMKEB4+KGcZkIv3I15+m6f7HZz8VaPwQMXS+KbODHbWMfs9z525+wi048/jG9E r3QXjBv1VvvcGHV7xhVX2DJZe/AGFHGiKUw6I= Received: by 10.180.93.168 with SMTP id cv8mr1340500wib.2.1326518728105; Fri, 13 Jan 2012 21:25:28 -0800 (PST) Received: by 10.223.118.10 with HTTP; Fri, 13 Jan 2012 21:25:28 -0800 (PST) In-Reply-To: Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Hi Pat, On Mon, Jan 9, 2012 at 14:43, Bert Wesarg = wrote: > The simple hunk parsing code did not recognize hunks when there is no > second number after the comma. Like in these cases: > > =C2=A0@@ -1 +0,0 @@ > =C2=A0-1 > > Which resulted in this hunk header: > > =C2=A0@@ -1 +0,1 +1 +0,0 @@ > > Or: > > =C2=A0@@ -1 +1 @@ > =C2=A0-1 > =C2=A0+2 > > Resulted in: > > =C2=A0@@ -1 +1 @@ > =C2=A0,1 +1 +1 @@ > =C2=A0,0 @@ > > While trying to stage only the '-1' line. > could you please consider pushing this into the 1.7.9 release. I see no point in waiting for the next release. Thanks. Bert