From 512ee3ff9b25cdc74ab7ab3dc6ff96f65c160aca Mon Sep 17 00:00:00 2001
From: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
Date: Sun, 31 Mar 2024 23:13:30 -0600
Subject: [PATCH] Simplify code and update comments

Co-authored-by: Andrew Murray <3112309+radarhere@users.noreply.github.com>
---
 Tests/helper.py | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/Tests/helper.py b/Tests/helper.py
index 863bd87cfcb..5d477144d2f 100644
--- a/Tests/helper.py
+++ b/Tests/helper.py
@@ -258,20 +258,20 @@ def hopper(mode: str | None = None) -> Image.Image:
 
     if mode is None:
         # Always return fresh not-yet-loaded version of image.
-        # Operations on not-yet-loaded images is separate class of errors
-        # what we should catch.
+        # Operations on not-yet-loaded images are a separate class of errors
+        # that we should catch.
         return Image.open("Tests/images/hopper.ppm")
 
     return _cached_hopper(mode).copy()
 
 
-@lru_cache(maxsize=None)
-def _cached_hopper(mode: str = None) -> Image.Image:
+@lru_cache
+def _cached_hopper(mode: str) -> Image.Image:
     if mode == "F":
-        im = hopper("L").convert(mode)
+        im = hopper("L")
     else:
-        im = hopper().convert(mode)
-    return im
+        im = hopper()
+    return im.convert(mode)
 
 
 def djpeg_available() -> bool: