[DMARC FAILURE] RFR (XS): 8222510: Small cleanup for JDK launcher's make file
Langer, Christoph
christoph.langer at sap.com
Tue Apr 16 06:09:36 UTC 2019
Hi Man,
the change looks good to me, thanks for doing this cleanup.
As for reviewers: I thought it depends whether the author of a change is a Reviewer himself. Then only one Reviewer needs to review the change. But I might be wrong here. So, let's wait for a review from the build group and maybe these folks can answer this question, too.
Best regards
Christoph
> -----Original Message-----
> From: build-dev <build-dev-bounces at openjdk.java.net> On Behalf Of Man
> Cao
> Sent: Dienstag, 16. April 2019 04:04
> To: build-dev at openjdk.java.net
> Subject: [DMARC FAILURE] RFR (XS): 8222510: Small cleanup for JDK
> launcher's make file
>
> Hi,
>
> Can I have reviews for this small cleanup? It basically removes some dead
> and redundant variables and updates comment.
> Webrev: https://cr.openjdk.java.net/~manc/8222510/webrev.00/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8222510
>
> I found these issues while working on the tsan project (
> http://hg.openjdk.java.net/tsan/dev/). Some relevant changes previously
> touched the code include JDK-8210931, JDK-8199639, JDK-8141444.
>
> And do I need only one review for this change? It seems many build files
> changes only have one reviewer.
>
> -Man
More information about the build-dev
mailing list