Resolve latest comments

Author: Agent-Hellboy

Lib/configparser.py

@@ -758,7 +758,6 @@ def read(self, filenames, encoding=None):
             if isinstance(filename, os.PathLike):
                 filename = os.fspath(filename)
             read_ok.append(filename)
-            self._loaded_sources.append(read_ok)
         return read_ok
 
     def read_file(self, f, source=None):
@@ -775,7 +774,6 @@ def read_file(self, f, source=None):
             except AttributeError:
                 source = '<???>'
         self._read(f, source)
-        self._loaded_sources.append(source)
 
     def read_string(self, string, source='<string>'):
         """Read configuration from a given string."""
@@ -812,7 +810,6 @@ def read_dict(self, dictionary, source='<dict>'):
                     raise DuplicateOptionError(section, key, source)
                 elements_added.add((section, key))
                 self.set(section, key, value)
-        self._loaded_sources.append(source)
 
     def get(self, section, option, *, raw=False, vars=None, fallback=_UNSET):
         """Get an option value for a given section.
@@ -1060,7 +1057,7 @@ def __str__(self):
         return f"<ConfigParser: {config_dict}>"
 
     def __repr__(self):
-        init_params = {
+        params = {
             "defaults": self._defaults if self._defaults else None,
             "dict_type": type(self._dict).__name__,
             "allow_no_value": self._allow_no_value,
@@ -1069,20 +1066,20 @@ def __repr__(self):
             "default_section": self.default_section,
             "interpolation": type(self._interpolation).__name__,
         }
-        init_params = {k: v for k, v in init_params.items() if v is not None}
+        params = {k: v for k, v in params.items() if v is not None}
         sections_count = len(self._sections)
-        state_summary = {
+        state = {
             "loaded_sources": self._loaded_sources,
             "sections_count": sections_count,
             "sections": list(self._sections)[:5],  # limit to 5 section names for readability
         }
 
         if sections_count > 5:
-            state_summary["sections_truncated"] = f"...and {sections_count - 5} more"
+            state["sections_truncated"] = f"...and {sections_count - 5} more"
 
         return (f"<{self.__class__.__name__}("
-                f"params={init_params}, "
-                f"state={state_summary})>")
+                f"params={params}, "
+                f"state={state})>")
 
     def _read(self, fp, fpname):
         """Parse a sectioned configuration file.
@@ -1104,6 +1101,7 @@ def _read(self, fp, fpname):
         try:
             ParsingError._raise_all(self._read_inner(fp, fpname))
         finally:
+            self._loaded_sources.append(fpname)
             self._join_multiline_values()
 
     def _read_inner(self, fp, fpname):