Ehhh... Don't worry about it
On Feb 9, 2014 12:22 PM, "soedar" <notifications@github.com> wrote:

> In lib/leaderboard.js:
>
> > -};
> > +}
> > +exports.updateLeaderboard = updateLeaderboard;
> > +
> > +
> > +function boardParams(game, board, sortDesc, page, limit, email) {
> > +    var start = page * limit;
> > +    var end = start + limit - 1;
> > +
> > +    return {key: 'leaderboards:' + game + ':' + board,
> > +            zRangeFunc: sortDesc ? 'zrevrange' : 'zrange',
> > +            start: start,
> > +            stop: end,
> > +            email: email};
> > +}
> > +exports.boardParams = boardParams;
>
> That is true. I didn't want to do repeated calculations on the individual
> functions.
>
> Should I extend the argument list, and create a couple of private helper
> functions instead?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/cvan/galaxy-api/pull/81/files#r9570617>
> .
>