refactor: share regexlist code between the two config formats

Author: nedbat

coverage/config.py

@@ -132,18 +132,7 @@ def getregexlist(self, section: str, option: str) -> list[str]:
 
         """
         line_list = self.get(section, option)
-        value_list = []
-        for value in line_list.splitlines():
-            value = value.strip()
-            try:
-                re.compile(value)
-            except re.error as e:
-                raise ConfigError(
-                    f"Invalid [{section}].{option} value {value!r}: {e}",
-                ) from e
-            if value:
-                value_list.append(value)
-        return value_list
+        return process_regexlist(section, option, line_list.splitlines())
 
 
 TConfigParser = Union[HandyConfigParser, TomlConfigParser]
@@ -557,6 +546,20 @@ def debug_info(self) -> list[tuple[str, Any]]:
         )
 
 
+def process_regexlist(name: str, option: str, values: list[str]) -> list[str]:
+    """Check the values in a regex list and keep the non-blank ones."""
+    value_list = []
+    for value in values:
+        value = value.strip()
+        try:
+            re.compile(value)
+        except re.error as e:
+            raise ConfigError(f"Invalid [{name}].{option} value {value!r}: {e}") from e
+        if value:
+            value_list.append(value)
+    return value_list
+
+
 def config_files_to_try(config_file: bool | str) -> list[tuple[str, bool, bool]]:
     """What config files should we try to read?
 

coverage/tomlconfig.py

@@ -11,7 +11,7 @@
 from typing import Any, Callable, TypeVar
 from collections.abc import Iterable
 
-from coverage import env
+from coverage import config, env
 from coverage.exceptions import ConfigError
 from coverage.misc import import_third_party, isolate_module, substitute_variables
 from coverage.types import TConfigSectionOut, TConfigValueOut
@@ -191,13 +191,7 @@ def getlist(self, section: str, option: str) -> list[str]:
 
     def getregexlist(self, section: str, option: str) -> list[str]:
         name, values = self._get_list(section, option)
-        for value in values:
-            value = value.strip()
-            try:
-                re.compile(value)
-            except re.error as e:
-                raise ConfigError(f"Invalid [{name}].{option} value {value!r}: {e}") from e
-        return values
+        return config.process_regexlist(name, option, values)
 
     def getint(self, section: str, option: str) -> int:
         name, value = self._get_single(section, option)