homepage

This issue tracker has been migrated to GitHub , and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author lukasz.langa
Recipients BreamoreBoy, aptshansen, brett.cannon, brian.curtin, eric.araujo, georg.brandl, lukasz.langa, michael.foord, tseaver
Date 2010年07月28日.11:15:12
SpamBayes Score 6.741082e-05
Marked as misclassified No
Message-id <1280315726.51.0.0673668069373.issue1682942@psf.upfronthosting.co.za>
In-reply-to
Content
New patch after review by Éric Araujo. The difference between the last one and the current is cosmetic:
--- Lib/configparser.py 2010年07月27日 11:36:51.000000000 +0200
+++ Lib/configparser.py.2 2010年07月28日 13:05:39.000000000 +0200
@@ -117,3 +117,2 @@
 import re
-import sre_parse
 import sys
@@ -140,2 +139,3 @@
 BaseException."""
+
 return self.__message
@@ -145,2 +145,3 @@
 BaseException."""
+
 self.__message = value
@@ -301,2 +302,3 @@
 """
+
 if section.lower() == "default":
@@ -338,2 +340,3 @@
 """
+
 if isinstance(filenames, str):
@@ -358,4 +361,4 @@
 used.
-
 """
+
 if filename is None:
@@ -419,2 +422,3 @@
 """Check for the existence of a given option in a given section."""
+
 if not section or section == DEFAULTSECT:
@@ -431,2 +435,3 @@
 """Set an option."""
+
 if not section or section == DEFAULTSECT:
@@ -444,2 +449,3 @@
 between keys and values are surrounded by spaces."""
+
 if space_around_delimiters:
@@ -456,2 +462,3 @@
 """Remove an option."""
+
 if not section or section == DEFAULTSECT:
@@ -471,2 +478,3 @@
 """Remove a file section."""
+
 existed = section in self._sections
@@ -529,2 +537,3 @@
 """
+
 cursect = None # None, or a dictionary
@@ -657,2 +666,3 @@
 """
+
 d = self._defaults.copy()
@@ -690,2 +700,3 @@
 """
+
 d = self._defaults.copy()
@@ -789,2 +800,3 @@
 """Set an option. Extend ConfigParser.set: check for string values."""
+
 # The only legal non-string value if we allow valueless
Some remarks on Éric's review:
- thanks for reviewing the patch, however you did review an outdated version
- comment_prefixes are by all means prefixes. Collins dictionary: "prefix, n. - 2. something coming or placed before". Other reviewers didn't pose the current name as inapt
- other remarks corrected in the current patch
History
Date User Action Args
2010年07月28日 11:15:26lukasz.langasetrecipients: + lukasz.langa, brett.cannon, georg.brandl, tseaver, aptshansen, eric.araujo, michael.foord, brian.curtin, BreamoreBoy
2010年07月28日 11:15:26lukasz.langasetmessageid: <1280315726.51.0.0673668069373.issue1682942@psf.upfronthosting.co.za>
2010年07月28日 11:15:25lukasz.langalinkissue1682942 messages
2010年07月28日 11:15:23lukasz.langacreate

AltStyle によって変換されたページ (->オリジナル) /