You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@cayenne.apache.org by Andrus Adamchik <an...@objectstyle.org> on 2008/08/21 21:31:44 UTC

Re: svn commit: r687826 - /cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/conn/DataSourceInfo.java

On Aug 21, 2008, at 9:39 PM, mgentry@apache.org wrote:

> -            return (PasswordEncoding) Util
> -                    .getJavaClass(getPasswordEncoderClass())
> -                    .newInstance();
> +            return (PasswordEncoding) Thread.currentThread()
> +                                            .getContextClassLoader()
> +
>                                             .loadClass
> (getPasswordEncoderClass())
> +                                            .newInstance();

Util.getJavaClass is more environment agnostic than  
Thread.currentThread().getContextClassLoader(), which is used  
internally as the first strategy. So maybe this can be undone?

Andrus

Re: svn commit: r687826 - /cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/conn/DataSourceInfo.java

Posted by Michael Gentry <bl...@gmail.com>.

I changed it.

Thanks!

On Thu, Aug 21, 2008 at 3:31 PM, Andrus Adamchik <an...@objectstyle.org> wrote:
>
> On Aug 21, 2008, at 9:39 PM, mgentry@apache.org wrote:
>
>> -            return (PasswordEncoding) Util
>> -                    .getJavaClass(getPasswordEncoderClass())
>> -                    .newInstance();
>> +            return (PasswordEncoding) Thread.currentThread()
>> +                                            .getContextClassLoader()
>> +
>>                                            .loadClass
>> (getPasswordEncoderClass())
>> +                                            .newInstance();
>
> Util.getJavaClass is more environment agnostic than
> Thread.currentThread().getContextClassLoader(), which is used internally as
> the first strategy. So maybe this can be undone?
>
> Andrus
>