On Sun, 2011-12-18 at 17:20 +0000, Mans Rullgard wrote:
> -    int age;
> +    attribute_deprecated int age;

IMO setting attribute_deprecated immediately is a bad idea. Unless you
want your application to break when used with libavcodec compiled
yesterday you can't remove mentions of the field immediately. Thus
warnings like this produce clutter that only hides real issues unless
you disable deprecation warnings completely or add redundant version
checks. In this case removing code setting the field now rather than
later doesn't even give any benefit whatsoever: setting it causes no
harm and there is no new API to test.

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to