[squid-dev] splay.h replacement

[Tsantilas Christos]

The compiler has right....


On 11/18/2014 05:23 AM, Amos Jeffries wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Y'all may have noticed the clang 3.5 errors.
>
> lib/MemPoolChunked.cc:370:10: error: 'this' pointer cannot be null in
> well-defined C++ code; pointer may be assumed to always convert to
> true [-Werror,-Wundefined-bool-conversion]
>
> include/splay.h:228:9: error: 'this' pointer cannot be null in
> well-defined C++ code; comparison may be assumed to always evaluate to
> false [-Werror,-Wtautological-undefined-compare]
>
> include/splay.h:198:9: error: 'this' pointer cannot be null in
> well-defined C++ code; comparison may be assumed to always evaluate to
> false [-Werror,-Wtautological-undefined-compare]
>
> include/splay.h:167:9: error: 'this' pointer cannot be null in
> well-defined C++ code; comparison may be assumed to always evaluate to
> false [-Werror,-Wtautological-undefined-compare]
>
> include/splay.h:228:9: error: 'this' pointer cannot be null in
> well-defined C++ code; comparison may be assumed to always evaluate to
> false [-Werror,-Wtautological-undefined-compare]
>
>
> Anyone in a position to update the splay tree code so it stops
> depending on NULL pointer dereferences having meaning?
>
> It is mandatory change for continued FreeBSD support.
>
> Amos
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v2.0.22 (MingW32)
>
> iQEcBAEBAgAGBQJUaruvAAoJELJo5wb/XPRjEQ4IAL2tCJsN2vK4VO/sp0RX1zMv
> /PPUYWUWGlU4dVhOBTHR8TQ3zrTrt+rwl+LQcOMbpPGkUMWAc9rg+y2HLk+EKiaw
> sV4emS9R645O7NlZItipilnDbfQLSx82g6gbd1BWfl5vqVpkx27jttSh/0SNVnFV
> B49LR9m9CcyRQaGQe5wVdwRHco2We/Kx3KB4JPNuoxjdVpBj2YDCNzRSfod++RZX
> tgcjjKMaOibo3L9Fmx91zWcguzg7s0fGwwYCPn7usI6TUAeZwfv96R2sY8U3MYjc
> 3oiI1rq6iU5alz77uQdh4f+DSWlOplRGGeh7wz3OGWhivj3oA62sVCJFPmpCuZM=
> =Fni5
> -----END PGP SIGNATURE-----
> _______________________________________________
> squid-dev mailing list
> squid-dev at lists.squid-cache.org
> http://lists.squid-cache.org/listinfo/squid-dev
>