From: Matt Wilmas <mattwil@php.net>
Date: Tue, 31 Mar 2009 11:20:31 +0000 (+0000)
Subject: - Removed unused LONG_SIGN_MASK definition, and a stray \
X-Git-Tag: php-5.4.0alpha1~191^2~4010
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=029b61b05e44a63439387db3287c08c2132104cc;p=php

- 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 6fdc37b558..58eef8e8c8 100644
--- a/ext/filter/logical_filters.c
+++ b/ext/filter/logical_filters.c
@@ -34,8 +34,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
@@ -74,7 +72,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) {
@@ -101,7 +99,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;
 		}