Skip to content

Commit 43a0a9c

Browse files
author
Rodrigo Fernandes
committed
Merge pull request #12 from rtfpessoa/fix-diff-parsing
diff parser rules matching beginning of line
2 parents fca82f2 + d64f8a2 commit 43a0a9c

File tree

5 files changed

+45
-33
lines changed

5 files changed

+45
-33
lines changed

bower.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "diff2html",
3-
"version": "0.2.3",
3+
"version": "0.2.4",
44
"homepage": "https://github.com/rtfpessoa/diff2html",
55
"authors": [
66
"Rodrigo Fernandes <rtfrodrigo@gmail.com>"

dist/diff2html.js

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -729,26 +729,26 @@ function require() {
729729
}
730730

731731
/* Diff */
732-
var oldMode = /old mode (\d{6})/;
733-
var newMode = /new mode (\d{6})/;
734-
var deletedFileMode = /deleted file mode (\d{6})/;
735-
var newFileMode = /new file mode (\d{6})/;
732+
var oldMode = /^old mode (\d{6})/;
733+
var newMode = /^new mode (\d{6})/;
734+
var deletedFileMode = /^deleted file mode (\d{6})/;
735+
var newFileMode = /^new file mode (\d{6})/;
736736

737-
var copyFrom = /copy from (.+)/;
738-
var copyTo = /copy to (.+)/;
737+
var copyFrom = /^copy from (.+)/;
738+
var copyTo = /^copy to (.+)/;
739739

740-
var renameFrom = /rename from (.+)/;
741-
var renameTo = /rename to (.+)/;
740+
var renameFrom = /^rename from (.+)/;
741+
var renameTo = /^rename to (.+)/;
742742

743-
var similarityIndex = /similarity index (\d+)%/;
744-
var dissimilarityIndex = /dissimilarity index (\d+)%/;
745-
var index = /index ([0-9a-z]+)..([0-9a-z]+) (\d{6})?/;
743+
var similarityIndex = /^similarity index (\d+)%/;
744+
var dissimilarityIndex = /^dissimilarity index (\d+)%/;
745+
var index = /^index ([0-9a-z]+)..([0-9a-z]+) (\d{6})?/;
746746

747747
/* Combined Diff */
748-
var combinedIndex = /index ([0-9a-z]+),([0-9a-z]+)..([0-9a-z]+)/;
749-
var combinedMode = /mode (\d{6}),(\d{6})..(\d{6})/;
750-
var combinedNewFile = /new file mode (\d{6})/;
751-
var combinedDeletedFile = /deleted file mode (\d{6}),(\d{6})/;
748+
var combinedIndex = /^index ([0-9a-z]+),([0-9a-z]+)..([0-9a-z]+)/;
749+
var combinedMode = /^mode (\d{6}),(\d{6})..(\d{6})/;
750+
var combinedNewFile = /^new file mode (\d{6})/;
751+
var combinedDeletedFile = /^deleted file mode (\d{6}),(\d{6})/;
752752

753753
var values = [];
754754
if (utils.startsWith(line, "diff")) {

dist/diff2html.min.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

sample/index.html

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,19 @@
203203
'+ <div data-bind="template: {nodes: ko.utils.parseHtmlFragment(htmlSrc())}"></div>\n' +
204204
'+ <!-- /ko -->\n' +
205205
' <div class="btn-load-more" data-bind="visible: loadMoreCount() > 0">\n' +
206-
' </div>\n';
206+
' </div>\n' +
207+
'diff --git a/test.js b/test.js\n' +
208+
'new file mode 100644\n' +
209+
'index 0000000..e1e22ec\n' +
210+
'--- /dev/null\n' +
211+
'+++ b/test.js\n' +
212+
'@@ -0,0 +1,6 @@\n' +
213+
"+var parser = require('./source/git-parser');\n" +
214+
'+\n' +
215+
"+var text = 'diff --git a/components/app/app.html b/components/app/app.html\\nindex ecb7a95..027bd9b 100644\\n--- a/components/app/app.html\\n+++ b/components/app/app.html\\n@@ -52,0 +53,3 @@\\n+\\n+\\n+\\n@@ -56,0 +60,3 @@\\n+\\n+\\n+\\n'\n" +
216+
'+var patchLineList = [ false, false, false, false ];\n' +
217+
'+\n' +
218+
'+console.log(parser.parsePatchDiffResult(text, patchLineList));\n';
207219

208220
$(document).ready(function () {
209221
var diff2Html = Diff2Html;

src/diff-parser.js

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -126,26 +126,26 @@
126126
}
127127

128128
/* Diff */
129-
var oldMode = /old mode (\d{6})/;
130-
var newMode = /new mode (\d{6})/;
131-
var deletedFileMode = /deleted file mode (\d{6})/;
132-
var newFileMode = /new file mode (\d{6})/;
129+
var oldMode = /^old mode (\d{6})/;
130+
var newMode = /^new mode (\d{6})/;
131+
var deletedFileMode = /^deleted file mode (\d{6})/;
132+
var newFileMode = /^new file mode (\d{6})/;
133133

134-
var copyFrom = /copy from (.+)/;
135-
var copyTo = /copy to (.+)/;
134+
var copyFrom = /^copy from (.+)/;
135+
var copyTo = /^copy to (.+)/;
136136

137-
var renameFrom = /rename from (.+)/;
138-
var renameTo = /rename to (.+)/;
137+
var renameFrom = /^rename from (.+)/;
138+
var renameTo = /^rename to (.+)/;
139139

140-
var similarityIndex = /similarity index (\d+)%/;
141-
var dissimilarityIndex = /dissimilarity index (\d+)%/;
142-
var index = /index ([0-9a-z]+)..([0-9a-z]+) (\d{6})?/;
140+
var similarityIndex = /^similarity index (\d+)%/;
141+
var dissimilarityIndex = /^dissimilarity index (\d+)%/;
142+
var index = /^index ([0-9a-z]+)..([0-9a-z]+) (\d{6})?/;
143143

144144
/* Combined Diff */
145-
var combinedIndex = /index ([0-9a-z]+),([0-9a-z]+)..([0-9a-z]+)/;
146-
var combinedMode = /mode (\d{6}),(\d{6})..(\d{6})/;
147-
var combinedNewFile = /new file mode (\d{6})/;
148-
var combinedDeletedFile = /deleted file mode (\d{6}),(\d{6})/;
145+
var combinedIndex = /^index ([0-9a-z]+),([0-9a-z]+)..([0-9a-z]+)/;
146+
var combinedMode = /^mode (\d{6}),(\d{6})..(\d{6})/;
147+
var combinedNewFile = /^new file mode (\d{6})/;
148+
var combinedDeletedFile = /^deleted file mode (\d{6}),(\d{6})/;
149149

150150
var values = [];
151151
if (utils.startsWith(line, "diff")) {

0 commit comments

Comments
 (0)