Skip to content

Commit e9e3d92

Browse files
committed
Only log failed hunk header parse if line starts with @@
1 parent 4bea8bb commit e9e3d92

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

src/diff-parser.js

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,11 @@
3232
var oldLine2 = null; // Used for combined diff
3333
var newLine = null;
3434

35+
/* Diff Header */
36+
var oldFileNameHeader = '--- ';
37+
var newFileNameHeader = '+++ ';
38+
var hunkHeaderPrefix = '@@';
39+
3540
/* Add previous block(if exists) before start a new file */
3641
var saveBlock = function() {
3742
if (currentBlock) {
@@ -91,7 +96,10 @@
9196
oldLine2 = values[2];
9297
newLine = values[3];
9398
} else {
94-
console.error('Failed to parse lines, starting in 0!');
99+
if (utils.startsWith(line, hunkHeaderPrefix)) {
100+
console.error('Failed to parse lines, starting in 0!');
101+
}
102+
95103
oldLine = 0;
96104
newLine = 0;
97105
currentFile.isCombined = false;
@@ -144,11 +152,6 @@
144152
.replace(/\r\n?/g, '\n')
145153
.split('\n');
146154

147-
/* Diff Header */
148-
var oldFileNameHeader = '--- ';
149-
var newFileNameHeader = '+++ ';
150-
var hunkHeaderPrefix = '@@';
151-
152155
/* Diff */
153156
var oldMode = /^old mode (\d{6})/;
154157
var newMode = /^new mode (\d{6})/;

0 commit comments

Comments
 (0)