fix: remove useless calls in ServiceInfo

bdraco · bdraco · commit 516812e8428c · 2023-09-01T23:28:14.000-05:00
diff --git a/src/zeroconf/_services/info.py b/src/zeroconf/_services/info.py
@@ -427,17 +427,17 @@ def _process_record_threadsafe(self, zc: 'Zeroconf', record: DNSRecord, now: flo
             return False
 
         record_key = record.key
-        if record_key == self.server_key and type(record) is DNSAddress:
+        record_type = type(record)
+        if record_key == self.server_key and record_type is DNSAddress:
+            if TYPE_CHECKING:
+                assert isinstance(record, DNSAddress)
             try:
                 ip_addr = _cached_ip_addresses(record.address)
             except ValueError as ex:
                 log.warning("Encountered invalid address while processing %s: %s", record, ex)
                 return False
 
             if type(ip_addr) is IPv4Address:
-                if self._ipv4_addresses:
-                    self._set_ipv4_addresses_from_cache(zc, now)
-
                 ipv4_addresses = self._ipv4_addresses
                 if ip_addr not in ipv4_addresses:
                     ipv4_addresses.insert(0, ip_addr)
@@ -448,9 +448,6 @@ def _process_record_threadsafe(self, zc: 'Zeroconf', record: DNSRecord, now: flo
 
                 return False
 
-            if not self._ipv6_addresses:
-                self._set_ipv6_addresses_from_cache(zc, now)
-
             ipv6_addresses = self._ipv6_addresses
             if ip_addr not in self._ipv6_addresses:
                 ipv6_addresses.insert(0, ip_addr)
@@ -464,11 +461,15 @@ def _process_record_threadsafe(self, zc: 'Zeroconf', record: DNSRecord, now: flo
         if record_key != self.key:
             return False
 
-        if record.type == _TYPE_TXT and type(record) is DNSText:
+        if record_type is DNSText:
+            if TYPE_CHECKING:
+                assert isinstance(record, DNSText)
             self._set_text(record.text)
             return True
 
-        if record.type == _TYPE_SRV and type(record) is DNSService:
+        if record_type is DNSService:
+            if TYPE_CHECKING:
+                assert isinstance(record, DNSService)
             old_server_key = self.server_key
             self.name = record.name
             self.server = record.server
