[8u] Request for approval for 8209184 - JCK Test Failure due to ResourceBundle

[Adam Farley8]

Hi Rob,

The webrev has been attached to the bug.

https://bugs.openjdk.java.net/browse/JDK-8209184

Best Regards

Adam Farley 

Rob McKenna <rob.mckenna at oracle.com> wrote on 20/08/2018 17:43:54:

> From: Rob McKenna <rob.mckenna at oracle.com>
> To: Adam Farley8 <adam.farley at uk.ibm.com>
> Cc: jdk8u-dev at openjdk.java.net
> Date: 20/08/2018 17:44
> Subject: Re: [8u] Request for approval for 8209184 - JCK Test 
> Failure due to ResourceBundle
> 
> Will do. Please send on the exported changeset / patch.
> 
>     -Rob
> 
> On 20/08/18 17:06, Adam Farley8 wrote:
> > Thanks Rob,
> > 
> > Could you merge this change?
> > 
> > Best Regards
> > 
> > Adam Farley 
> > 
> > Rob McKenna <rob.mckenna at oracle.com> wrote on 20/08/2018 14:15:26:
> > 
> > > From: Rob McKenna <rob.mckenna at oracle.com>
> > > To: Adam Farley8 <adam.farley at uk.ibm.com>
> > > Cc: jdk8u-dev at openjdk.java.net
> > > Date: 20/08/2018 14:15
> > > Subject: Re: [8u] Request for approval for 8209184 - JCK Test 
> > > Failure due to ResourceBundle
> > > 
> > > Approved
> > > 
> > >     -Rob
> > > 
> > > On 20/08/18 09:52, Adam Farley8 wrote:
> > > > Hi All,
> > > > 
> > > > I'd like to request approval for a change to 8u.
> > > > 
> > > > Bug: INVALID URI REMOVED
> > > 
> > 
> 
u=https-3A__bugs.openjdk.java.net_browse_JDK-2D8209184&d=DwIBAg&c=jf_iaSHvJObTbx-
> > > siA1ZOg&r=P5m8KWUXJf-
> > > CeVJc0hDGD9AQ2LkcXDC0PMV9ntVw5Ho&m=xcKHLoEswAd9iH6RCOGc--
> > > 
WjZydRfB5A3bda8nYwESk&s=s4MoyEgTyRZ4SfFmdTrbSCdE3guyhyA3kh5vdrYZuoU&e=
> > > > 
> > > > Latest email and review: 
> > > > INVALID URI REMOVED
> > > 
> > 
> 
u=http-3A__mail.openjdk.java.net_pipermail_i18n-2Ddev_2018-2DAugust_002619.html&d=DwIBAg&c=jf_iaSHvJObTbx-
> > > siA1ZOg&r=P5m8KWUXJf-
> > > CeVJc0hDGD9AQ2LkcXDC0PMV9ntVw5Ho&m=xcKHLoEswAd9iH6RCOGc--
> > > 
WjZydRfB5A3bda8nYwESk&s=XNeCUIDsdXLveqIV2uZziEUTmx0NK6RUrqP2RL0tFbk&e=
> > > > 
> > > > Latest form of the change (where the file is 
> > > > jdk/src/share/classes/java/util/ResourceBundle.java):
> > > > 
> > > > ------------------------------------------------------
> > > > @@ -1398,10 +1398,18 @@
> > > >              bundle = baseBundle;
> > > >          }
> > > > 
> > > > +        keepAlive(loader);
> > > >          return bundle;
> > > >      }
> > > > 
> > > >      /**
> > > > +     * Keeps the argument ClassLoader alive.
> > > > +     */
> > > > +    private static void keepAlive(ClassLoader loader){
> > > > +        // Do nothing.
> > > > +    }
> > > > +
> > > > +    /**
> > > >       * Checks if the given <code>List</code> is not null, not 
empty,
> > > >       * not having null in its elements.
> > > >       */
> > > > ------------------------------------------------------
> > > > 
> > > > Best Regards
> > > > 
> > > > Adam Farley 
> > > > OpenJDK Team 
> > > > Runtimes 
> > > > IBM 
> > > > 
> > > > Unless stated otherwise above:
> > > > IBM United Kingdom Limited - Registered in England and Wales with 
> > number 
> > > > 741598. 
> > > > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire 
PO6 
> > 3AU
> > > 
> > 
> > Unless stated otherwise above:
> > IBM United Kingdom Limited - Registered in England and Wales with 
number 
> > 741598. 
> > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 
3AU
> 

Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 
741598. 
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU