it's very much worth the effort to set up an auto-jshint check on every
file save.
saves tons of time and headache over otherwise very hard to notice issues.


On Tue, Nov 5, 2013 at 3:33 PM, ggrin <notifications@github.com> wrote:

> In src/inmemorystorage.js:
>
> > +    } else {
> > +      throw new Error('inMemoryStorage encountered invalid path : '+path)
> > +    }
> > +  }
> > +
> > +  RemoteStorage.InMemoryStorage = function(rs){
> > +    this.rs = rs
> > +    RemoteStorage.eventHandling(this, 'change', 'conflict')
> > +    this._storage = {};
> > +    this._changes = {};
> > +  }
> > +
> > +  RemoteStorage.InMemoryStorage.
> prototype = {
> > +    get: function(path){
> > +      var node = this._storage[path];
> > +      if(node)
>
> you mean for the if(node) thing? ahh ok, I remembered a discussion about
> this where we agreed upon putting ifs without braces into one line.
> that also isn't the case here. right:)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/remotestorage/remotestorage.js/pull/447/files#r7431501>
> .
>