From 96f434f27650b9c65c009a41183b08909cf6dd4f Mon Sep 17 00:00:00 2001 From: Ben Ogle & Nathan Sobo Date: Wed, 17 Jul 2013 14:59:46 -0700 Subject: [PATCH] Set languageMode and softTabs in constructor, not setBuffer The softTabs option is only in scope in the constructor, and it can sometimes be needed if we're unable to auto-determine the softTabs setting from the buffer. --- src/app/edit-session.coffee | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/app/edit-session.coffee b/src/app/edit-session.coffee index 75c36ed57..4fb137ba9 100644 --- a/src/app/edit-session.coffee +++ b/src/app/edit-session.coffee @@ -57,6 +57,9 @@ class EditSession @buildDisplayBuffer({tabLength}) @addCursorAtScreenPosition([0, 0]) + @languageMode = new LanguageMode(this, @buffer.getExtension()) + @softTabs = @buffer.usesSoftTabs() ? softTabs ? true + @state.on 'changed', ({key, newValue}) => switch key when 'scrollTop' @@ -66,8 +69,6 @@ class EditSession setBuffer: (@buffer) -> @buffer.retain() - @languageMode = new LanguageMode(this, @buffer.getExtension()) - @softTabs = @buffer.usesSoftTabs() ? softTabs ? true @subscribe @buffer, "path-changed", => project.setPath(path.dirname(@getPath())) unless project.getPath()? @trigger "title-changed"