Use functools.cache in redirects script.

QuLogic · QuLogic · commit 6fcc3b744733 · 2021-02-09T18:47:48.000-05:00
diff --git a/_websiteutils/make_redirects_links.py b/_websiteutils/make_redirects_links.py
@@ -1,6 +1,7 @@
 #!/usr/bin/env python3
 
 import argparse
+import functools
 import logging
 import multiprocessing
 import os
@@ -53,22 +54,17 @@
 logging.basicConfig(level=logging.DEBUG)
 
 
-# beware of triksy mutable defaults!
-def findlast(fname, tocheck, *, _cache={}):
+@functools.cache
+def findlast(fname, tocheck):
     """
     Check the directories listed in ``tocheck`` to see if they have
     ``fname`` in them.  Return the first one found, or None
     """
-    if fname in _cache:
-        return _cache[fname]
     for t in tocheck:
         pnew = t / fname
         if pnew.exists():
-            _cache[fname] = t
             return t
-    else:
-        _cache[fname] = None
-        return None
+    return None
 
 
 html_redirect = """<!DOCTYPE HTML>
