> I think the API should be clear and easy to understand, because people will 
> probably bind them to custom shortcuts. You should not have to think about 
> what that weird extra parameter (-1, false, ...) means and go read the docs.

I agree. One might replace the parameter with a string like "prev" and "next".

> But I agree that the code duplication is not optimal. The code in Ians repo 
> already has split out the ruby code. There is still code duplication. But 
> that can easily be split out into a function on its own. 

That's the way to go.

Bartosz

Reply via email to