<Sound Dev> [9] Review Request: 8074832 Resolve disabled warnings for libjsoundalsa

Alexander Zvegintsev alexander.zvegintsev at oracle.com
Tue Jul 14 12:05:57 UTC 2015


looks fine to me.

Thanks,

Alexander.

On 07/14/2015 02:18 PM, Sergey Bylokhov wrote:
> Hello.
> Please review the fix for jdk9:
>  - I "turn on warnings as error" for libjsoundalsa library.
>  - The last warning was fixed. The problem was in fact that gcc 
> suggest a parenthesesaround&& inside ||;
>
> Note that all new changes should be warning free, otherwise it will 
> cause a build breakage.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8074832
> Webrev can be found at: http://cr.openjdk.java.net/~serb/8074832/webrev.00
> -- 
> Best regards, Sergey.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/sound-dev/attachments/20150714/6972a26e/attachment.html>


More information about the sound-dev mailing list