[11u]: RFR(S): 8232370: Refactor some com.sun.jdi tests to enable IDE integration

Langer, Christoph christoph.langer at sap.com
Thu Dec 19 12:25:56 UTC 2019


Hi Goetz,

thanks for reviewing this.

Cheers
Christoph

> -----Original Message-----
> From: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>
> Sent: Donnerstag, 19. Dezember 2019 12:22
> To: Langer, Christoph <christoph.langer at sap.com>; jdk-updates-
> dev at openjdk.java.net
> Subject: RE: [11u]: RFR(S): 8232370: Refactor some com.sun.jdi tests to
> enable IDE integration
> 
> Hi Christoph,
> 
> change looks good.
> The test is a shell test in 11 and was reimplemented in Java in 14.
> I guess this will never be downported, so dropping the change
> is absolutely fine.
> 
> Best regards,
>   Geotz.
> 
> > -----Original Message-----
> > From: jdk-updates-dev <jdk-updates-dev-bounces at openjdk.java.net> On
> > Behalf Of Langer, Christoph
> > Sent: Donnerstag, 19. Dezember 2019 11:42
> > To: jdk-updates-dev at openjdk.java.net
> > Subject: [CAUTION] [11u]: RFR(S): 8232370: Refactor some com.sun.jdi
> tests to
> > enable IDE integration
> >
> > Hi,
> >
> > please review this backport of a test-only change to better support JDK11
> > development in the Eclipse IDE. The Original change applies cleanly for 2
> files
> > but test/jdk/com/sun/jdi/RedefineImplementor.java does not exist in 11u,
> so I
> > simply dropped this part.
> >
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8232370
> > Original Change: https://hg.openjdk.java.net/jdk/jdk/rev/5f14a659a8cb
> > Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8232370.11u/
> >
> > Thanks
> > Christoph



More information about the jdk-updates-dev mailing list