Re: [FFmpeg-devel] [PATCH] avutil/parseutils: add av_warn_unused_result

2015-10-29 Thread Michael Niedermayer
On Wed, Oct 28, 2015 at 11:04:24PM -0400, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 10:24 PM, Ganesh Ajjanagadde wrote: > > On Thu, Oct 15, 2015 at 10:22 PM, Ganesh Ajjanagadde > > wrote: > >> On Thu, Oct 15, 2015 at 10:16 PM, James Almer

Re: [FFmpeg-devel] [PATCH] avutil/parseutils: add av_warn_unused_result

2015-10-29 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 7:52 PM, Ganesh Ajjanagadde wrote: > This triggers a few warnings that will need to be fixed - not that bad, > the current code is mostly fine. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/parseutils.h | 5 +

Re: [FFmpeg-devel] [PATCH] avutil/parseutils: add av_warn_unused_result

2015-10-28 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 10:24 PM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 10:22 PM, Ganesh Ajjanagadde wrote: >> On Thu, Oct 15, 2015 at 10:16 PM, James Almer wrote: >>> On 10/15/2015 10:23 PM, Ganesh Ajjanagadde wrote: On

Re: [FFmpeg-devel] [PATCH] avutil/parseutils: add av_warn_unused_result

2015-10-15 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 8:06 PM, James Almer wrote: > On 10/15/2015 8:52 PM, Ganesh Ajjanagadde wrote: >> This triggers a few warnings that will need to be fixed - not that bad, >> the current code is mostly fine. >> > > I think i asked you before, but in any case i'll do it

Re: [FFmpeg-devel] [PATCH] avutil/parseutils: add av_warn_unused_result

2015-10-15 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 08:15:20PM -0400, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 8:06 PM, James Almer wrote: > > On 10/15/2015 8:52 PM, Ganesh Ajjanagadde wrote: > >> This triggers a few warnings that will need to be fixed - not that bad, > >> the current code is

[FFmpeg-devel] [PATCH] avutil/parseutils: add av_warn_unused_result

2015-10-15 Thread Ganesh Ajjanagadde
This triggers a few warnings that will need to be fixed - not that bad, the current code is mostly fine. Signed-off-by: Ganesh Ajjanagadde --- libavutil/parseutils.h | 5 + 1 file changed, 5 insertions(+) diff --git a/libavutil/parseutils.h b/libavutil/parseutils.h

Re: [FFmpeg-devel] [PATCH] avutil/parseutils: add av_warn_unused_result

2015-10-15 Thread James Almer
On 10/15/2015 8:52 PM, Ganesh Ajjanagadde wrote: > This triggers a few warnings that will need to be fixed - not that bad, > the current code is mostly fine. > I think i asked you before, but in any case i'll do it again. Please, send patchsets contained in a single thread. You're making things

Re: [FFmpeg-devel] [PATCH] avutil/parseutils: add av_warn_unused_result

2015-10-15 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 8:06 PM, James Almer wrote: > On 10/15/2015 8:52 PM, Ganesh Ajjanagadde wrote: >> This triggers a few warnings that will need to be fixed - not that bad, >> the current code is mostly fine. >> > > I think i asked you before, but in any case i'll do it

Re: [FFmpeg-devel] [PATCH] avutil/parseutils: add av_warn_unused_result

2015-10-15 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 10:22 PM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 10:16 PM, James Almer wrote: >> On 10/15/2015 10:23 PM, Ganesh Ajjanagadde wrote: >>> On Thu, Oct 15, 2015 at 8:06 PM, James Almer wrote: On

Re: [FFmpeg-devel] [PATCH] avutil/parseutils: add av_warn_unused_result

2015-10-15 Thread Ganesh Ajjanagadde
On Thu, Oct 15, 2015 at 10:16 PM, James Almer wrote: > On 10/15/2015 10:23 PM, Ganesh Ajjanagadde wrote: >> On Thu, Oct 15, 2015 at 8:06 PM, James Almer wrote: >>> On 10/15/2015 8:52 PM, Ganesh Ajjanagadde wrote: This triggers a few warnings that will

Re: [FFmpeg-devel] [PATCH] avutil/parseutils: add av_warn_unused_result

2015-10-15 Thread James Almer
On 10/15/2015 10:23 PM, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 8:06 PM, James Almer wrote: >> On 10/15/2015 8:52 PM, Ganesh Ajjanagadde wrote: >>> This triggers a few warnings that will need to be fixed - not that bad, >>> the current code is mostly fine. >>> >> >>