linux_termios: fix return types to actually match returned values

This fixes the following warning :

linux_termios.c: In function 'get_termios_get_ioctl':
linux_termios.c:42:2: warning: overflow in implicit constant conversion [-Woverflow]
  return TCGETS2;
  ^

This fixes bug #396.

diff --git a/linux_termios.c b/linux_termios.c
index 7c1540a2ddb4e0450a4d1e4c5b66b985a263fb6c..23c7ae96539c9d5155a243102e55e09045c9cd7d 100644 (file)
--- a/linux_termios.c
+++ b/linux_termios.c
@@ -33,10 +33,11 @@
  * TCSETX/TCGETX ioctls used with struct termiox, others do not.
  */
 
+#include <stdlib.h>
 #include <linux/termios.h>
 #include "linux_termios.h"
 
-SP_PRIV int get_termios_get_ioctl(void)
+SP_PRIV unsigned long get_termios_get_ioctl(void)
 {
 #ifdef HAVE_TERMIOS2
        return TCGETS2;
@@ -45,7 +46,7 @@ SP_PRIV int get_termios_get_ioctl(void)
 #endif
 }
 
-SP_PRIV int get_termios_set_ioctl(void)
+SP_PRIV unsigned long get_termios_set_ioctl(void)
 {
 #ifdef HAVE_TERMIOS2
        return TCSETS2;
@@ -54,7 +55,7 @@ SP_PRIV int get_termios_set_ioctl(void)
 #endif
 }
 
-SP_PRIV int get_termios_size(void)
+SP_PRIV size_t get_termios_size(void)
 {
 #ifdef HAVE_TERMIOS2
        return sizeof(struct termios2);
@@ -91,7 +92,7 @@ SP_PRIV void set_termios_speed(void *data, int speed)
 #endif
 
 #ifdef HAVE_TERMIOX
-SP_PRIV int get_termiox_size(void)
+SP_PRIV size_t get_termiox_size(void)
 {
        return sizeof(struct termiox);
 }

diff --git a/linux_termios.h b/linux_termios.h
index 37d4dc91e89be957c8bee2c7cb8c95bac705798b..e7c86e705b2c7fc78ba3c5e33e3a4f49c4f5599c 100644 (file)
--- a/linux_termios.h
+++ b/linux_termios.h
@@ -20,12 +20,14 @@
 #ifndef LIBSERIALPORT_LINUX_TERMIOS_H
 #define LIBSERIALPORT_LINUX_TERMIOS_H
 
-SP_PRIV int get_termios_get_ioctl(void);
-SP_PRIV int get_termios_set_ioctl(void);
-SP_PRIV int get_termios_size(void);
+#include <stdlib.h>
+
+SP_PRIV unsigned long get_termios_get_ioctl(void);
+SP_PRIV unsigned long get_termios_set_ioctl(void);
+SP_PRIV size_t get_termios_size(void);
 SP_PRIV int get_termios_speed(void *data);
 SP_PRIV void set_termios_speed(void *data, int speed);
-SP_PRIV int get_termiox_size(void);
+SP_PRIV size_t get_termiox_size(void);
 SP_PRIV int get_termiox_flow(void *data, int *rts, int *cts, int *dtr, int *dsr);
 SP_PRIV void set_termiox_flow(void *data, int rts, int cts, int dtr, int dsr);