abuf: use a define for the maximum size
authorFerry Huberts <ferry.huberts@pelagic.nl>
Thu, 26 Nov 2015 21:40:58 +0000 (22:40 +0100)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Thu, 2 Jun 2016 21:41:09 +0000 (23:41 +0200)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
src/common/autobuf.c
src/common/autobuf.h

index 2ecf825..1718538 100644 (file)
@@ -50,7 +50,6 @@
 #include <string.h>
 #include <stdlib.h>
 #include <errno.h>
-#include <limits.h>
 #include <assert.h>
 
 static int autobuf_enlarge(struct autobuf *autobuf, unsigned int new_size);
@@ -59,13 +58,13 @@ static int ROUND_UP_TO_POWER_OF_2(int val, int pow2) {
   assert(val >= 0);
   assert(pow2 > 0);
 
-  if (val <= (INT32_MAX - (pow2 - 1))) {
+  if (val <= (AUTOBUFSIZEMAX - (pow2 - 1))) {
     /* no overflow */
     return ((val + (pow2 - 1)) & ~(pow2 - 1));
   }
 
   /* overflow */
-  return (INT32_MAX & ~(pow2 - 1));
+  return (AUTOBUFSIZEMAX & ~(pow2 - 1));
 }
 
 int abuf_init(struct autobuf *autobuf, int initial_size) {
@@ -95,7 +94,7 @@ static int autobuf_enlarge(struct autobuf *autobuf, unsigned int new_size) {
   /* for the the string terminator */
   new_size++;
 
-  if (autobuf->size >= INT_MAX) {
+  if (autobuf->size >= AUTOBUFSIZEMAX) {
     return -1;
   }
 
index 42f8a3b..01f69da 100644 (file)
@@ -51,8 +51,12 @@ struct autobuf;
 #include "defs.h"
 #include <stdarg.h>
 #include <time.h>
+#include <limits.h>
 
 #define AUTOBUFCHUNK   4096
+
+#define AUTOBUFSIZEMAX INT_MAX
+
 struct autobuf {
     int size;
     int len;