Re: [FFmpeg-devel] [PATCH] avutil/pixdesc: add av_warn_unused_result to av_pix_fmt_get_chroma_sub_sample

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 29, 2015 at 6:13 PM, Michael Niedermayer wrote: > On Wed, Oct 28, 2015 at 11:03:33PM -0400, Ganesh Ajjanagadde wrote: >> On Fri, Oct 16, 2015 at 5:17 PM, Ganesh Ajjanagadde wrote: >> > On Fri, Oct 16, 2015 at 11:39 AM, Michael Niedermayer >>

Re: [FFmpeg-devel] [PATCH] avutil/pixdesc: add av_warn_unused_result to av_pix_fmt_get_chroma_sub_sample

2015-10-29 Thread Michael Niedermayer
On Wed, Oct 28, 2015 at 11:03:33PM -0400, Ganesh Ajjanagadde wrote: > On Fri, Oct 16, 2015 at 5:17 PM, Ganesh Ajjanagadde wrote: > > On Fri, Oct 16, 2015 at 11:39 AM, Michael Niedermayer > > wrote: > >> On Thu, Oct 15, 2015 at 07:20:30PM -0400, Ganesh

Re: [FFmpeg-devel] [PATCH] avutil/pixdesc: add av_warn_unused_result to av_pix_fmt_get_chroma_sub_sample

2015-10-28 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 5:17 PM, Ganesh Ajjanagadde wrote: > On Fri, Oct 16, 2015 at 11:39 AM, Michael Niedermayer > wrote: >> On Thu, Oct 15, 2015 at 07:20:30PM -0400, Ganesh Ajjanagadde wrote: >>> This will trigger a bunch of warnings (rightfully so).

Re: [FFmpeg-devel] [PATCH] avutil/pixdesc: add av_warn_unused_result to av_pix_fmt_get_chroma_sub_sample

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 11:39 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 07:20:30PM -0400, Ganesh Ajjanagadde wrote: >> This will trigger a bunch of warnings (rightfully so). This API has been >> abused, see the Doxygen comment above for what to do if the

Re: [FFmpeg-devel] [PATCH] avutil/pixdesc: add av_warn_unused_result to av_pix_fmt_get_chroma_sub_sample

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 07:20:30PM -0400, Ganesh Ajjanagadde wrote: > This will trigger a bunch of warnings (rightfully so). This API has been > abused, see the Doxygen comment above for what to do if the error code > is not meant to be checked. > > Signed-off-by: Ganesh Ajjanagadde

[FFmpeg-devel] [PATCH] avutil/pixdesc: add av_warn_unused_result to av_pix_fmt_get_chroma_sub_sample

2015-10-15 Thread Ganesh Ajjanagadde
This will trigger a bunch of warnings (rightfully so). This API has been abused, see the Doxygen comment above for what to do if the error code is not meant to be checked. Signed-off-by: Ganesh Ajjanagadde --- libavutil/pixdesc.h | 1 + 1 file changed, 1 insertion(+)