|
From: <jue...@we...> - 2003-11-13 07:46:03
|
Robert, Good point - just fixed it :-) We're using the same strategy in various = other parts of the framework, so it also makes sense to have ClassEditor = use that strategy for consistency reasons. On the occasion, I've made = all remaining framework classes use the thread context class loader too. Regarding the jars for building the source: There are quite a lot of = them, unfortunately, as we integrate with numerous third-party tools. = We've already considered offering an additional distribution zip that = contains all jars and also the test suite, at around 17 MB instead of = our current distro's 10 MB. We'd need to check the licenses of all third-party librarties first, = though, and adapt the distribution build scripts accordingly. This = hasn't been of high priority yet; so we'll probably still ship the = upcoming release 1.0 M3 in the current form, i.e. with one 10 MB zip. = Watch out for the 1.0 RC1 that will follow :-) Juergen -----Original Message----- From: spr...@li... [mailto:spr...@li...]On Behalf Of Robert Gothan Sent: Tuesday, November 18, 2003 11:27 AM To: spr...@li... Subject: [[W3-SPAM]] - [Springframework-developer] Class.forname - Email found in subject In class org.springframework.beans.propertyeditor.ClassEditor There appears to be a bug with the Class.forName(). Changing it to=20 clazz =3D Class.forName(text, true, =20 Thread.currentThread().getContextClassLoader()); makes it more reliable in J2EE environments. By the way, it would help if all the src jars were included in the = download,=20 with a build script! Cheers, Robert ------------------------------------------------------- This SF.Net email sponsored by: ApacheCon 2003, 16-19 November in Las Vegas. Learn firsthand the latest developments in Apache, PHP, Perl, XML, Java, MySQL, WebDAV, and more! http://www.apachecon.com/ _______________________________________________ Springframework-developer mailing list Spr...@li... https://lists.sourceforge.net/lists/listinfo/springframework-developer |