Index: trunk/Ohana/src/libfits/include/gfitsio.h
===================================================================
--- trunk/Ohana/src/libfits/include/gfitsio.h	(revision 41375)
+++ trunk/Ohana/src/libfits/include/gfitsio.h	(revision 41395)
@@ -19,5 +19,5 @@
 
 /* gfits_bintable_format returns 'byte' as the type associated with one-byte logical (non-char) values */
-typedef char byte;
+typedef unsigned char byte;
 
 /********** FITS Constants *********/
Index: trunk/Ohana/src/libfits/table/F_get_column.c
===================================================================
--- trunk/Ohana/src/libfits/table/F_get_column.c	(revision 41375)
+++ trunk/Ohana/src/libfits/table/F_get_column.c	(revision 41395)
@@ -1,5 +1,5 @@
 # include <ohana.h>
 # include <gfitsio.h>
-#include <inttypes.h>
+# include <inttypes.h>
 # define SWAP_BYTE { \
   char tmp; \
Index: trunk/Ohana/src/libfits/table/F_set_column.c
===================================================================
--- trunk/Ohana/src/libfits/table/F_set_column.c	(revision 41375)
+++ trunk/Ohana/src/libfits/table/F_set_column.c	(revision 41395)
@@ -338,5 +338,5 @@
 
   SET_VALUES("char",       char, "char", char, SWAP_NONE, 1);
-  SET_VALUES("byte",       char, "char", char, SWAP_NONE, 1);
+  SET_VALUES("byte",       byte, "char", char, SWAP_NONE, 1);
   SET_VALUES("short",     short, "char", char, SWAP_BYTE, 1);
   SET_VALUES("int",         int, "char", char, SWAP_WORD, 1);
@@ -345,14 +345,14 @@
   SET_VALUES("double",   double, "char", char, SWAP_DBLE, 1);
 
-  SET_VALUES("char",       char, "byte", char, SWAP_NONE, 1);
-  SET_VALUES("byte",       char, "byte", char, SWAP_NONE, 1);
-  SET_VALUES("short",     short, "byte", char, SWAP_BYTE, 1);
-  SET_VALUES("int",         int, "byte", char, SWAP_WORD, 1);
-  SET_VALUES("int64_t", int64_t, "byte", char, SWAP_DBLE, 1);
-  SET_VALUES("float",     float, "byte", char, SWAP_WORD, 1);
-  SET_VALUES("double",   double, "byte", char, SWAP_DBLE, 1);
+  SET_VALUES("char",       char, "byte", byte, SWAP_NONE, 1);
+  SET_VALUES("byte",       byte, "byte", byte, SWAP_NONE, 1);
+  SET_VALUES("short",     short, "byte", byte, SWAP_BYTE, 1);
+  SET_VALUES("int",         int, "byte", byte, SWAP_WORD, 1);
+  SET_VALUES("int64_t", int64_t, "byte", byte, SWAP_DBLE, 1);
+  SET_VALUES("float",     float, "byte", byte, SWAP_WORD, 1);
+  SET_VALUES("double",   double, "byte", byte, SWAP_DBLE, 1);
 
   SET_VALUES("char",       char, "short", short, SWAP_NONE, 2);
-  SET_VALUES("byte",       char, "short", short, SWAP_NONE, 2);
+  SET_VALUES("byte",       byte, "short", short, SWAP_NONE, 2);
   SET_VALUES("short",     short, "short", short, SWAP_BYTE, 2);
   SET_VALUES("int",         int, "short", short, SWAP_WORD, 2);
@@ -362,5 +362,5 @@
 
   SET_VALUES("char",       char, "int", int, SWAP_NONE, 4);
-  SET_VALUES("byte",       char, "int", int, SWAP_NONE, 4);
+  SET_VALUES("byte",       byte, "int", int, SWAP_NONE, 4);
   SET_VALUES("short",     short, "int", int, SWAP_BYTE, 4);
   SET_VALUES("int",         int, "int", int, SWAP_WORD, 4);
@@ -370,5 +370,5 @@
 
   SET_VALUES("char",       char, "int64_t", int64_t, SWAP_NONE, 8);
-  SET_VALUES("byte",       char, "int64_t", int64_t, SWAP_NONE, 8);
+  SET_VALUES("byte",       byte, "int64_t", int64_t, SWAP_NONE, 8);
   SET_VALUES("short",     short, "int64_t", int64_t, SWAP_BYTE, 8);
   SET_VALUES("int",         int, "int64_t", int64_t, SWAP_WORD, 8);
@@ -378,5 +378,5 @@
 
   SET_VALUES("char",       char, "float", float, SWAP_NONE, 4);
-  SET_VALUES("byte",       char, "float", float, SWAP_NONE, 4);
+  SET_VALUES("byte",       byte, "float", float, SWAP_NONE, 4);
   SET_VALUES("short",     short, "float", float, SWAP_BYTE, 4);
   SET_VALUES("int",         int, "float", float, SWAP_WORD, 4);
@@ -386,5 +386,5 @@
 
   SET_VALUES("char",       char, "double", double, SWAP_NONE, 8);
-  SET_VALUES("byte",       char, "double", double, SWAP_NONE, 8);
+  SET_VALUES("byte",       byte, "double", double, SWAP_NONE, 8);
   SET_VALUES("short",     short, "double", double, SWAP_BYTE, 8);
   SET_VALUES("int",         int, "double", double, SWAP_WORD, 8);
