From d2933a9ba8a8813900e05b4051b625180b01958c Mon Sep 17 00:00:00 2001 From: Antonio Ospite Date: Mon, 28 Nov 2016 13:44:55 +0100 Subject: [PATCH 1/1] vidi-player.py: move the actual processing after the arguments validation --- vidi-player.py | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/vidi-player.py b/vidi-player.py index 303d55d..491a3f3 100755 --- a/vidi-player.py +++ b/vidi-player.py @@ -86,13 +86,6 @@ def main(): usage() return 1 - midi_file = mido.MidiFile(sys.argv[1]) - - overlapping_notes = check_overlapping_notes(midi_file) - if overlapping_notes: - sys.stderr.write("Sorry, supporting only midi file with no overlapping notes on channel 0\n") - return 1 - if not os.path.isdir(sys.argv[2]): sys.stderr.write("The second argument must be the path of the videofont directory\n") usage() @@ -102,6 +95,13 @@ def main(): sys.stderr.write("File '%s' exists, exiting!\n" % sys.argv[3]) return 1 + midi_file = mido.MidiFile(sys.argv[1]) + + overlapping_notes = check_overlapping_notes(midi_file, 0) + if overlapping_notes: + sys.stderr.write("Sorry, supporting only midi file with no overlapping notes on channel 0\n") + return 1 + video_font_path = os.path.realpath(sys.argv[2]) timeline = timeline_from_midi(midi_file, video_font_path) -- 2.1.4