comparison hgext/zeroconf/Zeroconf.py @ 51688:25e7f9dcad0f

zeroconf: fix boolean return value

This was (wrongly) flagged by Pytype as being undefined since it doesn't
seem to understand `try` blocks. However, the caller is expecting a boolean
and the fix to appease Pytype is simple, so we do both.

51687:1eab9e40c0c8

        """
        now = currentTimeMillis()
        delay = _LISTENER_TIME
        next = now + delay
        last = now + timeout
        try:
            zeroconf.addListener(
                self, DNSQuestion(self.name, _TYPE_ANY, _CLASS_IN)
            )
            while (

                    next = now + delay
                    delay = delay * 2

                zeroconf.wait(min(next, last) - now)
                now = currentTimeMillis()
            result = 1
        finally:
            zeroconf.removeListener(self)

        return result

51688:25e7f9dcad0f

        """
        now = currentTimeMillis()
        delay = _LISTENER_TIME
        next = now + delay
        last = now + timeout
        result = False
        try:
            zeroconf.addListener(
                self, DNSQuestion(self.name, _TYPE_ANY, _CLASS_IN)
            )
            while (

                    next = now + delay
                    delay = delay * 2

                zeroconf.wait(min(next, last) - now)
                now = currentTimeMillis()
            result = True
        finally:
            zeroconf.removeListener(self)

        return result