<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial, Helvetica, sans-serif;font-size:10pt" ><div dir="ltr" >Hi Ivan,</div>
<div dir="ltr" > </div>
<div dir="ltr" >Changes look good to me.</div>
<div dir="ltr" > </div>
<div dir="ltr" >Thanks,</div>
<div dir="ltr" >Vyom</div>
<div dir="ltr" > </div>
<blockquote data-history-content-modified="1" dir="ltr" style="border-left:solid #aaaaaa 2px; margin-left:5px; padding-left:5px; direction:ltr; margin-right:0px" >----- Original message -----<br>From: Ivan Gerasimov <ivan.gerasimov@oracle.com><br>Sent by: "nio-dev" <nio-dev-bounces@openjdk.java.net><br>To: core-libs-dev <core-libs-dev@openjdk.java.net>, "nio-dev@openjdk.java.net" <nio-dev@openjdk.java.net>, security-dev@openjdk.java.net<br>Cc:<br>Subject: [EXTERNAL] RFR 8211360 : Change #if DEF to #if defined(DEF)<br>Date: Tue, Aug 20, 2019 11:09 PM<br> 
<div><font size="2" face="Default Monospace,Courier New,Courier,monospace" >Hello!<br><br>It's a followup for JDK-8211146.<br><br>With that fix several C-preprocessor statements of form #elif __linux__<br>were changed to more accurate #elif defined(__linux__).<br><br>grep found a few more occurrences of the same pattern, which also would<br>better be cleaned up.<br><br>BUGURL: <a href="https://bugs.openjdk.java.net/browse/JDK-8211360" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8211360</a> <br>WEBREV: <a href="http://cr.openjdk.java.net/~igerasim/8211360/00/webrev/" target="_blank">http://cr.openjdk.java.net/~igerasim/8211360/00/webrev/</a> <br><br>Mach5 control build/testing went fine on all platforms.<br>Would you please help review?<br><br>--<br>With kind regards,<br>Ivan Gerasimov</font><br><br> </div></blockquote>
<div dir="ltr" > </div></div><BR>