UsageConfig: write str(i) instead of "%d"%i
authorMike Looijmans <milo-software@users.sourceforge.net>
Sat, 30 May 2015 17:15:19 +0000 (19:15 +0200)
committerMike Looijmans <milo-software@users.sourceforge.net>
Sat, 30 May 2015 17:15:19 +0000 (19:15 +0200)
Shorter. Clearer. Faster.

lib/python/Components/UsageConfig.py

index c9f4138..da004ee 100644 (file)
@@ -33,7 +33,7 @@ def InitUsageConfig():
 
        choicelist = [("-1", _("Disable"))]
        for i in range(0,1300,100):
-               choicelist.append(("%d" % i, ngettext("%d pixel wide", "%d pixels wide", i) % i))
+               choicelist.append((str(i), ngettext("%d pixel wide", "%d pixels wide", i) % i))
        config.usage.servicelist_column = ConfigSelection(default="-1", choices=choicelist)
        config.usage.servicelist_column.addNotifier(refreshServiceList)
 
@@ -51,7 +51,7 @@ def InitUsageConfig():
        config.usage.e1like_radio_mode = ConfigYesNo(default = True)
        choicelist = [("0", _("No timeout"))]
        for i in range(1, 12):
-               choicelist.append(("%d" % i, ngettext("%d second", "%d seconds", i) % i))
+               choicelist.append((str(i), ngettext("%d second", "%d seconds", i) % i))
        config.usage.infobar_timeout = ConfigSelection(default = "5", choices = choicelist)
        config.usage.show_infobar_on_zap = ConfigYesNo(default = True)
        config.usage.show_infobar_on_skip = ConfigYesNo(default = True)
@@ -68,13 +68,13 @@ def InitUsageConfig():
        config.usage.enable_tt_caching = ConfigYesNo(default = True)
        choicelist = []
        for i in (10, 30):
-               choicelist.append(("%d" % i, ngettext("%d second", "%d seconds", i) % i))
+               choicelist.append((str(i), ngettext("%d second", "%d seconds", i) % i))
        for i in (60, 120, 300, 600, 1200, 1800):
                m = i / 60
-               choicelist.append(("%d" % i, ngettext("%d minute", "%d minutes", m) % m))
+               choicelist.append((str(i), ngettext("%d minute", "%d minutes", m) % m))
        for i in (3600, 7200, 14400):
                h = i / 3600
-               choicelist.append(("%d" % i, ngettext("%d hour", "%d hours", h) % h))
+               choicelist.append((str(i), ngettext("%d hour", "%d hours", h) % h))
        config.usage.hdd_standby = ConfigSelection(default = "300", choices = [("0", _("No standby"))] + choicelist)
        config.usage.output_12V = ConfigSelection(default = "do not change", choices = [
                ("do not change", _("Do not change")), ("off", _("Off")), ("on", _("On")) ])
@@ -87,7 +87,7 @@ def InitUsageConfig():
        choicelist = [("-1", _("Disabled")), ("0", _("No timeout"))]
        for i in [60, 300, 600, 900, 1800, 2700, 3600]:
                m = i/60
-               choicelist.append(("%d" % i, ngettext("%d minute", "%d minutes", m) % m))
+               choicelist.append((str(i), ngettext("%d minute", "%d minutes", m) % m))
        config.usage.pip_last_service_timeout = ConfigSelection(default = "0", choices = choicelist)
 
        config.usage.default_path = ConfigText(default = resolveFilename(SCOPE_HDD))
@@ -135,7 +135,7 @@ def InitUsageConfig():
        for i in range(3600, 21601, 3600):
                h = abs(i / 3600)
                h = ngettext("%d hour", "%d hours", h) % h
-               choicelist.append(("%d" % i, _("Standby in ") + h))
+               choicelist.append((str(i), _("Standby in ") + h))
        config.usage.inactivity_timer = ConfigSelection(default = "0", choices = choicelist)
        config.usage.inactivity_timer_blocktime = ConfigYesNo(default = True)
        config.usage.inactivity_timer_blocktime_begin = ConfigClock(default = time.mktime((0, 0, 0, 18, 0, 0, 0, 0, 0)))
@@ -148,14 +148,14 @@ def InitUsageConfig():
        for i in range(900, 7201, 900):
                m = abs(i / 60)
                m = ngettext("%d minute", "%d minutes", m) % m
-               choicelist.append(("%d" % i, _("Standby in ") + m))
+               choicelist.append((str(i), _("Standby in ") + m))
        config.usage.sleep_timer = ConfigSelection(default = "0", choices = choicelist)
 
        choicelist = [("0", _("Disabled"))]
        for i in [60, 300, 600] + range(900, 7201, 900):
                m = abs(i / 60)
                m = ngettext("%d minute", "%d minutes", m) % m
-               choicelist.append(("%d" % i, _("after ") + m))
+               choicelist.append((str(i), _("after ") + m))
        config.usage.standby_to_shutdown_timer = ConfigSelection(default = "0", choices = choicelist)
        config.usage.standby_to_shutdown_timer_blocktime = ConfigYesNo(default = True)
        config.usage.standby_to_shutdown_timer_blocktime_begin = ConfigClock(default = time.mktime((0, 0, 0, 6, 0, 0, 0, 0, 0)))
@@ -170,10 +170,10 @@ def InitUsageConfig():
 
        choicelist = [("0", _("Disabled"))]
        for i in (2, 3, 4, 5, 10, 20, 30):
-               choicelist.append(("%d" % i, ngettext("%d second", "%d seconds", i) % i))
+               choicelist.append((str(i), ngettext("%d second", "%d seconds", i) % i))
        for i in (60, 120, 300):
                m = i / 60
-               choicelist.append(("%d" % i, ngettext("%d minute", "%d minutes", m) % m))
+               choicelist.append((str(i), ngettext("%d minute", "%d minutes", m) % m))
        config.usage.timeshift_start_delay = ConfigSelection(default = "0", choices = choicelist)
 
        config.usage.alternatives_priority = ConfigSelection(default = "0", choices = [
@@ -385,7 +385,7 @@ def InitUsageConfig():
                        open(SystemInfo["VFD_scroll_repeats"], "w").write(el.value)
                choicelist = []
                for i in range(1, 11, 1):
-                       choicelist.append(("%d" % i))
+                       choicelist.append((str(i)))
                config.usage.vfd_scroll_repeats = ConfigSelection(default = "3", choices = choicelist)
                config.usage.vfd_scroll_repeats.addNotifier(scroll_repeats, immediate_feedback = False)
 
@@ -394,7 +394,7 @@ def InitUsageConfig():
                        open(SystemInfo["VFD_scroll_delay"], "w").write(el.value)
                choicelist = []
                for i in range(0, 1001, 50):
-                       choicelist.append(("%d" % i))
+                       choicelist.append((str(i)))
                config.usage.vfd_scroll_delay = ConfigSelection(default = "150", choices = choicelist)
                config.usage.vfd_scroll_delay.addNotifier(scroll_delay, immediate_feedback = False)
 
@@ -403,7 +403,7 @@ def InitUsageConfig():
                        open(SystemInfo["VFD_initial_scroll_delay"], "w").write(el.value)
                choicelist = []
                for i in range(0, 20001, 500):
-                       choicelist.append(("%d" % i))
+                       choicelist.append((str(i)))
                config.usage.vfd_initial_scroll_delay = ConfigSelection(default = "1000", choices = choicelist)
                config.usage.vfd_initial_scroll_delay.addNotifier(initial_scroll_delay, immediate_feedback = False)
 
@@ -412,7 +412,7 @@ def InitUsageConfig():
                        open(SystemInfo["VFD_final_scroll_delay"], "w").write(el.value)
                choicelist = []
                for i in range(0, 20001, 500):
-                       choicelist.append(("%d" % i))
+                       choicelist.append((str(i)))
                config.usage.vfd_final_scroll_delay = ConfigSelection(default = "1000", choices = choicelist)
                config.usage.vfd_final_scroll_delay.addNotifier(final_scroll_delay, immediate_feedback = False)
 
@@ -434,7 +434,7 @@ def InitUsageConfig():
                if i == 0:
                        subtitle_delay_choicelist.append(("0", _("No delay")))
                else:
-                       subtitle_delay_choicelist.append(("%d" % i, "%2.1f sec" % (i / 90000.)))
+                       subtitle_delay_choicelist.append((str(i), "%2.1f sec" % (i / 90000.)))
        config.subtitles.subtitle_noPTSrecordingdelay = ConfigSelection(default = "315000", choices = subtitle_delay_choicelist)
 
        config.subtitles.dvb_subtitles_yellow = ConfigYesNo(default = False)