Plans to integrate class loading patch

Having problems with building or using the JCEF Java binding? Ask your questions here.

Plans to integrate class loading patch

Postby leha2000 » Tue Sep 01, 2015 11:10 am

Are there any plans to integrate class loading patch
https://bitbucket.org/chromiumembedded/ ... /issues/99

Another option is to cache the classloader
http://stackoverflow.com/questions/1326 ... ndroid-jni
leha2000
Newbie
 
Posts: 8
Joined: Mon Aug 31, 2015 2:23 pm

Re: Plans to integrate class loading patch

Postby leha2000 » Thu Sep 10, 2015 2:32 am

It would be nice to have this fix integrated to one of the next builds
The caching class loader on load looks the better option
leha2000
Newbie
 
Posts: 8
Joined: Mon Aug 31, 2015 2:23 pm

Re: Plans to integrate class loading patch

Postby magreenblatt » Thu Sep 10, 2015 3:19 am

See comments on the issue for why the patch is not acceptable. If you wish to develop an acceptable solution then it might be considered.
magreenblatt
Site Admin
 
Posts: 12409
Joined: Fri May 29, 2009 6:57 pm

Re: Plans to integrate class loading patch

Postby leha2000 » Fri Sep 18, 2015 3:49 am

magreenblatt wrote:See comments on the issue for why the patch is not acceptable. If you wish to develop an acceptable solution then it might be considered.

I saw only your comments about pre-loading all the classes.
What do you think about other approach: caching the class loader on JNI load and use it for class loading later?
leha2000
Newbie
 
Posts: 8
Joined: Mon Aug 31, 2015 2:23 pm

Re: Plans to integrate class loading patch

Postby magreenblatt » Fri Sep 18, 2015 4:24 am

leha2000 wrote:
magreenblatt wrote:See comments on the issue for why the patch is not acceptable. If you wish to develop an acceptable solution then it might be considered.

I saw only your comments about pre-loading all the classes.
What do you think about other approach: caching the class loader on JNI load and use it for class loading later?

I didn't see the class loader patch, thanks for pointing it out. I think that approach could be acceptable. Did you test it and does it work for you?
magreenblatt
Site Admin
 
Posts: 12409
Joined: Fri May 29, 2009 6:57 pm

Re: Plans to integrate class loading patch

Postby leha2000 » Fri Sep 25, 2015 2:18 am

magreenblatt wrote:
leha2000 wrote:
magreenblatt wrote:See comments on the issue for why the patch is not acceptable. If you wish to develop an acceptable solution then it might be considered.

I saw only your comments about pre-loading all the classes.
What do you think about other approach: caching the class loader on JNI load and use it for class loading later?

I didn't see the class loader patch, thanks for pointing it out. I think that approach could be acceptable. Did you test it and does it work for you?

The patch links are dead. So I can't test it :(
Need to create a new (by stackoverflow post) or ask the author
leha2000
Newbie
 
Posts: 8
Joined: Mon Aug 31, 2015 2:23 pm

Re: Plans to integrate class loading patch

Postby magreenblatt » Fri Sep 25, 2015 2:33 am

leha2000 wrote:The patch links are dead. So I can't test it :(

You need to click the papper-clip at the top of the issue to download the patch.
magreenblatt
Site Admin
 
Posts: 12409
Joined: Fri May 29, 2009 6:57 pm

Re: Plans to integrate class loading patch

Postby JavaProgramer » Sat Nov 07, 2015 5:59 pm

Class loading problem is commited and resolved.

Thank you magreenblatt and others resolving this issue
JavaProgramer
Newbie
 
Posts: 8
Joined: Sat Nov 30, 2013 10:05 am


Return to JCEF Forum

Who is online

Users browsing this forum: No registered users and 16 guests

cron