Closed yacoob closed 8 years ago
I've started fixing play.py
, and I see that there's some duplicated code there. I'll add this to this PR.
OK, good for review.
Allright, here's a different take on this, using https://github.com/bw2/ConfigArgParse. The downsides:
configargparse
that something should be just a flag, or just a config file entry. You always get everything.[section]
s are ignored by configargparse
fullscreen = no
in the config file won't work, as it defaults to argparse's behavior of "flag found, setting it to True". example.cfg
has been adjusted accordingly to avoid giving an impression that it'd work.I may refactor this at some point depending on what direction the config file stuff takes, but this is good for now.
Sane defaults in case of no config file or no option in a config file. Used argparse for cmdline handling, so now you can just run
blitzloop
. Isn't future nice? :DObviously this is dependent on https://github.com/marcan/blitzloop/pull/10.