Patches item #1682942, was opened at 2007-03-18 02:54
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1682942&group_id=5470
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Library (Lib)
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Stephen Hansen (aptshansen)
Assigned to: Nobody/Anonymous (nobody)
Summary: ConfigParser support for alt delimiters
Initial Comment:
This patch is in response to 1165404, which I was reviewing. It turns out I
actually rather like the idea, since it would have made my life easier in a
current project to not have to totally roll my own ConfigParser to read an
external configuration file. So!
I updated the previous patch to apply against the current HEAD, and then added
some documentation and test cases (and made sure it didn't break existing test
cases)
I did alter the previously proposed API a touch too; 'delimiters' instead of
'delimiter' as its taking a list, and to actually take a sequence instead of a
string containing characters because IMHO it just looks cleaner. Passing in
":=" as indicating either : or = are delimiters looks odd ot me (Hi, Pascal),
it looks like it'd make the _string_ of ":=" the delimiter. I also adjusted the
names slightly to fit into the style of the module. ('_delimiter' instead of
'delim').
I also updated the docstring on the module to document dict_type being an
option (it wasn't before)
My text editor seems to think the documentation is OK LaTeX-- but until uh, two
days ago, I never touched LaTeX, so someone should eye that :)
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1682942&group_id=5470
_______________________________________________
Patches mailing list
[email protected]
http://mail.python.org/mailman/listinfo/patches
