[11u] RFR: 8240487: Cleanup whitespace in .cc, .hh, .m, and .mm files

Vladimir Kempik vkempik at azul.com
Thu Apr 15 20:52:07 UTC 2021


Hello
thanks for notice, updated webrev - http://cr.openjdk.java.net/~vkempik/8240487/webrev.02/ <http://cr.openjdk.java.net/~vkempik/8240487/webrev.02/>

still the webrev isn’t the best tool for whitespace patches, to see white spaces need to select that area, and I can’t tell that webrev displays them correctly all the time.

Regards, Vladimir
> 15 апр. 2021 г., в 23:38, Hohensee, Paul <hohensee at amazon.com> написал(а):
> 
> The webrev is empty (i.e., reports the changed files, but not the changes). Use "webrev -b".
> 
> Thanks,
> Paul
> 
> -----Original Message-----
> From: jdk-updates-dev <jdk-updates-dev-retn at openjdk.java.net> on behalf of Vladimir Kempik <vkempik at azul.com>
> Date: Thursday, April 15, 2021 at 1:27 AM
> To: jdk-updates-dev <jdk-updates-dev at openjdk.java.net>
> Subject: RE: [11u] RFR: 8240487: Cleanup whitespace in .cc, .hh, .m, and .mm files
> 
> Correcting the title
> 
>> 15 апр. 2021 г., в 10:40, Vladimir Kempik <vkempik at azul.com> написал(а):
>> 
>> Hello
>> please review this backport of JDK-8240487 to jdk11u
>> I’m working on a series of patches for JNF dependency removal for macos and this backport will allow it to apply a lot more cleanly.
>> Very safe, white-space only changes.
>> Applies almost cleanly, I only removed second part of patch for src/jdk.hotspot.agent/macosx/native/libsaproc/MacosxDebuggerLocal.m
>> as it wasn’t needed.
>> 
>> The bug - https://bugs.openjdk.java.net/browse/JDK-8240487 <https://bugs.openjdk.java.net/browse/JDK-8240487>
>> The original commit - https://hg.openjdk.java.net/jdk/jdk/rev/f532ef5561c3 <https://hg.openjdk.java.net/jdk/jdk/rev/f532ef5561c3>
>> The webrev - http://cr.openjdk.java.net/~vkempik/8240487/webrev.01/ <http://cr.openjdk.java.net/~vkempik/8240487/webrev.01/>
>> 
>> Regards, Vladimir
> 
> 



More information about the jdk-updates-dev mailing list