Re: AssertionError in WildcardTypeImpl.getUpperBoundASTs

2017-10-17 Thread Peter Levart
Hi Dawid, The [2] mentions a thread which resulted in a patch by Martin Bucholtz for JDK 9: https://bugs.openjdk.java.net/browse/JDK-8065172 ...but it seems this has not been backported to 8u (yet). The question is, will there be any more 8u releases? Would you like this to be backported to

Re: AssertionError in WildcardTypeImpl.getUpperBoundASTs

2017-10-17 Thread Dawid Weiss
Hi Peter, Thanks for the pointer. Whether or not this fix should be backported to 8u is probably not for me to decide... but if I can provide some rationale then I think it'd be better to backport for the following reasons: - This assertion isn't frequent, but it does happen from time to time. Ro

Re: RFR: 8188858: Caching latestUserDefinedLoader() results in ObjectInputStream.readObject()

2017-10-17 Thread Kazunori Ogata
Hi Peter, Thank you for your comments and the fix. It's a good idea to mark cachedLoader with the Thread object. I think we need to check the marking thread of cachedLoader before updating it. Otherwise, there is a scenario that can leak a CachedLoader object: //1. Thread-A enters readObje

Re: AssertionError in WildcardTypeImpl.getUpperBoundASTs

2017-10-17 Thread Peter Levart
Hi Dawid, So if we want this to be included in 8u162 [1] (to be released on January 2018), we better hurry up and submit a backported patch before end of October... I can sponsor the patch. Will you be willing to prepare it or do you need some help? I think it should be a straightforward b