On Feb 5, 2014, at 12:25 PM, Matt Sergeant <notifications@github.com> wrote:

> In plugins/note.js:
> 
> > +
> > +    note.human = r.join(',  ');
> > +    note.human_html = r.join(',  &#10;'); // #10 = newline within HTML title
> > +    return r.join(',  ');
> > +};
> > +
> > +exports.note_init = function (obj) {
> > +    if (!validate_obj(obj)) throw("invalid obj!")
> > +    if (!obj.plugin) throw "plugin is required during init!";
> > +    var conn = obj.conn;
> > +    var pi   = obj.plugin || this;
> > +    var name = get_note_name(pi);
> > +    var note = obj.txn ? conn.transaction.notes[name] : conn.notes[name];
> > +    if (note && note !== undefined) return; // init once per connection
> > +
> > +    var obj = {
> 
> Did you really mean to overwrite with the same variable name as in the function parameters? Seems confusing to me.

I didn't. And yes, it was confusing. Fixed.

Matt