tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Konstantin Kolinko <knst.koli...@gmail.com>
Subject Re: Adding utility classes
Date Fri, 03 Dec 2010 16:46:22 GMT
2010/12/3 Christopher Schultz <chris@christopherschultz.net>:
> All,
>
> There's an annoyance bug
> (https://issues.apache.org/bugzilla/show_bug.cgi?id=49125) that requires
> a utility to convert byte arrays into readable strings. I have a patch
> that includes a method to do that, but I'm unsure of where that
> "toHexString" method should go.
>
> For now, I have placed it into
> java/org/apache/tomcat/util/buf/HexUtils.java because that's where I
> found other hex-encoding utility methods.
>
> Is that an appropriate place? My only concern is that it's in the
> "utils/buf" package when it's not necessarily targeted at the message
> buffer code.
>

Personally, I do not really care where it is.  An important thing is
into what jar the class goes. We keep the jasper self-contained in 5.5
and 6.0. In TC7 we allow it to depend on the classes packed into
tomcat-util.jar.

Don't forget to set svn:eol-style=native on the added file.

Best regards,
Konstantin Kolinko

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message