Unnecessary operation in IPv4Interface and IPv6Interface

[mssalvatore]

The __hash__() methods of the IPv4Interface and IPv6Interface classes are implemented on the main branch as follows:

    def __hash__(self):
        return hash((self._ip, self._prefixlen, int(self.network.network_address)))

https://github.com/python/cpython/blob/0afbd4e42ac28240b484cabe1710fdb6c572fb1f/Lib/ipaddress.py#L1457C1-L1459

As best as I can figure from tracing the relevant __init__() methods, self._ip and int(self.network.network_address) will always be identical. If that is so, this function could be simplified and save a few CPU cycles:

     def __hash__(self):
-        return hash((self._ip, self._prefixlen, int(self.network.network_address)))
+        return hash((self._ip, self._prefixlen))

Linked PRs

• gh-134012: Simplify IPv*Interface.__hash__() #134013

[mssalvatore]

I missed some details and the conclusion that self._ip and self.network.network_address are always identical are incorrect.