On 1/6/20 1:48 PM, Alex Kashchenko wrote: > On 01/06/2020 10:54 AM, Aleksey Shipilev wrote: >> On 1/6/20 10:57 AM, Alex Kashchenko wrote: >>> On 12/24/2019 11:54 AM, Alex Kashchenko wrote: >>>> On 12/24/2019 01:39 AM, Philip Race wrote: >>>>> Have you verified this with VS 2017 ? >>>>> Not that I can see a problem but I doubt we want to trade breaking >>>>> 2017 to fix 2013 ... >>>> >>>> Yes, VS 2017 compiles fine with this fix. >>>> >>>> >>>>> [...] >>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8236125 >>>>> Webrev: http://cr.openjdk.java.net/~akasko/jdk/8236125/webrev.00/ >>>>> [...] >>> >>> Gentle reminder, it would be great to have this included before jdk14 >>> enters rampdown 2. >> >> The patch looks good to me. (I did the same in the original bug report). > > Thanks for the review! Sponsor is required for this patch to be pushed > (I am not an author in "jdk" and cannot export committed changeset).
I can sponsor jdk/jdk14 push. But I need someone else to acknowledge (I don't feel fancy acking basically my own patch :]). Phil, maybe? -- Thanks, -Aleksey
