- MFH: Removed unused LONG_SIGN_MASK definition, and a stray \

  - Changed sign flag to int (it's no longer being used in a calculation)

diff --git a/ext/filter/logical_filters.c b/ext/filter/logical_filters.c
index 84a54f22bd5c9d0ddbc4f9433d8f4d69fecca645..40011a9e2122c55622a950d3ef87443795749f42 100644 (file)
--- a/ext/filter/logical_filters.c
+++ b/ext/filter/logical_filters.c
@@ -30,8 +30,6 @@
 # include <arpa/inet.h>
 #endif
 
-#define LONG_SIGN_MASK (1L << (8*sizeof(long)-1))
-
 #ifndef INADDR_NONE
 # define INADDR_NONE ((unsigned long int) -1)
 #endif
@@ -70,7 +68,7 @@
 
 static int php_filter_parse_int(const char *str, unsigned int str_len, long *ret TSRMLS_DC) { /* {{{ */
        long ctx_value;
-       long sign = 0;
+       int sign = 0;
        const char *end = str + str_len;
 
        switch (*str) {
@@ -97,7 +95,7 @@ static int php_filter_parse_int(const char *str, unsigned int str_len, long *ret
 
        while (str < end) {
                if (*str >= '0' && *str <= '9') {
-                       ctx_value = (ctx_value * 10) + (*(str++) - '0');                                                                \
+                       ctx_value = (ctx_value * 10) + (*(str++) - '0');
                } else {
                        return -1;
                }