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 --- altosui/AltosPad.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'altosui/AltosPad.java') diff --git a/altosui/AltosPad.java b/altosui/AltosPad.java index 2f87b681..d056e256 100644 --- a/altosui/AltosPad.java +++ b/altosui/AltosPad.java @@ -49,10 +49,10 @@ public class AltosPad extends AltosUIFlightTab { hide(); } else { if (state.flight != 0) { - if (state.state <= Altos.ao_flight_pad) + if (state.state() <= Altos.ao_flight_pad) show("Ready to record"); - else if (state.state < Altos.ao_flight_landed || - state.state == AltosLib.ao_flight_stateless) + else if (state.state() < Altos.ao_flight_landed || + state.state() == AltosLib.ao_flight_stateless) show("Recording data"); else show("Recorded data"); @@ -121,8 +121,8 @@ public class AltosPad extends AltosUIFlightTab { } boolean report_pad(AltosState state) { - if ((state.state == AltosLib.ao_flight_stateless || - state.state < AltosLib.ao_flight_pad) && + if ((state.state() == AltosLib.ao_flight_stateless || + state.state() < AltosLib.ao_flight_pad) && state.gps != null && state.gps.lat != AltosLib.MISSING) { -- cgit v1.2.3