From: Max M. <max...@ma...> - 2002-09-30 18:56:46
|
Hi Camille, Thanks for the patch! Yesterday I actually started breaking all of the encoding and decoding methods out into an ERXEOEncodingUtilities class (adding in a few nice missing ones). I'll add your changes into ERXDirectActionHyperlink and see if we can't slim down the .java file a bit. Regards, Max On Monday, September 30, 2002, at 10:22 AM, Camille Troillard wrote: > Hi Max, > > As we agreed, attached is a new version of ERXDirectActionHyperlink. > I've tried to clean and comment as much as possible, but this class is > already very long and I hope you'll find a place to move some code > somewhere else (you seemed to have idea about this). > > I think the name of the class is long as well. Without breaking > anything, we could have a class for static methods and implement > ERXDirectActionHyperlink with those methods. > > Regards, > Camille > > P.S.: I CC'ed the list for those who are curious and willing to do > comments ... > > > <ERXDirectActionHyperlink.java> |