<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
looks fine to me.<br>
<pre class="moz-signature" cols="72">Thanks,
Alexander.</pre>
<div class="moz-cite-prefix">On 07/14/2015 02:18 PM, Sergey Bylokhov
wrote:<br>
</div>
<blockquote cite="mid:55A4F007.5040404@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
<tt>Hello.</tt><tt><br>
</tt><tt>Please review the fix for jdk9:</tt><tt><br>
</tt><tt> - I "turn on warnings as error" for libjsoundalsa
library.</tt><tt><br>
</tt><tt> - The last warning was fixed. The problem was in fact
that gcc </tt><tt>suggest a </tt><tt>parentheses</tt><tt> arou</tt><tt>nd</tt><tt>
&& inside ||;</tt><tt><br>
</tt>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
<tt><br>
</tt><tt>Note that all new changes should be warning free,
otherwise it will cause a build breakage.</tt><tt><br>
</tt><tt><br>
</tt><tt>Bug: <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8074832">https://bugs.openjdk.java.net/browse/JDK-8074832</a></tt><tt><br>
</tt><tt>Webrev can be found at: <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Eserb/8074832/webrev.00">http://cr.openjdk.java.net/~serb/8074832/webrev.00</a></tt><br>
<pre class="moz-signature" cols="72">--
Best regards, Sergey. </pre>
</blockquote>
<br>
</body>
</html>