]> granicus.if.org Git - python/commitdiff
Issue #20505: BaseEventLoop uses again the resolution of the clock to decide if
authorVictor Stinner <victor.stinner@gmail.com>
Mon, 10 Feb 2014 22:42:32 +0000 (23:42 +0100)
committerVictor Stinner <victor.stinner@gmail.com>
Mon, 10 Feb 2014 22:42:32 +0000 (23:42 +0100)
scheduled tasks should be executed or not.

Lib/asyncio/base_events.py
Lib/test/test_asyncio/test_events.py

index 4207a7e9f1eb2e7ad0b160dab791f84db9646dc1..377ea216f7cd6ea69acc6501b4fcddcc26e79581 100644 (file)
@@ -96,6 +96,7 @@ class BaseEventLoop(events.AbstractEventLoop):
         self._default_executor = None
         self._internal_fds = 0
         self._running = False
+        self._clock_resolution = time.get_clock_info('monotonic').resolution
 
     def _make_socket_transport(self, sock, protocol, waiter=None, *,
                                extra=None, server=None):
@@ -643,10 +644,10 @@ class BaseEventLoop(events.AbstractEventLoop):
         self._process_events(event_list)
 
         # Handle 'later' callbacks that are ready.
-        now = self.time()
+        end_time = self.time() + self._clock_resolution
         while self._scheduled:
             handle = self._scheduled[0]
-            if handle._when > now:
+            if handle._when >= end_time:
                 break
             handle = heapq.heappop(self._scheduled)
             self._ready.append(handle)
index 2a01b1c954b2e717f82fba0fd917bad0c44fbe84..3f99da4c98174f2582c08b1478821081736f3be8 100644 (file)
@@ -1178,15 +1178,15 @@ class EventLoopTestsMixin:
             yield from asyncio.sleep(1e-4, loop=loop)
             yield from asyncio.sleep(1e-6, loop=loop)
             yield from asyncio.sleep(1e-8, loop=loop)
+            yield from asyncio.sleep(1e-10, loop=loop)
 
         self.loop.run_until_complete(wait())
-        # The ideal number of call is 10, but on some platforms, the selector
+        # The ideal number of call is 12, but on some platforms, the selector
         # may sleep at little bit less than timeout depending on the resolution
-        # of the clock used by the kernel. Tolerate 5 useless calls on these
-        # platforms.
-        self.assertLessEqual(self.loop._run_once_counter, 15,
-            {'time_info': time.get_clock_info('time'),
-             'monotonic_info': time.get_clock_info('monotonic'),
+        # of the clock used by the kernel. Tolerate a few useless calls on
+        # these platforms.
+        self.assertLessEqual(self.loop._run_once_counter, 20,
+            {'clock_resolution': self.loop._clock_resolution,
              'selector': self.loop._selector.__class__.__name__})