Skip to content

Commit 9d7ce7a

Browse files
committed
clean up comments
1 parent dc6d7cd commit 9d7ce7a

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

ly2video/cli.py

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -958,17 +958,11 @@ def parseOptions():
958958

959959
options = parser.parse_args()
960960

961-
# TODO: use ConfigArgParse maybe?
962-
# Look for a config file and override options if they exist.
963-
# this is not a good way to do it, we need to populate options
964-
# from file and then parse the command line options which will
965-
# override the file options. For now let's just see if it works.
961+
# Check for options in config file. Don't apply them if cli arguments
962+
# were supplied.
966963
config = ConfigParser()
967-
968964
if config.read('ly2video.cfg'):
969965
section = config['External programs']
970-
# We can check to see if the option variable is empty ""
971-
972966
options.lilypond = options.lilypond or section.get('lilypond')
973967
options.ffmpeg = options.ffmpeg or section.get('ffmpeg')
974968
options.timidity = options.timidity or section.get('timidity')

0 commit comments

Comments
 (0)