Message50720
Logged In: YES
user_id=340931
Thanks for the patch Ken! This change sounds reasonable and
low-risk. However, some patch suggestions:
1. Diff against HEAD of trunk/ from svn, not your platform's
python installation
2. Add some tests in Lib/test/test_cfgparser.py (this will
be in a source distribution from svn, but not in your
platform's binary distribution)
3. Update the doc in Doc/lib/libcfgparser.tex (again, this
will be in a source dist)
FWIW, these are mentioned in the (admittedly somewhat
obtuse) patch guidelines:
http://www.python.org/dev/patches/
|
|
Date |
User |
Action |
Args |
2007-08-23 15:53:38 | admin | link | issue1524825 messages |
2007-08-23 15:53:38 | admin | create | |
|