diff options
author | Keith Packard <keithp@keithp.com> | 2012-09-09 12:29:32 -0700 |
---|---|---|
committer | Keith Packard <keithp@keithp.com> | 2012-09-09 12:29:32 -0700 |
commit | 9682e9e6fe730417a77b47795fbe1f06c9a51177 (patch) | |
tree | be836448407436123ae611ddd841989aefaae12b /altoslib | |
parent | d65d921b9b2340fa23d3b55b4ae755324d392303 (diff) |
altosui: Use helper functions to access arrays in AltosLib class
These deal with out-of-range values correctly, instead of causing
exceptions that will just break stuff.
Signed-off-by: Keith Packard <keithp@keithp.com>
Diffstat (limited to 'altoslib')
-rw-r--r-- | altoslib/AltosLib.java | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/altoslib/AltosLib.java b/altoslib/AltosLib.java index 2402331e..d36b2ff7 100644 --- a/altoslib/AltosLib.java +++ b/altoslib/AltosLib.java @@ -111,7 +111,7 @@ public class AltosLib { public static final int ao_telemetry_0_8 = 3; public static final int ao_telemetry_max = 3; - public static final String[] ao_telemetry_name = { + private static final String[] ao_telemetry_name = { "Off", "Standard Telemetry", "TeleMetrum v0.9", "TeleMetrum v0.8" }; @@ -121,13 +121,13 @@ public class AltosLib { public static final int ao_telemetry_0_9_len = 95; public static final int ao_telemetry_0_8_len = 94; - public static final int[] ao_telemetry_len = { + private static final int[] ao_telemetry_len = { 0, 32, 95, 94 }; - public static HashMap<String,Integer> string_to_state = new HashMap<String,Integer>(); + private static HashMap<String,Integer> string_to_state = new HashMap<String,Integer>(); - public static boolean map_initialized = false; + private static boolean map_initialized = false; public static void initialize_map() { @@ -159,7 +159,7 @@ public class AltosLib { telemetry)); } - public static String[] state_to_string = { + private static String[] state_to_string = { "startup", "idle", "pad", @@ -172,7 +172,7 @@ public class AltosLib { "invalid", }; - public static String[] state_to_string_capital = { + private static String[] state_to_string_capital = { "Startup", "Idle", "Pad", @@ -199,6 +199,12 @@ public class AltosLib { return state_to_string[state]; } + public static String state_name_capital(int state) { + if (state < 0 || state_to_string.length <= state) + return "invalid"; + return state_to_string_capital[state]; + } + public static final int AO_GPS_VALID = (1 << 4); public static final int AO_GPS_RUNNING = (1 << 5); public static final int AO_GPS_DATE_VALID = (1 << 6); |