On Sun, Jun 19, 2011 at 05:43:46PM +0100, Mans Rullgard wrote:
> These flags are accepted without error but produce an annoying
> warning. Filtering them out makes the build less noisy.
>
> --- a/configure
> +++ b/configure
> @@ -2070,6 +2070,15 @@ elif $cc -v 2>&1 | grep -q 'PathScale\|Path64'; then
> size_cflags='-Os'
> + filter_cflags=pathscale_cflags
> + pathscale_cflags(){
> + for flag; do
> + case $flag in
> + -Wdisabled-optimization) continue ;;
> + esac
> + echo $flag
> + done
> + }
> elif $cc -v 2>&1 | grep -q Open64; then
> @@ -2078,6 +2087,17 @@ elif $cc -v 2>&1 | grep -q Open64; then
> size_cflags='-Os'
> + filter_cflags=open64_cflags
> + open64_cflags(){
> + for flag; do
> + case $flag in
> + -Wdisabled-optimization) continue ;;
> + -Wtype-limits) continue ;;
> + -fno-signed-zeros) continue ;;
> + esac
> + echo $flag
> + done
> + }
> fi
This code duplication calls for refactoring at some point.
That said, I don't think it's a particularly pressing concern,
so patch OK.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
