Changed how arguments are parsed.

worldveil / dejavu

Audio fingerprinting and recognition in Python

MIT License

6.36k stars 1.43k forks source link

Changed how arguments are parsed. #60

Closed tjj5036 closed 9 years ago

worldveil commented 9 years ago

Is this purely a cosmetic PR for sanity of code cleanliness, or does this add anything new to the command line parsing? I don't want this to break anyone's scripts if it doesn't have to.

tjj5036 commented 9 years ago

Purely cosmetic. This would break existing scripts actually.

worldveil commented 9 years ago

By "purely cosmetic", I was asking if the code changes had the same output when utilized on the command like, but were more nicely written in the code.

If you can add a section to the README.md detailing how the script is used with a few examples, I'd probably merge this request -- better to have a well documented version that people will actually use.

tuxdna commented 9 years ago

Please check following pull request:

https://github.com/worldveil/dejavu/pull/64

I have merged code from this PR as well.