get rid of buffer stupidity which makes it seem like PIC sends NULs

diff --git a/TODO b/TODO
index f3d9d58a9e24ded53b9ba74c625075f354e71642..f67896dec94c09f1e9a311c792b72008a685c3be 100644 (file)
--- a/TODO
+++ b/TODO
@@ -1,19 +1,3 @@
-hostside/realtime -s/dev/ttyUSB0 sometimes says
-  picioh out 10
-  picio out on
-  picioh out 11
-  stastate 5
-  picio out ping 556
-  picioh out 8c 2c
-  realtime: fatal: PIC sent NMRADONE in Resolving
-  picioh in msg 8c 2c
-  picio in pong 556
-  picioh in msg 00
-  picio in nmradone 0
-  realtime: fatal: PIC sent NMRADONE in Resolving
-  liberator:hostside> 
-
-
 layout polarity diagram colourful segment encoding:
  make moveable feature selection in subseg2display bitwise rather than
   setwise

diff --git a/hostside/realtime.c b/hostside/realtime.c
index bfab9e21f52106fc29fe3f2217964dde045cc3bb..8c90a9bd39d70c341e80304f90592e6abfb7a184 100644 (file)
--- a/hostside/realtime.c
+++ b/hostside/realtime.c
@@ -261,8 +261,9 @@ static void *serial_readable(oop_source *evts, int fd,
                             oop_event evt, void *u0) {
   int r, buf_used;
 
-  r= read(serial_fd, &serial_buf.d + serial_buf.l,
+  r= read(serial_fd, serial_buf.d + serial_buf.l,
          sizeof(serial_buf.d) - serial_buf.l);
+
   if (r==0) die("serial port - eof");
   if (r==-1) {
     if (errno == EWOULDBLOCK || errno == EINTR)
@@ -271,8 +272,7 @@ static void *serial_readable(oop_source *evts, int fd,
   }
   assert(r>0);
 
-  buf_used= serial_buf.l;
-  buf_used += r;
+  buf_used= serial_buf.l + r;
 
   for (;;) {
     serial_buf.l= buf_used;