From cfc27b3df587daa6282097cb5bf09b368a35a8a0 Mon Sep 17 00:00:00 2001 From: Xidorn Quan Date: Tue, 19 Jul 2016 10:02:10 +1000 Subject: [PATCH] Convert platform dictionary to if-else Makes it more extendible. --- ports/geckolib/gecko_bindings/tools/regen.py | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/ports/geckolib/gecko_bindings/tools/regen.py b/ports/geckolib/gecko_bindings/tools/regen.py index eebe31305fc..15c2a916bd4 100755 --- a/ports/geckolib/gecko_bindings/tools/regen.py +++ b/ports/geckolib/gecko_bindings/tools/regen.py @@ -135,11 +135,13 @@ def platform_dependent_defines(): if os.name == "posix": ret.append("-DOS_POSIX=1") - ret.append({ - "Linux": "-DOS_LINUX=1", - "Darwin": "-DOS_MACOSX=1", - # TODO: Windows? - }[platform.system()]) + system = platform.system() + if system == "Linux": + ret.append("-DOS_LINUX=1") + elif system == "Darwin": + ret.append("-DOS_MACOSX=1") + else: + raise Exception("Unknown platform") return ret