oh, i get it. makes sense


On Wed, Feb 26, 2014 at 10:58 AM, Miguel Manso <notifications@github.com>wrote:

> In service_container.js:
>
> > +    var self = this;
> > +    return Object.keys(this.defers)
> > +        .map(
> > +            function(key){
> > +                var state = self.defers[key].promise.inspect().state;
> > +                var reason = self.defers[key].promise.inspect().reason || "";
> > +                return ([key, state === 'rejected' ? state + " because " + reason : state]);
> > +            }
> > +        )
> > +        .reduce(
> > +            function(obj, pair){
> > +                obj[pair[0].toString()]=pair[1];
> > +                return obj;
> > +            },
> > +            {}
> > +        );
>
> Nop.
> I mean like this:
>
> ...
>         .reduce(
>             function(obj, pair){
>                 obj[pair[0].toString()]=pair[1];
>                 return obj;
>             },
>             {}
>         )
>     ;
> };
>
> You always close the sections on the same indentation column from the
> start of the instruction.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/uniplaces/async-container/pull/1/files#r10075563>
> .
>