Minor speed improvement: only compute uniqueId (at most) once.

FWeinb · Jul 1, 2014 · 4ec1249 · 4ec1249
1 parent 600fe66
commit 4ec1249
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/tasks/svgstore.js b/tasks/svgstore.js
@@ -76,6 +76,7 @@ module.exports = function (grunt) {
         // Map to store references from id to uniqueId + id;
         // N.B.: only IDs that are referenced are mapped.
         var mappedIds = {};
+        var uniqueId;
         $('*').each(function () {
           var $elem = $(this);
           var attrs = $elem.attr();
@@ -87,7 +88,9 @@ module.exports = function (grunt) {
 
               // Add id mapping if not already present
               if (!mappedIds[refId]) {
-                var uniqueId = md5(contentStr);
+                if (!uniqueId) {
+                  uniqueId = md5(contentStr);
+                }
                 var newId = 'svgstore' + uniqueId + refId;
                 mappedIds[refId] = newId;
               }