build: fix two more warnings

format/slave.c: In function 'eb_device_slave': format/slave.c:165:7: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] tag = be32toh(*(uint32_t*)&buffer[4]); transport/dev.c: In function 'eb_dev_send': transport/dev.c:185:3: warning: ignoring return value of 'write', declared with attribute warn_unused_result [-Wunused-result] write(link->fdes, buf, len);

build: fix two more warnings
format/slave.c: In function 'eb_device_slave': format/slave.c:165:7: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] tag = be32toh(*(uint32_t*)&buffer[4]); transport/dev.c: In function 'eb_dev_send': transport/dev.c:185:3: warning: ignoring return value of 'write', declared with attribute warn_unused_result [-Wunused-result] write(link->fdes, buf, len);
edc60201 · Wesley W. Terpstra · e098ecc8 · edc60201 · edc60201
Commit edc60201 authored Apr 18, 2016 by Wesley W. Terpstra
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

slave.c api/format/slave.c +1 -1

dev.c api/transport/dev.c +2 -1

No files found.
--- a/api/format/slave.c
+++ b/api/format/slave.c
@@ -162,7 +162,7 @@ int eb_device_slave(eb_socket_t socketp, eb_transport_t transportp, eb_device_t
      if (passive) goto kill; /* passive link not responded! */
      
      /* Find device by probe id */
-      tag = be32toh(*(uint32_t*)&buffer[4]);
+      tag = EB_LOAD(&buffer[4], EB_DATA32);
      
      for (devp = socket->first_device; devp != EB_NULL; devp = dev->next) {
        dev = EB_DEVICE(devp);

--- a/api/transport/dev.c
+++ b/api/transport/dev.c
@@ -182,7 +182,8 @@ void eb_dev_send(struct eb_transport* transportp, struct eb_link* linkp, const u
  /* Set blocking */
  eb_dev_set_blocking(link, 1);

-  write(link->fdes, buf, len);
+  /* Wrap this in an if(); to silence warning about ignored result */
+  if (write(link->fdes, buf, len));
 }

 void eb_dev_send_buffer(struct eb_transport* transportp, struct eb_link* linkp, int on) {