From 4b3e2ccca4e08ae41f23f9f7dd31305ffd7105de Mon Sep 17 00:00:00 2001 From: Matthew Dapena-Tretter Date: Thu, 9 Mar 2017 15:14:06 -0800 Subject: [PATCH] Remove the buildTextEditor binding The comment was actually no longer accurate since arrow functions aren't newable. After doing an audit, @maxbrunsfeld determined that it could just be removed. --- src/workspace.js | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/workspace.js b/src/workspace.js index f411cf823..8ff8aa51d 100644 --- a/src/workspace.js +++ b/src/workspace.js @@ -57,12 +57,6 @@ module.exports = class Workspace extends Model { this.defaultDirectorySearcher = new DefaultDirectorySearcher() this.consumeServices(this.packageManager) - // One cannot simply .bind here since it could be used as a component with - // Etch, in which case it'd be `new`d. And when it's `new`d, `this` is always - // the newly created object. - const realThis = this - this.buildTextEditor = (params) => Workspace.prototype.buildTextEditor.call(realThis, params) - this.panelContainers = { top: new PanelContainer({location: 'top'}), left: new PanelContainer({location: 'left'}),