Merge pull request #8960 from le717/remove-collection-utils

Remove CollectionUtils and StringUtils.htmlEscape()
adobe · Oct 21, 2014 · 9400017 · 9400017
2 parents 451fd32 + 297e6a7
commit 9400017
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 105 deletions.
diff --git a/src/brackets.js b/src/brackets.js
@@ -135,8 +135,6 @@ define(function (require, exports, module) {
     require("extensibility/ExtensionManagerDialog");
     require("editor/ImageViewer");
 
-    // Deprecated modules loaded just so extensions can still use them for now
-    require("utils/CollectionUtils");
     // Compatibility shims for filesystem API migration
     require("project/FileIndexManager");
     require("file/NativeFileSystem");

diff --git a/src/utils/CollectionUtils.js b/src/utils/CollectionUtils.js
diff --git a/src/utils/StringUtils.js b/src/utils/StringUtils.js
@@ -51,14 +51,6 @@ define(function (require, exports, module) {
         });
     }
 
-    /**
-     * @deprecated Use lodash `escape()` instead.
-     */
-    function htmlEscape(str) {
-        console.warn("StringUtils.htmlEscape is deprecated. Use _.escape instead.");
-        return _.escape(str);
-    }
-
     function regexEscape(str) {
         return str.replace(/([.?*+\^$\[\]\\(){}|\-])/g, "\\$1");
     }
@@ -221,7 +213,6 @@ define(function (require, exports, module) {
 
     // Define public API
     exports.format              = format;
-    exports.htmlEscape          = htmlEscape;
     exports.regexEscape         = regexEscape;
     exports.jQueryIdEscape      = jQueryIdEscape;
     exports.getLines            = getLines;