I'll do it the sans var way. It's more in line with the rest of the code.

On 24 January 2013 12:45, Rod Vagg <notifications@github.com> wrote:

> In lib/levelup.js:
>
> > -  this._options  = extend(extend({}, defaultOptions), options)
> > -  this._location = location
> > -  this._status   = 'new'
> > -}
> > -
> > -inherits(LevelUP, EventEmitter)
> > +var createLevelUP = function (location, options, callback) {
> > +  // Possible status values:
> > +  //  - 'new'     - newly created, not opened or closed
> > +  //  - 'opening' - waiting for the database to be opened, post open()
> > +  //  - 'open'    - successfully opened the database, available for use
> > +  //  - 'closing' - waiting for the database to be closed, post close()
> > +  //  - 'closed'  - database has been successfully closed, should not be used
> > +  //                except for another open() operation
> > +  var status = 'new'
> > +  , message
>
> well, you have a couple of choices, you could just declare it as a
> standard function, sans-var, and have indentation at a minimum, or follow
> the prevailing style and indent even further (obviously I'm cool with lots
> of indenting).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rvagg/node-levelup/pull/62/files#r2756948>.
>
>