You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Steve Downey <st...@netfolio.com> on 2002/09/26 23:58:05 UTC

[lang] [PATCH] StringUtils documentation

Added @throws for methods that throw exceptions, and brought docs in line with 
actual behavior for a couple of methods. In particular String.trim() doesn't 
actually trim just whitespace (!). I suspect poor original coding, then being 
stuck with it.

WRT deleteWhitespace(String), I propose that this be broken into two methods, 
deleteSpace() and deleteWhitespace(), which would cut characters based on 
isSpace() and isWhitespace() respectively?

This patch just documents the existing behavior.



--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>


Re: [lang] [PATCH] StringUtils documentation

Posted by Steve Downey <st...@netfolio.com>.
Tried to get this out before being called to dinner. Should have waited til 
after. Patch attached.

On Thursday 26 September 2002 05:58 pm, Steve Downey wrote:
> Added @throws for methods that throw exceptions, and brought docs in line
> with actual behavior for a couple of methods. In particular String.trim()
> doesn't actually trim just whitespace (!). I suspect poor original coding,
> then being stuck with it.
>
> WRT deleteWhitespace(String), I propose that this be broken into two
> methods, deleteSpace() and deleteWhitespace(), which would cut characters
> based on isSpace() and isWhitespace() respectively?
>
> This patch just documents the existing behavior.