visionmedia/jade — Issue #459: Superfluous object null prevention in jade.js
Author: jcolebrand
Created: 2011-12-31 08:33:16+00:00
State: open

https://github.com/visionmedia/jade/blob/master/lib/jade.js#L148

, parse(String(str), options || {})

the trailing `|| {}` is superfluous, because of the variable declaration above:

var options = options || {}

and in the interest of having solid code for referring others too, I suggest removing that. I'm not making it a pull request because I think this is minor and can be done just at some point in the future, and a bug ticket is easier (in my mind) than pull requests for ya'll to manage (I know, a huge assumption on my part)