Yes, but you are a Committer, so that is fine in this case, since we have two Reviewers already :)
Thank you! I'll push now. ..Thomas On Wed, Jun 20, 2018 at 5:00 PM, Baesken, Matthias <matthias.baes...@sap.com> wrote: > Hi Thomas , looks good (however I am not a Reviewer) ! > > >> From: Thomas Stüfe <thomas.stu...@gmail.com> >> Date: Wed, Jun 20, 2018 at 10:00 AM >> Subject: RFR(xxs): 8205407: [windows, vs<2017] C4800 after 8203197 >> To: build-dev <build-dev@openjdk.java.net> >> >> >> Hi all, >> >> May I please have reviews for this small build fix? >> >> https://bugs.openjdk.java.net/browse/JDK-8205407 >> http://cr.openjdk.java.net/~stuefe/webrevs/8205497-windows- >> c4800/webrev.00/webrev/ >> >> In short, we need to disable C4800 for the JVM build as well since 8203197. >> >> Thank you, >> >> Thomas
