diff options
| author | Keith Packard <keithp@keithp.com> | 2015-09-12 19:20:49 -0700 |
|---|---|---|
| committer | Keith Packard <keithp@keithp.com> | 2015-10-13 13:54:28 -0700 |
| commit | 0afa07d3c1dcb5e301fcb8b4edfecdd961662478 (patch) | |
| tree | bd0742a81d3ef60933358fa5151e5d49910cacbe /altoslib/AltosState.java | |
| parent | 7064bc685aebeef07711e525dea4d5fbe33d235b (diff) | |
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 <keithp@keithp.com>
Diffstat (limited to 'altoslib/AltosState.java')
| -rw-r--r-- | altoslib/AltosState.java | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/altoslib/AltosState.java b/altoslib/AltosState.java index cf4fb9b0..12cd5adf 100644 --- a/altoslib/AltosState.java +++ b/altoslib/AltosState.java @@ -284,7 +284,7 @@ public class AltosState implements Cloneable, Serializable { } } - public int state; + private int state; public int flight; public int serial; public int altitude_32; @@ -1056,6 +1056,10 @@ public class AltosState implements Cloneable, Serializable { } } + public int state() { + return state; + } + public void set_device_type(int device_type) { this.device_type = device_type; switch (device_type) { |