From 285c3fefc59f9e3645ef538368a70a2c666c62f5 Mon Sep 17 00:00:00 2001
From: Rodrigo Fernandes <rtfrodrigo@gmail.com>
Date: Fri, 20 May 2016 23:17:03 +0100
Subject: [PATCH 1/2] Improve rename paths

* Identify smallest different sub path of the changed files paths and
highlight the changed part
---
 sample/index.html           | 12 +++++--
 src/printer-utils.js        | 65 ++++++++++++++++++++++++++++++++++---
 test/printer-utils-tests.js | 29 ++++++++++++++---
 3 files changed, 95 insertions(+), 11 deletions(-)

diff --git a/sample/index.html b/sample/index.html
index 2fd1e0d5..7befc75a 100644
--- a/sample/index.html
+++ b/sample/index.html
@@ -9,12 +9,12 @@
       Author: rtfpessoa
   -->
 
-  <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/9.3.0/styles/github.min.css">
+  <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/9.4.0/styles/github.min.css">
 
   <script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.3/jquery.js"></script>
 
-  <script src="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/9.3.0/highlight.min.js"></script>
-  <script src="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/9.3.0/languages/scala.min.js"></script>
+  <script src="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/9.4.0/highlight.min.js"></script>
+  <script src="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/9.4.0/languages/scala.min.js"></script>
 
   <!-- diff2html -->
   <link rel="stylesheet" type="text/css" href="../dist/diff2html.css">
@@ -24,6 +24,12 @@
 
   <script>
     var lineDiffExample =
+      "--- a/src/my/really/big/path/sample.js\n" +
+      "+++ b/src/my/small/path/sample.js\n" +
+      "@@ -1 +1,2 @@\n" +
+      "-test\n" +
+      "+test1r\n" +
+      "+test2r\n" +
       'diff --git a/src/core/init.java b/src/core/init.java\n' +
       'index e49196a..50f310c 100644\n' +
       '--- a/src/core/init.java\n' +
diff --git a/src/printer-utils.js b/src/printer-utils.js
index 6783cccf..d8683741 100644
--- a/src/printer-utils.js
+++ b/src/printer-utils.js
@@ -11,6 +11,8 @@
   var utils = require('./utils.js').Utils;
   var Rematch = require('./rematch.js').Rematch;
 
+  var separator = '/';
+
   function PrinterUtils() {
   }
 
@@ -32,18 +34,65 @@
   };
 
   PrinterUtils.prototype.getDiffName = function(file) {
-    var oldFilename = file.oldName;
-    var newFilename = file.newName;
+    var oldFilename = unifyPath(file.oldName);
+    var newFilename = unifyPath(file.newName);
 
     if (oldFilename && newFilename && oldFilename !== newFilename && !isDevNullName(oldFilename) && !isDevNullName(newFilename)) {
-      return oldFilename + ' -> ' + newFilename;
+      var prefixPaths = [];
+      var suffixPaths = [];
+
+      var oldFilenameParts = oldFilename.split(separator);
+      var newFilenameParts = newFilename.split(separator);
+
+      var oldFilenamePartsSize = oldFilenameParts.length;
+      var newFilenamePartsSize = newFilenameParts.length;
+
+      var i = 0;
+      var j = oldFilenamePartsSize - 1;
+      var k = newFilenamePartsSize - 1;
+
+      while (i < j && i < k) {
+        if (oldFilenameParts[i] === newFilenameParts[i]) {
+          prefixPaths.push(newFilenameParts[i]);
+          i = i + 1;
+        } else {
+          break;
+        }
+      }
+
+      while (j > i && k > i) {
+        if (oldFilenameParts[j] === newFilenameParts[k]) {
+          suffixPaths.unshift(newFilenameParts[k]);
+          j = j - 1;
+          k = k - 1;
+        } else {
+          break;
+        }
+      }
+
+      var finalPrefix = prefixPaths.join(separator);
+      var finalSuffix = suffixPaths.join(separator);
+
+      var oldRemainingPath = oldFilenameParts.slice(i, j + 1).join(separator);
+      var newRemainingPath = newFilenameParts.slice(i, k + 1).join(separator);
+
+      if (finalPrefix.length && finalSuffix.length) {
+        return finalPrefix + separator + '{' + oldRemainingPath + ' → ' + newRemainingPath + '}' + separator + finalSuffix;
+      } else if (finalPrefix.length) {
+        return finalPrefix + separator + '{' + oldRemainingPath + ' → ' + newRemainingPath + '}';
+      } else if (finalSuffix.length) {
+        return '{' + oldRemainingPath + ' → ' + newRemainingPath + '}' + separator + finalSuffix;
+      } else {
+        return oldFilename + ' → ' + newFilename;
+      }
+
     } else if (newFilename && !isDevNullName(newFilename)) {
       return newFilename;
     } else if (oldFilename) {
       return oldFilename;
     }
 
-    return 'Unknown filename';
+    return 'unknown/file/path';
   };
 
   PrinterUtils.prototype.diffHighlight = function(diffLine1, diffLine2, config) {
@@ -128,6 +177,14 @@
     };
   };
 
+  function unifyPath(path) {
+    if (path) {
+      return path.replace('\\', '/');
+    } else {
+      return path;
+    }
+  }
+
   function isDevNullName(name) {
     return name.indexOf('dev/null') !== -1;
   }
diff --git a/test/printer-utils-tests.js b/test/printer-utils-tests.js
index 4357b215..4d603b9d 100644
--- a/test/printer-utils-tests.js
+++ b/test/printer-utils-tests.js
@@ -28,12 +28,33 @@ describe('Utils', function() {
       });
       assert.equal('sample.js', result);
     });
-    it('should generate the file name for a changed file and renamed', function() {
+    it('should generate the file name for a changed file and full rename', function() {
       var result = PrinterUtils.getDiffName({
         oldName: 'sample1.js',
         newName: 'sample2.js'
       });
-      assert.equal('sample1.js -> sample2.js', result);
+      assert.equal('sample1.js → sample2.js', result);
+    });
+    it('should generate the file name for a changed file and prefix rename', function() {
+      var result = PrinterUtils.getDiffName({
+        oldName: 'src/path/sample.js',
+        newName: 'source/path/sample.js'
+      });
+      assert.equal('{src → source}/path/sample.js', result);
+    });
+    it('should generate the file name for a changed file and suffix rename', function() {
+      var result = PrinterUtils.getDiffName({
+        oldName: 'src/path/sample1.js',
+        newName: 'src/path/sample2.js'
+      });
+      assert.equal('src/path/{sample1.js → sample2.js}', result);
+    });
+    it('should generate the file name for a changed file and middle rename', function() {
+      var result = PrinterUtils.getDiffName({
+        oldName: 'src/really/big/path/sample.js',
+        newName: 'src/small/path/sample.js'
+      });
+      assert.equal('src/{really/big → small}/path/sample.js', result);
     });
     it('should generate the file name for a deleted file', function() {
       var result = PrinterUtils.getDiffName({
@@ -49,9 +70,9 @@ describe('Utils', function() {
       });
       assert.equal('src/my/file.js', result);
     });
-    it('should generate handle undefined filenames', function() {
+    it('should generate handle undefined filename', function() {
       var result = PrinterUtils.getDiffName({});
-      assert.equal('Unknown filename', result);
+      assert.equal('unknown/file/path', result);
     });
   });
 

From ce909771637e80bae336dbf2a708ef531fd9d06b Mon Sep 17 00:00:00 2001
From: Rodrigo Fernandes <rtfrodrigo@gmail.com>
Date: Fri, 20 May 2016 23:25:10 +0100
Subject: [PATCH 2/2] Use operator assignement and remove redundant else

---
 src/printer-utils.js | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/printer-utils.js b/src/printer-utils.js
index d8683741..adf879e3 100644
--- a/src/printer-utils.js
+++ b/src/printer-utils.js
@@ -54,7 +54,7 @@
       while (i < j && i < k) {
         if (oldFilenameParts[i] === newFilenameParts[i]) {
           prefixPaths.push(newFilenameParts[i]);
-          i = i + 1;
+          i += 1;
         } else {
           break;
         }
@@ -63,8 +63,8 @@
       while (j > i && k > i) {
         if (oldFilenameParts[j] === newFilenameParts[k]) {
           suffixPaths.unshift(newFilenameParts[k]);
-          j = j - 1;
-          k = k - 1;
+          j -= 1;
+          k -= 1;
         } else {
           break;
         }
@@ -82,10 +82,10 @@
         return finalPrefix + separator + '{' + oldRemainingPath + ' → ' + newRemainingPath + '}';
       } else if (finalSuffix.length) {
         return '{' + oldRemainingPath + ' → ' + newRemainingPath + '}' + separator + finalSuffix;
-      } else {
-        return oldFilename + ' → ' + newFilename;
       }
 
+      return oldFilename + ' → ' + newFilename;
+
     } else if (newFilename && !isDevNullName(newFilename)) {
       return newFilename;
     } else if (oldFilename) {
@@ -180,9 +180,9 @@
   function unifyPath(path) {
     if (path) {
       return path.replace('\\', '/');
-    } else {
-      return path;
     }
+
+    return path;
   }
 
   function isDevNullName(name) {