From: Jakub Pawlowicz <contact@jakubpawlowicz.com>
Date: Thu, 15 Dec 2016 16:05:04 +0000 (+0100)
Subject: Moves all reading-related modules to `reader` folder.
X-Git-Url: https://git.ndcode.org/public/gitweb.cgi?a=commitdiff_plain;h=7e839e08a6d57c9191514f89be37b6c0e6968b1f;p=clean-css.git

Moves all reading-related modules to `reader` folder.

Why:

* Grouping it together for clarity sake.
---

diff --git a/lib/clean.js b/lib/clean.js
index 7d634d06..9c6ac071 100644
--- a/lib/clean.js
+++ b/lib/clean.js
@@ -13,8 +13,8 @@ var Validator = require('./properties/validator');
 var override = require('./utils/override');
 var DEFAULT_TIMEOUT = 5000;
 
-var inputSourceMapTracker = require('./utils/input-source-map-tracker');
-var readSources = require('./utils/read-sources');
+var inputSourceMapTracker = require('./reader/input-source-map-tracker');
+var readSources = require('./reader/read-sources');
 var basicOptimize = require('./optimizer/basic');
 var advancedOptimize = require('./optimizer/advanced');
 var simpleStringify = require('./stringifier/simple');
diff --git a/lib/utils/apply-source-maps.js b/lib/reader/apply-source-maps.js
similarity index 97%
rename from lib/utils/apply-source-maps.js
rename to lib/reader/apply-source-maps.js
index e565d893..c65f371b 100644
--- a/lib/utils/apply-source-maps.js
+++ b/lib/reader/apply-source-maps.js
@@ -2,14 +2,14 @@ var fs = require('fs');
 var path = require('path');
 
 var isAllowedResource = require('./is-allowed-resource');
-var isDataUriResource = require('./is-data-uri-resource');
-var isRemoteResource = require('./is-remote-resource');
 var loadRemoteResource = require('./load-remote-resource');
 var matchDataUri = require('./match-data-uri');
 var rebaseLocalMap = require('./rebase-local-map');
 var rebaseRemoteMap = require('./rebase-remote-map');
 
 var Token = require('../tokenizer/token');
+var isDataUriResource = require('../utils/is-data-uri-resource');
+var isRemoteResource = require('../utils/is-remote-resource');
 
 var MAP_MARKER_PATTERN = /^\/\*# sourceMappingURL=(\S+) \*\/$/;
 
@@ -107,7 +107,7 @@ function extractInputSourceMapFrom(sourceMapComment, applyContext, whenSourceMap
       }
     });
   } else {
-    // at this point `uri` is already rebased, see read-sources.js#rebaseSourceMapComment
+    // at this point `uri` is already rebased, see lib/reader/rebase.js#rebaseSourceMapComment
     // it is rebased to be consistent with rebasing other URIs
     // however here we need to resolve it back to read it from disk
     rebasedToCurrentPath = path.resolve(applyContext.rebaseTo, uri);
diff --git a/lib/utils/extract-import-url-and-media.js b/lib/reader/extract-import-url-and-media.js
similarity index 100%
rename from lib/utils/extract-import-url-and-media.js
rename to lib/reader/extract-import-url-and-media.js
diff --git a/lib/utils/input-source-map-tracker.js b/lib/reader/input-source-map-tracker.js
similarity index 100%
rename from lib/utils/input-source-map-tracker.js
rename to lib/reader/input-source-map-tracker.js
diff --git a/lib/utils/is-allowed-resource.js b/lib/reader/is-allowed-resource.js
similarity index 94%
rename from lib/utils/is-allowed-resource.js
rename to lib/reader/is-allowed-resource.js
index d2797192..e7d0b06a 100644
--- a/lib/utils/is-allowed-resource.js
+++ b/lib/reader/is-allowed-resource.js
@@ -1,6 +1,6 @@
 var url = require('url');
 
-var hasProtocol = require('./has-protocol');
+var hasProtocol = require('../utils/has-protocol');
 
 var HTTP_PROTOCOL = 'http:';
 
diff --git a/lib/utils/load-original-sources.js b/lib/reader/load-original-sources.js
similarity index 98%
rename from lib/utils/load-original-sources.js
rename to lib/reader/load-original-sources.js
index ba661998..dba1e54b 100644
--- a/lib/utils/load-original-sources.js
+++ b/lib/reader/load-original-sources.js
@@ -2,9 +2,10 @@ var fs = require('fs');
 var path = require('path');
 
 var isAllowedResource = require('./is-allowed-resource');
-var isRemoteResource = require('./is-remote-resource');
 var loadRemoteResource = require('./load-remote-resource');
 
+var isRemoteResource = require('../utils/is-remote-resource');
+
 function loadOriginalSources(context, callback) {
   var loadContext = {
     callback: callback,
diff --git a/lib/utils/load-remote-resource.js b/lib/reader/load-remote-resource.js
similarity index 88%
rename from lib/utils/load-remote-resource.js
rename to lib/reader/load-remote-resource.js
index 97605749..371a20a7 100644
--- a/lib/utils/load-remote-resource.js
+++ b/lib/reader/load-remote-resource.js
@@ -2,10 +2,10 @@ var http = require('http');
 var https = require('https');
 var url = require('url');
 
-var hasProtocol = require('./has-protocol');
-var isHttpResource = require('./is-http-resource');
-var isHttpsResource = require('./is-https-resource');
-var override = require('./override');
+var hasProtocol = require('../utils/has-protocol');
+var isHttpResource = require('../utils/is-http-resource');
+var isHttpsResource = require('../utils/is-https-resource');
+var override = require('../utils/override');
 
 var HTTP_PROTOCOL = 'http:';
 
diff --git a/lib/utils/match-data-uri.js b/lib/reader/match-data-uri.js
similarity index 100%
rename from lib/utils/match-data-uri.js
rename to lib/reader/match-data-uri.js
diff --git a/lib/utils/read-sources.js b/lib/reader/read-sources.js
similarity index 98%
rename from lib/utils/read-sources.js
rename to lib/reader/read-sources.js
index 10c4f9dd..05310792 100644
--- a/lib/utils/read-sources.js
+++ b/lib/reader/read-sources.js
@@ -3,10 +3,7 @@ var path = require('path');
 
 var applySourceMaps = require('./apply-source-maps');
 var extractImportUrlAndMedia = require('./extract-import-url-and-media');
-var isAbsoluteResource = require('./is-absolute-resource');
 var isAllowedResource = require('./is-allowed-resource');
-var isImport = require('./is-import');
-var isRemoteResource = require('./is-remote-resource');
 var loadOriginalSources = require('./load-original-sources');
 var loadRemoteResource = require('./load-remote-resource');
 var rebase = require('./rebase');
@@ -16,6 +13,9 @@ var restoreImport = require('./restore-import');
 
 var tokenize = require('../tokenizer/tokenize');
 var Token = require('../tokenizer/token');
+var isAbsoluteResource = require('../utils/is-absolute-resource');
+var isImport = require('../utils/is-import');
+var isRemoteResource = require('../utils/is-remote-resource');
 
 var UNKNOWN_SOURCE = 'unknown-source';
 
diff --git a/lib/utils/rebase-local-map.js b/lib/reader/rebase-local-map.js
similarity index 100%
rename from lib/utils/rebase-local-map.js
rename to lib/reader/rebase-local-map.js
diff --git a/lib/utils/rebase-remote-map.js b/lib/reader/rebase-remote-map.js
similarity index 100%
rename from lib/utils/rebase-remote-map.js
rename to lib/reader/rebase-remote-map.js
diff --git a/lib/utils/rebase.js b/lib/reader/rebase.js
similarity index 96%
rename from lib/utils/rebase.js
rename to lib/reader/rebase.js
index 78487970..3f4bdd34 100644
--- a/lib/utils/rebase.js
+++ b/lib/reader/rebase.js
@@ -1,9 +1,9 @@
 var extractImportUrlAndMedia = require('./extract-import-url-and-media');
-var isImport = require('./is-import');
 var restoreImport = require('./restore-import');
+var rewriteUrl = require('./rewrite-url');
 
-var rewriteUrl = require('../urls/rewrite');
 var Token = require('../tokenizer/token');
+var isImport = require('../utils/is-import');
 
 var SOURCE_MAP_COMMENT_PATTERN = /^\/\*# sourceMappingURL=(\S+) \*\/$/;
 
diff --git a/lib/utils/restore-import.js b/lib/reader/restore-import.js
similarity index 100%
rename from lib/utils/restore-import.js
rename to lib/reader/restore-import.js
diff --git a/lib/urls/rewrite.js b/lib/reader/rewrite-url.js
similarity index 100%
rename from lib/urls/rewrite.js
rename to lib/reader/rewrite-url.js
diff --git a/test/optimizer/extract-properties-test.js b/test/optimizer/extract-properties-test.js
index 75fa125f..c07c9401 100644
--- a/test/optimizer/extract-properties-test.js
+++ b/test/optimizer/extract-properties-test.js
@@ -1,7 +1,7 @@
 var vows = require('vows');
 var assert = require('assert');
 var tokenize = require('../../lib/tokenizer/tokenize');
-var inputSourceMapTracker = require('../../lib/utils/input-source-map-tracker');
+var inputSourceMapTracker = require('../../lib/reader/input-source-map-tracker');
 var extractProperties = require('../../lib/optimizer/extract-properties');
 
 function _tokenize(source) {
diff --git a/test/optimizer/reorderable-test.js b/test/optimizer/reorderable-test.js
index ff5ad9bf..8b7a8e5d 100644
--- a/test/optimizer/reorderable-test.js
+++ b/test/optimizer/reorderable-test.js
@@ -2,7 +2,7 @@ var vows = require('vows');
 var assert = require('assert');
 
 var tokenize = require('../../lib/tokenizer/tokenize');
-var inputSourceMapTracker = require('../../lib/utils/input-source-map-tracker');
+var inputSourceMapTracker = require('../../lib/reader/input-source-map-tracker');
 var extractProperties = require('../../lib/optimizer/extract-properties');
 var canReorder = require('../../lib/optimizer/reorderable').canReorder;
 var canReorderSingle = require('../../lib/optimizer/reorderable').canReorderSingle;
diff --git a/test/properties/longhand-overriding-test.js b/test/properties/longhand-overriding-test.js
index a13ada4d..b0bb3466 100644
--- a/test/properties/longhand-overriding-test.js
+++ b/test/properties/longhand-overriding-test.js
@@ -4,7 +4,7 @@ var assert = require('assert');
 var optimize = require('../../lib/properties/optimizer');
 
 var tokenize = require('../../lib/tokenizer/tokenize');
-var inputSourceMapTracker = require('../../lib/utils/input-source-map-tracker');
+var inputSourceMapTracker = require('../../lib/reader/input-source-map-tracker');
 var compatibility = require('../../lib/utils/compatibility');
 var Validator = require('../../lib/properties/validator');
 
diff --git a/test/properties/optimizer-test.js b/test/properties/optimizer-test.js
index 05425a75..d8bc5a1e 100644
--- a/test/properties/optimizer-test.js
+++ b/test/properties/optimizer-test.js
@@ -4,7 +4,7 @@ var assert = require('assert');
 var optimize = require('../../lib/properties/optimizer');
 
 var tokenize = require('../../lib/tokenizer/tokenize');
-var inputSourceMapTracker = require('../../lib/utils/input-source-map-tracker');
+var inputSourceMapTracker = require('../../lib/reader/input-source-map-tracker');
 var compatibility = require('../../lib/utils/compatibility');
 var Validator = require('../../lib/properties/validator');
 
diff --git a/test/properties/override-compacting-test.js b/test/properties/override-compacting-test.js
index a43c7905..4cb428a4 100644
--- a/test/properties/override-compacting-test.js
+++ b/test/properties/override-compacting-test.js
@@ -4,7 +4,7 @@ var assert = require('assert');
 var optimize = require('../../lib/properties/optimizer');
 
 var tokenize = require('../../lib/tokenizer/tokenize');
-var inputSourceMapTracker = require('../../lib/utils/input-source-map-tracker');
+var inputSourceMapTracker = require('../../lib/reader/input-source-map-tracker');
 var compatibility = require('../../lib/utils/compatibility');
 var Validator = require('../../lib/properties/validator');
 
diff --git a/test/properties/shorthand-compacting-test.js b/test/properties/shorthand-compacting-test.js
index 9980303c..e7af8114 100644
--- a/test/properties/shorthand-compacting-test.js
+++ b/test/properties/shorthand-compacting-test.js
@@ -4,7 +4,7 @@ var assert = require('assert');
 var optimize = require('../../lib/properties/optimizer');
 
 var tokenize = require('../../lib/tokenizer/tokenize');
-var inputSourceMapTracker = require('../../lib/utils/input-source-map-tracker');
+var inputSourceMapTracker = require('../../lib/reader/input-source-map-tracker');
 var compatibility = require('../../lib/utils/compatibility');
 var Validator = require('../../lib/properties/validator');
 
diff --git a/test/tokenizer/tokenize-test.js b/test/tokenizer/tokenize-test.js
index 379a4645..0f726c2d 100644
--- a/test/tokenizer/tokenize-test.js
+++ b/test/tokenizer/tokenize-test.js
@@ -1,7 +1,7 @@
 var vows = require('vows');
 var assert = require('assert');
 var tokenize = require('../../lib/tokenizer/tokenize');
-var inputSourceMapTracker = require('../../lib/utils/input-source-map-tracker');
+var inputSourceMapTracker = require('../../lib/reader/input-source-map-tracker');
 
 var fs = require('fs');
 var path = require('path');