Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit 81aa896

Browse files
committed
Left over code review comments from #2844
1 parent 26401fe commit 81aa896

File tree

2 files changed

+5
-11
lines changed

2 files changed

+5
-11
lines changed

src/language/LanguageManager.js

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
* language.setLineComment("--");
4747
* language.setBlockComment("{-", "-}");
4848
*
49-
* Some CodeMirror modes define variations of themselves. The are called MIME modes.
49+
* Some CodeMirror modes define variations of themselves. They are called MIME modes.
5050
* To find out existing MIME modes, search for "CodeMirror.defineMIME" in thirdparty/CodeMirror2/mode
5151
* For instance, C++, C# and Java all use the clike mode.
5252
* You can refine the mode definition by specifying the MIME mode as well:
@@ -462,18 +462,15 @@ define(function (require, exports, module) {
462462
html._setLanguageForMode("xml", html);
463463

464464
// Currently we override the above mentioned "xml" in TokenUtils.getModeAt, instead returning "html".
465-
// When the CSSInlineEditor and the hint providers are no longer based on moded, this can be changed.
465+
// When the CSSInlineEditor and the hint providers are no longer based on modes, this can be changed.
466466
// But for now, we need to associate this madeup "html" mode with our HTML language object.
467467
_setLanguageForMode("html", html);
468468

469469
// The fallback language for unknown modes and file extensions
470470
_fallbackLanguage = getLanguage("unknown");
471471

472-
473472
// Public methods
474-
module.exports = {
475-
defineLanguage: defineLanguage,
476-
getLanguage: getLanguage,
477-
getLanguageForFileExtension: getLanguageForFileExtension
478-
};
473+
exports.defineLanguage = defineLanguage;
474+
exports.getLanguage = getLanguage;
475+
exports.getLanguageForFileExtension = getLanguageForFileExtension;
479476
});

src/utils/ExtensionLoader.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,6 @@ define(function (require, exports, module) {
4141
var _init = false,
4242
/** @type {Object<string, Object>} Stores require.js contexts of extensions */
4343
contexts = {},
44-
/** @type {Object<string, string>} Stores what entry points where used to load an extension (usually "main" to load main.js) */
45-
entryPoints = {},
4644
srcPath = FileUtils.getNativeBracketsDirectoryPath();
4745

4846
// The native directory path ends with either "test" or "src". We need "src" to
@@ -96,7 +94,6 @@ define(function (require, exports, module) {
9694
locale: brackets.getLocale()
9795
});
9896
contexts[name] = extensionRequire;
99-
entryPoints[name] = entryPoint;
10097

10198
// console.log("[Extension] starting to load " + config.baseUrl);
10299

0 commit comments

Comments
 (0)