Feb 24, 2022 · Feb 22, 2022 · Feb 24, 2022 · Feb 24, 2022 · Feb 24, 2022 · Feb 23, 2022
diff --git a/Lib/asyncio/tasks.py b/Lib/asyncio/tasks.py
        task will be marked as cancelled when the wrapped coroutine
        terminates with a CancelledError exception (even if cancel()
        was not called).

        This also increases the task's count of cancellation requests.
        """
        self._log_traceback = False
        if self.done():
        return True

    def cancelling(self):
        """Return the count of the task's cancellation requests.

        This count is incremented when .cancel() is called
        and may be decremented using .uncancel().
        """
        return self._num_cancels_requested

    def uncancel(self):
        """Decrement the task's count of cancellation requests.

        This should be used by tasks that catch CancelledError
        and wish to continue indefinitely until they are cancelled again.

        Returns the remaining number of cancellation requests.
        """
        if self._num_cancels_requested > 0:
            self._num_cancels_requested -= 1
        return self._num_cancels_requested
diff --git a/Modules/_asynciomodule.c b/Modules/_asynciomodule.c
 task will be marked as cancelled when the wrapped coroutine
 terminates with a CancelledError exception (even if cancel()
 was not called).

 This also increases the task's count of cancellation requests.
 [clinic start generated code]*/

 static PyObject *
 /*[clinic input]
 _asyncio.Task.cancelling

 Return True if the task is in the process of being cancelled.

 This is set once .cancel() is called
 and remains set until .uncancel() is called.
 Return the count of the task's cancellation requests.

 As long as this flagisset, further .cancel()calls will be ignored,
 until .uncancel() is called to reset it.
 This countisincremented when .cancel()is called
 and may be decremented using .uncancel().
 [clinic start generated code]*/

 static PyObject *
 /*[clinic input]
 _asyncio.Task.uncancel

 Reset theflag returned by cancelling().
 Decrement thetask's count of cancellation requests.

 This should be used by tasks that catch CancelledError
 and wish to continue indefinitely until they are cancelled again.

 Returns theprevious value ofthe flag.
 Returns theremaining number ofcancellation requests.
 [clinic start generated code]*/

 static PyObject *
Original file line number	Diff line number	Diff line change
Expand Up		@@ -198,6 +198,8 @@ def cancel(self, msg=None):
		task will be marked as cancelled when the wrapped coroutine
		terminates with a CancelledError exception (even if cancel()
		was not called).

		This also increases the task's count of cancellation requests.
		"""
		self._log_traceback = False
		if self.done():
Expand All		@@ -217,9 +219,21 @@ def cancel(self, msg=None):
		return True

		def cancelling(self):
Copy link Contributor asvetlovFeb 23, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Is the`cancelling()` the best name for returning the number or cancel requests? It was good for bool flag, a counter needs better name IMHO. ambv reacted with thumbs up emoji Copy link Member gvanrossumFeb 23, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Hm, do you have a suggestion? I was thinking that it's still usable as a "truthy" value. Usually when you're interested in the cancel count you should call`uncancel()` and use its return value. Copy link Contributor asvetlovFeb 23, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. I have no good answer. `cancel_requests_count()` describes exact meaning but the name is too long. `cancelling()` can return bool but it looks like procrastination: why return less info than we really have. The third option is dropping`cancelling()` method entirely, it is not required by the current asyncio code. We can consider adding the method later on-demand. Copy link Contributor asvetlovFeb 23, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Note:`.cancelling()` doesn't exist in Python 3.10 Copy link Member gvanrossumFeb 23, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. It's used in TaskGroup currently, so if we were to delete it we'd have to rewrite the code there in this same PR. It's also used by`Task.__repr__()` (via`_task_repr_info()`). But of those will work even though we upgraded it from a bool to an int. I'd say let's keep it for now, if we regret it we can rename or remove it before beta 1. Copy link Contributor asvetlovFeb 23, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Deal!
		"""Return the count of the task's cancellation requests.

		This count is incremented when .cancel() is called
		and may be decremented using .uncancel().
		"""
		return self._num_cancels_requested

		def uncancel(self):
		"""Decrement the task's count of cancellation requests.

		This should be used by tasks that catch CancelledError
		and wish to continue indefinitely until they are cancelled again.

		Returns the remaining number of cancellation requests.
		"""
		if self._num_cancels_requested > 0:
		self._num_cancels_requested -= 1
		return self._num_cancels_requested
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -2191,6 +2191,8 @@ not return True (unless the task was already cancelled). A
		task will be marked as cancelled when the wrapped coroutine
		terminates with a CancelledError exception (even if cancel()
		was not called).

		This also increases the task's count of cancellation requests.
		[clinic start generated code]*/

		static PyObject *
Expand DownExpand Up		@@ -2238,13 +2240,10 @@ _asyncio_Task_cancel_impl(TaskObj self, PyObject msg)
		/*[clinic input]
		_asyncio.Task.cancelling

		Return True if the task is in the process of being cancelled.

		This is set once .cancel() is called
		and remains set until .uncancel() is called.
		Return the count of the task's cancellation requests.

		As long as this flagisset, further .cancel()calls will be ignored,
		until .uncancel() is called to reset it.
		This countisincremented when .cancel()is called
		and may be decremented using .uncancel().
		[clinic start generated code]*/

		static PyObject *
Expand All		@@ -2258,12 +2257,12 @@ _asyncio_Task_cancelling_impl(TaskObj *self)
		/*[clinic input]
		_asyncio.Task.uncancel

		Reset theflag returned by cancelling().
		Decrement thetask's count of cancellation requests.

		This should be used by tasks that catch CancelledError
		and wish to continue indefinitely until they are cancelled again.

		Returns theprevious value ofthe flag.
		Returns theremaining number ofcancellation requests.
		[clinic start generated code]*/

		static PyObject *
Expand Down