From: Keith Packard <keithp@keithp.com>
Date: Fri, 28 Jan 2022 22:13:53 +0000 (-0800)
Subject: altos/ao_stdio: use uint8_t for stdio index
X-Git-Tag: 1.9.10.4~38
X-Git-Url: https://git.gag.com/?p=fw%2Faltos;a=commitdiff_plain;h=3d276e01ba96e8174e590b101cc9ef002e6cb67b

altos/ao_stdio: use uint8_t for stdio index

This makes type usage of this value consistent. Also fix another
-Wconversion warning.

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

diff --git a/src/kernel/ao.h b/src/kernel/ao.h
index c1a8609c..806b4946 100644
--- a/src/kernel/ao.h
+++ b/src/kernel/ao.h
@@ -724,8 +724,8 @@ struct ao_stdio {
 };
 
 extern struct ao_stdio ao_stdios[];
-extern int8_t ao_cur_stdio;
-extern int8_t ao_num_stdios;
+extern uint8_t ao_cur_stdio;
+extern uint8_t ao_num_stdios;
 
 void
 flush(void);
@@ -735,7 +735,7 @@ extern uint8_t ao_stdin_ready;
 uint8_t
 ao_echo(void);
 
-int8_t
+uint8_t
 ao_add_stdio(int (*pollchar)(void),
 	     void (*putchar)(char) ,
 	     void (*flush)(void));
diff --git a/src/kernel/ao_stdio.c b/src/kernel/ao_stdio.c
index c578c57c..7f74f811 100644
--- a/src/kernel/ao_stdio.c
+++ b/src/kernel/ao_stdio.c
@@ -75,10 +75,10 @@
 struct ao_stdio ao_stdios[AO_NUM_STDIOS];
 
 #if AO_NUM_STDIOS > 1
-int8_t ao_cur_stdio;
-int8_t ao_num_stdios;
+uint8_t ao_cur_stdio;
+uint8_t ao_num_stdios;
 #else
-int8_t ao_cur_stdio;
+uint8_t ao_cur_stdio;
 #define ao_cur_stdio	0
 #define ao_num_stdios	0
 #endif
@@ -114,7 +114,7 @@ char
 ao_getchar(void) 
 {
 	int c;
-	int8_t stdio;
+	uint8_t stdio;
 
 	ao_arch_block_interrupts();
 	stdio = ao_cur_stdio;
@@ -133,7 +133,7 @@ ao_getchar(void)
 	ao_cur_stdio = stdio;
 #endif
 	ao_arch_release_interrupts();
-	return c;
+	return (char) c;
 }
 
 uint8_t
@@ -142,7 +142,7 @@ ao_echo(void)
 	return ao_stdios[ao_cur_stdio].echo;
 }
 
-int8_t
+uint8_t
 ao_add_stdio(int (*_pollchar)(void),
 	     void (*putchar)(char),
 	     void (*_flush)(void))