GeoIPv6 patch

Gregor Kališnik gregor at unimatrix-one.org
Sat Jun 11 22:19:52 MSD 2011


Hi.

On Saturday 11 of June 2011 16:54:18 you wrote:
> Unrelated whitespace breakage.  There are multiple other style
> issues, too.

There souldn't be any now in this patch.


> I don't really like this aproach.  Result looks to cluttered with
> #if's and hard to read.  Probably it's good idea to try to produce
> something better.

Put one #if .. #endif block at the begining. Not sure if it's good enough 
(does inherit compile errors, but makes two identical pathways)...

> Using inet_ntop/sprintf/inet_pton to map ipv4 binary address to
> ipv4-mapped ipv6 binary address is cool.  :)
> 
> What about just setting appropriate bytes?

Now using binary operators :).


> It's probably good idea to centralize all these checks at
> configuration stage.
> 
> Please also note that nginx style is to use 4 spaces for
> indentation, and always use {}.

I added three variables to configure struct (is_<type>_ipv6).


> Some configure checks for ipv6 support in libgeoip may also be
> needed as I have no idea if recent libgeoip versions are widely
> availabe on some known-to-be-slow Linux'es.  Not sure, but it at
> least deserves some investigation.
> 
> Maxim Dounin

Added a check into auto/unix. Check for availability of database type names.

Best regards,
Gregor Kališnik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: nginx-geoipv6.patch
Type: text/x-patch
Size: 8883 bytes
Desc: not available
URL: <http://nginx.org/pipermail/nginx-devel/attachments/20110611/0e2b72d9/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://nginx.org/pipermail/nginx-devel/attachments/20110611/0e2b72d9/attachment.pgp>


More information about the nginx-devel mailing list