Agreed :) Nice catch.
I'll also deprecate the chomp(String,String) version for removal in 4.0.
Hen
On Tue, Aug 23, 2011 at 11:02 PM, Verneri Åberg <verneri.ab...@gmail.com> wrote:
> Hi all,
>
> 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?
>
> Verneri
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org
