Re: cvs: phpweb /include ip-to-country.inc

From: Date: Sat, 24 May 2003 20:12:16 +0000
Subject: Re: cvs: phpweb /include ip-to-country.inc
References: 1  Groups: php.mirrors 
Request: Send a blank email to php-mirrors+get-18038@lists.php.net to get a copy of this message
jacques Sat May 24 16:01:56 2003 EDT
  Modified files:                  /phpweb/include	ip-to-country.inc   Log:
- Quickfix to get rasmus IP to lookup.
     // Find the index to start search from
     $idx = i2c_search_in_index($ipn);
+    // For some odd reason I need to remove 2 from the idx value to
+    // get the country of rasmus and various other IP's to lookup.
+    $idx--;
+    $idx--;
Hm, while you are at this, I would be very happy if you could look at the indexing code too and find out why this is a problem. I have written the indexing code, and there were no serious reports of that so far, so it is strange if it is that bad (to miss IP regions by two). But there can be errors in there, I am not saying that the code is good... But IMHO it would be better to find out the real problem, instead of checking for two more regions... Sorry, but I don't have time to look into this. As you may have seen I am on fixing some other stuff. Goba

« previous php.mirrors (#18038) next »