don't merge loc's (source maps doesn't support it anyway)

css · Feb 5, 2017 · a42433c · a42433c
1 parent 6920da5
commit a42433c
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 22 deletions.
diff --git a/lib/compressor/restructure/2-mergeAtrule.js b/lib/compressor/restructure/2-mergeAtrule.js
@@ -20,11 +20,13 @@ function processAtrule(node, item, list) {
         prev.expression &&
         node.expression.id === prev.expression.id) {
         prev.block.children.appendList(node.block.children);
-        prev.loc = {
-            primary: prev.loc,
-            merged: node.loc
-        };
         list.remove(item);
+
+        // TODO: use it when we can refer to several points in source
+        // prev.loc = {
+        //     primary: prev.loc,
+        //     merged: node.loc
+        // };
     }
 }
 

diff --git a/lib/compressor/restructure/4-restructShorthand.js b/lib/compressor/restructure/4-restructShorthand.js
@@ -233,12 +233,16 @@ TRBL.prototype.add = function(name, declaration) {
         return false;
     }
 
-    if (this.loc) {
-        this.loc = {
-            primary: this.loc,
-            merged: declaration.loc
-        };
-    } else {
+    // TODO: use it when we can refer to several points in source
+    // if (this.loc) {
+    //     this.loc = {
+    //         primary: this.loc,
+    //         merged: declaration.loc
+    //     };
+    // } else {
+    //     this.loc = declaration.loc;
+    // }
+    if (!this.loc) {
         this.loc = declaration.loc;
     }
 

diff --git a/lib/compressor/restructure/6-restructBlock.js b/lib/compressor/restructure/6-restructBlock.js
@@ -214,26 +214,32 @@ function processRule(rule, item, list, props, fingerprints) {
                 };
 
                 prev.block.remove(prev.item);
-                declaration.loc = {
-                    primary: declaration.loc,
-                    merged: prev.item.data.loc
-                };
+
+                // TODO: use it when we can refer to several points in source
+                // declaration.loc = {
+                //     primary: declaration.loc,
+                //     merged: prev.item.data.loc
+                // };
             } else {
                 declarations.remove(declarationItem);
-                prev.item.data.loc = {
-                    primary: prev.item.data.loc,
-                    merged: declaration.loc
-                };
+
+                // TODO: use it when we can refer to several points in source
+                // prev.item.data.loc = {
+                //     primary: prev.item.data.loc,
+                //     merged: declaration.loc
+                // };
             }
         } else {
             var prev = needless(props, declaration, fingerprints);
 
             if (prev) {
                 declarations.remove(declarationItem);
-                prev.item.data.loc = {
-                    primary: prev.item.data.loc,
-                    merged: declaration.loc
-                };
+
+                // TODO: use it when we can refer to several points in source
+                // prev.item.data.loc = {
+                //     primary: prev.item.data.loc,
+                //     merged: declaration.loc
+                // };
             } else {
                 declaration.fingerprint = fingerprint;