From 0afa07d3c1dcb5e301fcb8b4edfecdd961662478 Mon Sep 17 00:00:00 2001 From: Keith Packard Date: Sat, 12 Sep 2015 19:20:49 -0700 Subject: altoslib: Hide 'state' member and use accessor function Someone was smashing the state to 'landed' when no packets had been received for a while. Found that by making it impossible for anyone outside of AltosState to change the value. Signed-off-by: Keith Packard --- altoslib/AltosKML.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'altoslib/AltosKML.java') diff --git a/altoslib/AltosKML.java b/altoslib/AltosKML.java index 81433958..1a364106 100644 --- a/altoslib/AltosKML.java +++ b/altoslib/AltosKML.java @@ -112,8 +112,8 @@ public class AltosKML implements AltosWriter { boolean started = false; void state_start(AltosState state) { - String state_name = AltosLib.state_name(state.state); - String state_color = state_color(state.state); + String state_name = AltosLib.state_name(state.state()); + String state_color = state_color(state.state()); out.printf(kml_style_start, state_name, state_color); out.printf("\tState: %s\n", state_name); out.printf("%s", kml_style_end); @@ -171,8 +171,8 @@ public class AltosKML implements AltosWriter { } if (prev != null && prev.gps_sequence == state.gps_sequence) return; - if (state.state != flight_state) { - flight_state = state.state; + if (state.state() != flight_state) { + flight_state = state.state(); if (prev != null) { coord(state); state_end(prev); -- cgit v1.2.3