On 05/13/2013 05:22 PM, Petr Spacek wrote:
On 13.5.2013 16:49, Simo Sorce wrote:
On Mon, 2013-05-13 at 16:32 +0200, Petr Spacek wrote:
+       if ((ip = inet_addr(ip_str)) == INADDR_NONE) {



This kind of construct is hard to read and debug in gdb
I would suggest you do:
ip = inet_addr(ip_str);
if (ip == INADDER_NONE) {

I agree, done.



_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
I tested the patch, works fine. ACK

Tomas
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to