altos: Make ao_fec_tx_test build cleanly with -Wall

Signed-off-by: Keith Packard <keithp@keithp.com>

diff --git a/src/test/Makefile b/src/test/Makefile
index 024a54c1321055f11ab7246a029e1397b2981f72..7bcf24e1bf9257cebb233104974201c69266abb9 100644 (file)
--- a/src/test/Makefile
+++ b/src/test/Makefile
@@ -4,7 +4,7 @@ PROGS=ao_flight_test ao_flight_test_baro ao_flight_test_accel ao_flight_test_noi
 
 KALMAN=make-kalman 
 
-CFLAGS=-I.. -I. -I../core -I../drivers -O0 -g
+CFLAGS=-I.. -I. -I../core -I../drivers -O0 -g -Wall
 
 all: $(PROGS)
 

diff --git a/src/test/ao_fec_tx_test.c b/src/test/ao_fec_tx_test.c
index d01eadc5c364b790ee784c88c4e93453a2c6981d..1b1fd56d47d54fd1d8d7ea149b2b4ae41304eff7 100644 (file)
--- a/src/test/ao_fec_tx_test.c
+++ b/src/test/ao_fec_tx_test.c
@@ -77,7 +77,6 @@ ao_encode(uint8_t *input, int input_len, uint8_t *output)
 {
        uint8_t         prepare[PREPARE_LEN(input_len)];
        uint8_t         encode[ENCODE_LEN(input_len)];
-       uint8_t         interleave[INTERLEAVE_LEN(input_len)];
        uint8_t         prepare_len;
        uint8_t         encode_len;
        uint8_t         interleave_len;
@@ -159,6 +158,7 @@ ao_decode(uint8_t *bytes, int bytes_len, uint8_t *bits)
        bits_len = ao_fec_decode(bytes, bytes_len, bits);
 
        ao_fec_dump_bytes(bits, bits_len, "Decode");
+       return bits_len;
 }
 
 int
@@ -182,9 +182,11 @@ main(int argc, char **argv)
        transmit_len = ao_radio(encode, encode_len, transmit);
 
        /* apply gaussian noise to test viterbi code against errors */
-       receive_len = ao_fuzz(transmit, transmit_len, receive, 0x80);
+       receive_len = ao_fuzz(transmit, transmit_len, receive, 0x70);
 
        decode_len = ao_decode(receive, receive_len, decode);
+
+       return decode_len >= sizeof(original);
 }