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

Remove deprecated file system APIs #9622

Merged
merged 3 commits into from
Sep 22, 2015
Merged

Remove deprecated file system APIs #9622

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
9c7386d
Remove deprecated file system APIs
Oct 21, 2014
c4e924e
Minor optimization
Oct 21, 2014
8c92756
Merge remote-tracking branch 'upstream/master' into remove-old-filesys
Sep 22, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 2 additions & 7 deletions src/brackets.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ define(function (require, exports, module) {
require("thirdparty/path-utils/path-utils.min");
require("thirdparty/smart-auto-complete-local/jquery.smart_autocomplete");

// Load CodeMirror add-ons--these attach themselves to the CodeMirror module
// Load CodeMirror add-ons--these attach themselves to the CodeMirror module
require("thirdparty/CodeMirror2/addon/fold/xml-fold");
require("thirdparty/CodeMirror2/addon/edit/matchtags");
require("thirdparty/CodeMirror2/addon/edit/matchbrackets");
Expand Down Expand Up @@ -135,11 +135,6 @@ define(function (require, exports, module) {
require("extensibility/ExtensionManagerDialog");
require("editor/ImageViewer");

// Compatibility shims for filesystem API migration
require("project/FileIndexManager");
require("file/NativeFileSystem");
require("file/NativeFileError");

// Compatibility shim for PanelManager to WorkspaceManager migration
require("view/PanelManager");

Expand Down Expand Up @@ -314,7 +309,7 @@ define(function (require, exports, module) {
});

// Check for updates
if (!params.get("skipUpdateCheck") && !brackets.inBrowser) {
if (!brackets.inBrowser && !params.get("skipUpdateCheck")) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was a part of the auto-install extensions PR, but was reverted during review. Since I was editing this file, I thought I would go ahead and restore it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fwiw, there's basically zero chance that this will make any performance difference. But it's not a big deal to change either way.

AppInit.appReady(function () {
// launches periodic checks for updates cca every 24 hours
UpdateNotification.launchAutomaticUpdate();
Expand Down
53 changes: 0 additions & 53 deletions src/file/NativeFileError.js
View file
Open in desktop

This file was deleted.

284 changes: 0 additions & 284 deletions src/file/NativeFileSystem.js
View file
Open in desktop

This file was deleted.

Loading