From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Shawn O. Pearce" Subject: Re: [JGIT PATCH 4/5] Define Patch to parse a sequence of patch FileHeaders Date: Thu, 11 Dec 2008 12:27:01 -0800 Message-ID: <20081211202701.GI32487@spearce.org> References: <1228971522-28764-1-git-send-email-spearce@spearce.org> <200812111934.13218.robin.rosenberg.lists@dewire.com> <20081211183954.GH32487@spearce.org> <200812112123.49659.robin.rosenberg.lists@dewire.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: git@vger.kernel.org To: Robin Rosenberg X-From: git-owner@vger.kernel.org Thu Dec 11 21:28:29 2008 Return-path: Envelope-to: gcvg-git-2@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by lo.gmane.org with esmtp (Exim 4.50) id 1LAs90-0005l3-Tr for gcvg-git-2@gmane.org; Thu, 11 Dec 2008 21:28:23 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756751AbYLKU1F (ORCPT ); Thu, 11 Dec 2008 15:27:05 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755735AbYLKU1E (ORCPT ); Thu, 11 Dec 2008 15:27:04 -0500 Received: from george.spearce.org ([209.20.77.23]:41507 "EHLO george.spearce.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755450AbYLKU1D (ORCPT ); Thu, 11 Dec 2008 15:27:03 -0500 Received: by george.spearce.org (Postfix, from userid 1001) id B81F038200; Thu, 11 Dec 2008 20:27:01 +0000 (UTC) Content-Disposition: inline In-Reply-To: <200812112123.49659.robin.rosenberg.lists@dewire.com> User-Agent: Mutt/1.5.17+20080114 (2008-01-14) Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Robin Rosenberg wrote: > torsdag 11 december 2008 19:39:54 skrev Shawn O. Pearce: > > Robin Rosenberg wrote: > > > > + assertEquals(572, fRepositoryConfigTest.startOffset); > > > > + assertEquals(1490, fRepositoryConfig.startOffset); > > > > > > 1487 here > > > > Really? 1490 is the only value that the test vector passes with. > > What's the 3 bytes you think I'm off by? > > Ah, --whitespace=fix did that. Ok. I know you like to apply with --whitespace=fix, but I would prefer to leave these *.patch test input files[*1*] exactly as they were created by git format-patch or git diff, so I'm sure we are parsing the same thing git would have produced and sent to us. *1*: I have more patches coming which add 2 more test inputs to the same PatchTest suite. -- Shawn.