[Patches] [ python-Patches-1551339 ] Fix for rpartition() end-case
SourceForge.net
noreply at sourceforge.net
Tue Sep 5 00:20:23 CEST 2006
Patches item #1551339, was opened at 2006-09-03 01:48
Message generated for change (Comment added) made by rhettinger
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1551339&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: Core (C code)
Group: Python 2.5
Status: Open
Resolution: None
Priority: 7
Submitted By: Raymond Hettinger (rhettinger)
Assigned to: Neal Norwitz (nnorwitz)
Summary: Fix for rpartition() end-case
Initial Comment:
Per discussion on python-dev
----------------------------------------------------------------------
>Comment By: Raymond Hettinger (rhettinger)
Date: 2006-09-04 17:20
Message:
Logged In: YES
user_id=80475
Neal, please move this from the head to the release branch
when you're ready. Nick, thanks for the review.
----------------------------------------------------------------------
Comment By: Nick Coghlan (ncoghlan)
Date: 2006-09-04 08:07
Message:
Logged In: YES
user_id=1038590
Aside from the tests, the patch looks OK. (A search through
the standard library showed that the only current use of
rpartition() is in UserString.py, which will correctly
inherit the behaviour of the underlying string type).
For the tests, the degenerate cases are already tested
(second test in each group), so new test cases aren't
necessary. Instead, the expected result for the degenerate
rpartition test needs to be updated. (Also, the second test
case added by the patch invoked partition() instead of
rpartition())
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1551339&group_id=5470
More information about the Patches
mailing list