+1 for table-york. -1 display09 I believe that table-york is not that great of a departure from Ed's original work. That is a good thing. I believe that it can be used immediately to begin building out enhancements. The decision is not made just because John and I are both working in Ann Arbor, MI. The code shows restraint to change too much at once. It makes correct use of Interfaces, although not enough of them yet.
Overall comments on Fabrizio's module: ** I have edited out the trivial comments to shorten this email. It was too much to handle in its longer form. The key points are found below: ** This module is not ready to be built upon, while table-york is. There are not enough interfaces (only ColumnDecorator is an appropriate use). Josh Bloch's book "Effective Java" explains why implementing an interface to get access to constants is a bad practice. I think he mentions it on the server side in his JDK 1.5 talk as well. It is worth listening to. http://www.theserverside.com/events/index.jsp I remember reading John York's comment that it seems unduly complex. I second that opinion. I do like the effort that has been put into it although. It appears that the intent to develop an API is driving the package names and build out. Enums. Again, they are coming in 1.5 and Josh Bloch talks about them at the server side and did at JavaOne too. Why can't we wait for the official version instead of using commons-lang? It seems to me an unnecessary addition. Frankly, it would be painful to me to have to "upgrade" to display09. I don't want to use this tree. It is not superior to the original work that Ed and Jay did. It is a great effort but misses the mark. Ben ------------------------------------------------------- This SF.net email is sponsored by: VM Ware With VMware you can run multiple operating systems on a single machine. WITHOUT REBOOTING! Mix Linux / Windows / Novell virtual machines at the same time. Free trial click here: http://www.vmware.com/wl/offer/345/0 _______________________________________________ displaytag-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/displaytag-devel
