Minor cleanup

msantos · Mar 13, 2012 · 8afa967 · 8afa967
1 parent 4928a68
commit 8afa967
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/c_src/serctl.c b/c_src/serctl.c
@@ -130,7 +130,7 @@ nif_read(ErlNifEnv *env, int argc, const ERL_NIF_TERM argv[])
     if (!enif_alloc_binary(len, &buf))
         return error_tuple(env, ENOMEM);
 
-    if ( (bufsz = read(sp->fd, buf.data, buf.size)) == -1) {
+    if ( (bufsz = read(sp->fd, buf.data, buf.size)) < 0) {
         int err = errno;
         enif_release_binary(&buf);
         return error_tuple(env, err);
@@ -157,7 +157,7 @@ nif_write(ErlNifEnv *env, int argc, const ERL_NIF_TERM argv[])
     if (!enif_inspect_binary(env, argv[1], &buf))
         return enif_make_badarg(env);
 
-    if (write(sp->fd, buf.data, buf.size) == -1)
+    if (write(sp->fd, buf.data, buf.size) < 0)
         return error_tuple(env, errno);
 
     return atom_ok;

diff --git a/c_src/serctl.h b/c_src/serctl.h
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011, Michael Santos <michael.santos@gmail.com>
+ * Copyright (c) 2011-2012, Michael Santos <michael.santos@gmail.com>
  * All rights reserved.
  * 
  * Redistribution and use in source and binary forms, with or without
@@ -45,8 +45,8 @@
 #define SRLY_VERSION   "0.01"
 
 struct SERCTL_DEF {
-    const char  *key;
-    u_int32_t     val;
+    const char *key;
+    u_int32_t val;
 };
 
 const struct SERCTL_DEF serctl_const[] = {