Feature #21552: allow String.strip and similar to take a parameter similar to String.delete

• Status: Open
• Assignee: -
• Target version: -

[MSP-Greg (Greg L)]

Regrading String.strip (and lstrip, rstrip, and ! versions)

Some text data representations differentiate between what one might call vertical and horizontal white space, and the 'strip' methods currently strip both.

It would be helpful if they had an optional parameter similar to String.delete with a one multi-character selector, so one could do:

t = str.strip " \t"

One can use a regex for this, but this much simpler.

[Dan0042 (Daniel DeLorme)]

Agreed. I tend to use str.sub(/[\ \t]+\z/,'') for this, but an end-anchored regexp has pretty bad worst-case performance. Try to benchmark the previous when str = " "*1000+"a" 😦

[mame (Yusuke Endoh)]

• Related to Feature #7845: Strip doesn't handle unicode space characters in ruby 1.9.2 & 1.9.3 (does in 1.9.1) added