Closed bburky closed 10 years ago
thank you for your contribution @bburky ! shouldn't the variables from line 10 to 15 also be included in main() ?
These lines? Probably doesn't matter. I wasn't sure if they were only used by the main code. Yeah, you could move those too.
I think it may be nice to move prototypes
and hashcatModes
to the top level though. Theoretically a program integrating this could override them and add to them or filter them down.
You've got a good point there with the prototypes
and hashcatModes
modes, I'm currently not home (I'm on my mobile right now) so I cant change it myself, but if you create a pull request from the latest version on github I'll happily merge it in.
For the variable lines, yes they are only used by the main code now since argparse moved to main() so I guess I could move them to main() too.
Move argparse configuration and input processing into a
main()
function.Example Usage: