[Webkit-unassigned] [Bug 38454] New: svn-apply: Have parseDiffHeader() call parseGitDiffHeader().
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon May 3 04:02:24 PDT 2010
https://bugs.webkit.org/show_bug.cgi?id=38454
Summary: svn-apply: Have parseDiffHeader() call
parseGitDiffHeader().
Product: WebKit
Version: 528+ (Nightly build)
Platform: All
OS/Version: All
Status: NEW
Severity: Normal
Priority: P2
Component: Tools / Tests
AssignedTo: webkit-unassigned at lists.webkit.org
ReportedBy: cjerdonek at webkit.org
CC: dbates at webkit.org
In this patch, I think it would make sense to recast the existing
parseDiffHeader() as parseSvnDiffHeader() and then create a new
parseDiffHeader() that calls the Git or SVN version based on the first line
(whether it begins with "Index:" or "diff --git"). The unit tests for the new
parseDiffHeader() can have enough tests just to ensure that it supports
carrying over each diffHash key value correctly (since the more thorough tests
will be in the tests for the Git and SVN versions).
This patch can also delete the gitdiff2svndiff() subroutine.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list