peterbraden/node-opencv — pull request #141 by coolblade (2014-06-19 02:30:48+00:00)
Title: backwards compatible proposal for full opencv drawing api support

It would be nice if node-opencv supported all the parameters of the c api but there are cases, like the ellipse function, where a parameter is out of sequence (e.g. thickness would be the 7th parameter of the c api but is the 6th parameter in node-opencv. I don't want to break backwards compatibility but I also don't think it's a good idea to have parameters out of sequence (devs will generally use the c api documentation I'd imagine). I'm thinking that allowing a single options object argument which explicitly spells out the parameter names might be the best compromise because we can continue backwards support and allow full use of the c APIs going forwards. If you're ok with this then I'll continue refactoring the other methods in a similar fashion so we have full parameter support on each method.