From: Matt Wilmas <mattwil@php.net>
Date: Tue, 31 Mar 2009 11:22:19 +0000 (+0000)
Subject: - MFH: Removed unused LONG_SIGN_MASK definition, and a stray \
X-Git-Tag: php-5.2.10RC1~253
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9c3e1e96b84af8d67c64cc474655a8bd0ad230bd;p=php

- 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 84a54f22bd..40011a9e21 100644
--- 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;
 		}