Jason van Zyl <[EMAIL PROTECTED]> writes: > On 1/15/02 8:04 AM, "[EMAIL PROTECTED]" <[EMAIL PROTECTED]> wrote: > >> dlr 02/01/15 05:04:14 >> >> Modified: src/java/org/apache/stratum/xo Mapper.java >> Log: >> o Clarified JavaDoc for basePackage instance member. >> >> o Added map(String, String) overload which (again) tries reading from >> the classpath before going to the file system. Now wrappers >> InputStream overload (which incidently may not get a usable value for >> basePath if called directly). > > I addressed this in a previous email. The logic for searching for resources > in different sources shouldn't be located in the mapper. If we add pulling > configurations from an LDAP server than this class will get convoluted > rather quickly. > > Again, I believe I mentioned at the meeting that the resources package would > be used for I/O strategies. I would like to agree on this which is why I > brought it up at the meeting.
Like I said in the last mail, it wasn't available when I wrote this. If you're going to change the API, you need to change the references as well. I put in a lot of work to get classpath resource loading working smoothly, and already count on it in SourceCast -- please do not remove the classpath loading without modifying jakarta-turbine-3 (your changes broke stuff). Thanks. > I will get the resources package working with Turbine so that a WAR works in > a consistent way by finding resources using the same mechanism. Cool! That's what I'm sayin'. Dan -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
