Allow --simplifystate and --runtime to be specified as timecodes

Author: jonathanhogg

docs/install.md

@@ -186,8 +186,9 @@ begin again fresh.
 
 `--simplifystate` *SECONDS*
 : Sets the period after which the [language
-simplifier](language.md#simplification) will simplify on static state values.
-The default is 10 seconds. Set this to `0` to disable simplifying on state.
+simplifier](language.md#simplification) will simplify on static state values, in
+seconds (may be specified as a [time-code](language.md#time-codes)). The default
+is 10 seconds. Set this to `0` to disable simplifying on state.
 
 `--nosimplify`
 : Completely disable the language simplifier. Generally there is no reason to
@@ -204,7 +205,8 @@ output from non-interactive programs. It also has a specific [effect on how the
 physics system runs](physics.md#non-realtime-mode).
 
 `--runtime` *SECONDS*
-: Run for a specific period of time and then exit automatically. This is
+: Run for a specific period of time and then exit automatically, in seconds
+(may be specified as a [time-code](language.md#time-codes)). This is
 particularly useful for capturing videos of a specific length unattended. In
 `--lockstep` mode, this is a period of the internal frame clock not the wall
 clock.

src/flitter/engine/__main__.py

@@ -46,11 +46,11 @@ def main():
     parser.add_argument('--state', type=str, help="State save/restore file")
     parser.add_argument('--resetonswitch', action='store_true', default=False, help="Reset state when switching pages")
     parser.add_argument('--nosimplify', action='store_true', default=False, help="Disable the language simplifier")
-    parser.add_argument('--simplifystate', type=float, default=10, help="Simplify on state after stable period")
+    parser.add_argument('--simplifystate', type=convert_timecode_to_float, default=10, help="Simplify on state after stable period")
     parser.add_argument('--lockstep', action='store_true', default=False, help="Run clock in non-realtime mode")
     parser.add_argument('--define', '-D', action='append', default=[], type=keyvalue, dest='defines', help="Define name for evaluation")
     parser.add_argument('--vmstats', action='store_true', default=False, help="Report VM statistics")
-    parser.add_argument('--runtime', type=float, help="Seconds to run for before exiting")
+    parser.add_argument('--runtime', type=convert_timecode_to_float, help="Seconds to run for before exiting")
     parser.add_argument('--offscreen', action='store_true', default=False, help="Swap windows for offscreens")
     parser.add_argument('--opengles', action='store_true', default=False, help="Use OpenGL ES")
     parser.add_argument('program', nargs='+', help="Program(s) to load")

src/flitter/language/parser.py

@@ -43,10 +43,12 @@ def __init__(self, msg, line, column, context):
 
 def convert_timecode_to_float(t):
     parts = t.split(':')
-    if len(parts) not in (2, 3):
+    if not 1 <= len(parts) <= 3:
         raise ValueError("Incorrect format")
-    seconds = 60*int(parts[-2]) + float(parts[-1])
-    if len(parts) == 3:
+    seconds = float(parts[-1])
+    if len(parts) > 1:
+        seconds += 60*int(parts[-2])
+    if len(parts) > 2:
         seconds += 3600*int(parts[-3])
     return seconds