Re: [FFmpeg-devel] [PATCH 1/5] lavf: add FFERROR_REDO to let demuxer return no packet.

2015-12-03 Thread Stefano Sabatini
On date Friday 2015-11-27 19:16:06 +0100, Nicolas George encoded: > Signed-off-by: Nicolas George > --- > libavformat/internal.h | 6 ++ > libavformat/utils.c| 5 + > 2 files changed, 11 insertions(+) > > > Apparently, the most unhappyness was caused by the error

Re: [FFmpeg-devel] [PATCH 1/5] lavf: add FFERROR_REDO to let demuxer return no packet.

2015-11-28 Thread Nicolas George
Le septidi 7 frimaire, an CCXXIV, Clement Boesch a écrit : > "returned internally by demuxers" maybe That sounds to me like an oxymoron. IMHO, being in internal.h and named FFERROR instead of AVERROR should be enough (even separately). > nit: space issue Locally fixed. > when they data?

Re: [FFmpeg-devel] [PATCH 1/5] lavf: add FFERROR_REDO to let demuxer return no packet.

2015-11-28 Thread Clément Bœsch
On Sat, Nov 28, 2015 at 09:08:52AM +0100, Nicolas George wrote: > Le septidi 7 frimaire, an CCXXIV, Clement Boesch a écrit : > > "returned internally by demuxers" maybe > > That sounds to me like an oxymoron. IMHO, being in internal.h and named > FFERROR instead of AVERROR should be enough (even

Re: [FFmpeg-devel] [PATCH 1/5] lavf: add FFERROR_REDO to let demuxer return no packet.

2015-11-27 Thread Clément Bœsch
On Fri, Nov 27, 2015 at 07:16:06PM +0100, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > libavformat/internal.h | 6 ++ > libavformat/utils.c| 5 + > 2 files changed, 11 insertions(+) > > > Apparently, the most unhappyness was caused by the error

[FFmpeg-devel] [PATCH 1/5] lavf: add FFERROR_REDO to let demuxer return no packet.

2015-11-27 Thread Nicolas George
Signed-off-by: Nicolas George --- libavformat/internal.h | 6 ++ libavformat/utils.c| 5 + 2 files changed, 11 insertions(+) Apparently, the most unhappyness was caused by the error code in the public API. This is not the case here. diff --git