Bug in String rstrip method

[rcouplan]

BPO	1104923
Nosy	@tim-one

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2005-01-19.01:56:23.000>
created_at = <Date 2005-01-19.00:58:34.000>
labels = ['invalid', 'library']
title = 'Bug in String rstrip method'
updated_at = <Date 2005-01-19.01:56:23.000>
user = 'https://bugs.python.org/rcouplan'

bugs.python.org fields:

activity = <Date 2005-01-19.01:56:23.000>
actor = 'tim.peters'
assignee = 'none'
closed = True
closed_date = None
closer = None
components = ['Library (Lib)']
creation = <Date 2005-01-19.00:58:34.000>
creator = 'rcouplan'
dependencies = []
files = []
hgrepos = []
issue_num = 1104923
keywords = []
message_count = 3.0
messages = ['23978', '23979', '23980']
nosy_count = 3.0
nosy_names = ['tim.peters', 'dsm001', 'rcouplan']
pr_nums = []
priority = 'normal'
resolution = 'not a bug'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue1104923'
versions = []

[rcouplan]

There appears to be a data sensitive bug in the string rstrip 
method as demonstrated by the following code: 
 
>>> "ecigrcal.fle".rstrip(".fle") 
'ecigrca' 
 
As you can see, it is striping an extra character.

[dsm001]

Logged In: YES
user_id=1175690

I don't think this is a bug. The documentation for the
rstrip method reads:

rstrip(...)
    S.rstrip([chars]) -> string or unicode
    
    Return a copy of the string S with trailing whitespace
removed.
    If chars is given and not None, remove characters in
chars instead.

When you pass the method ".rle", you're telling it to treat
the four characters ".", "r", "l", and "e" as trailing
whitespace and remove them from the right side of the
string. That's why it removes the extra "l".

[tim-one]

Logged In: YES
user_id=31435

Yes, dsm001 is right. The argument is treated as a _set_ of
characters to be stripped, not as a literal string. This is a
natural generalization of .rstrip() without any characters,
which chops off all whitespace characters (no matter how
many there, and no matter what order they appear in).
Passing an argument just changes the set of characters it
deletes.