Message45560
Logged In: YES
user_id=995755
Updated the patch as follows:
1) added optional line wrapping using a wrapcolumn argument
to make_file and make_table methods.
2) eliminated a number of optional arguments to simplify API
(as discussed).
3) made everything protected (by naming convention) except
for the public make_table and make_file methods (as
discussed).
NOTE1
I've attached the patch (you will also need the new file
test_difflib_expect.html). I've zipped the patched code as
well as example side by side differences of the patch. Diffs
ending with _UPDATE.html show differences between the
patched code and the last version of the patched code (what
I've done since my last posting). Diffs ending with
_PATCH.html show differences between the patched code
and what I obtained from CVS a month or so back:
python/python/dist/src/Lib/difflib.py -- rev 1.21
python/python/dist/src/Tools/scripts/diff.py -- rev 1.2
python/python/dist/src/Lib/test/test_difflib.py -- rev 1.10
python/python/dist/src/Doc/lib/libdifflib.tex -- rev 1.17
NOTE2
I will not be monitoring emails or this patch from Saturday
Aug21 thru Wednesday Aug25. When I break my internet
silence I'll see if I've blundered this update (unlikely because I
tested it pretty well but none the less possible). If
everything went well I will be soliciting Python-Dev to try to
get this in alpha3. I need someone with checkin privledges to
do it. Should I be trying to gather more support for its
inclusion or does this only need to be done to convince
someone with checkin privledges to apply the patch? |
|
Date |
User |
Action |
Args |
2007-08-23 15:36:27 | admin | link | issue914575 messages |
2007-08-23 15:36:27 | admin | create | |
|