Actually, have you tested this with the IPv4 case at all? `!!!Integer(ip)` will either raise an exception (your IPv6 case) or it will set `ip = false`, the IPv4 case. I don't see how line 559 could turn `false` into a human-readable IPv4 address.