[LANG-748] Change StringUtils.chomp(String, String) to be the same as StringUtils.removeEnd

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.1
• Assignee: Unassigned
• Reporter: Henri Yandell
• Created: 26/Aug/11 04:51
• Updated: 26/Aug/11 05:19
• Resolved: 26/Aug/11 05:19

Description

Verneri Åberg reported on the list:

I was just browsing through StringUtils Api (versions 2.5, 2.6 and
3.0.1) and found two very similar looking methods

chomp(String,String)
and
removeEnd(String, String)

So I started to wonder what is the difference here and looked at the
source code.

To my astonishment the only difference is that chomp returns the source
string immediately if the removed string is null and remove end
additionally checks for empty strings too.

So wouldn't it be better to substitute the duplication by simply
replacing the chomp method with following version?

public String chomp(String source, String separator) {
return removeEnd(source,separator);
}

Or is there some hidden idea for two different implementations of the
same string chompping function? if there is maybe it should be added to
both methods javadocs?