break lots before other checks, minor phrasing improvement in docstring

Author: jakkdl

src/trio/_core/_run.py

@@ -1821,6 +1821,11 @@ async def python_wrapper(orig_coro: Awaitable[RetT]) -> RetT:
         return task
 
     def task_exited(self, task: Task, outcome: Outcome[Any]) -> None:
+        if task in GLOBAL_PARKING_LOT_BREAKER:
+            for lot in GLOBAL_PARKING_LOT_BREAKER[task]:
+                lot.break_lot(task)
+            del GLOBAL_PARKING_LOT_BREAKER[task]
+
         if (
             task._cancel_status is not None
             and task._cancel_status.abandoned_by_misnesting
@@ -1860,12 +1865,6 @@ def task_exited(self, task: Task, outcome: Outcome[Any]) -> None:
             assert task._parent_nursery is not None, task
             task._parent_nursery._child_finished(task, outcome)
 
-        # before or after the other stuff in this function?
-        if task in GLOBAL_PARKING_LOT_BREAKER:
-            for lot in GLOBAL_PARKING_LOT_BREAKER[task]:
-                lot.break_lot(task)
-            del GLOBAL_PARKING_LOT_BREAKER[task]
-
         if "task_exited" in self.instruments:
             self.instruments.call("task_exited", task)
 

src/trio/_sync.py

@@ -783,7 +783,7 @@ async def acquire(self) -> None:
         """Acquire the underlying lock, blocking if necessary.
 
         Raises:
-          BrokenResourceError: if the owner of the lock exits without releasing.
+          BrokenResourceError: if the owner of the underlying lock exits without releasing.
         """
         await self._lock.acquire()