Sitelet https://github.com/sockjs/sockjs-node/pull/175
Skip to content

Incompatible with coffeescript 1.9.0 - #175

Closed
vilasmaciel wants to merge 1 commit into
sockjs:masterfrom
vilasmaciel:master
Closed

vilasmaciel wants to merge 1 commit into
sockjs:masterfrom
vilasmaciel:master

Conversation

@vilasmaciel

Copy link
Copy Markdown

With newer versións of coffescript ( > 1.8.0 ) the code

class Listener
    constructor: (@options, emit) ->
        @app = new App()
        @app.options = options

compiles to

Listener = (function() {
    function Listener(options1, emit) {
      this.options = options1;
      this.handler = __bind(this.handler, this);
      this.app = new App();
      this.app.options = options;

returning a ReferenceError: options is not defined on the line 135 this.app.options = options;

@brycekahle brycekahle changed the title Force coffescript version to 1.8.0 Incompatible with coffeescript 1.9.0 Mar 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant