Re: [FFmpeg-devel] [PATCH] ffmpeg_filter: refuse to configure input without a decoder.

2014-07-30 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at gmx.at writes: LGTM Ping. Please add a comment about ticket #3779 to the commit message. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] ffmpeg_filter: refuse to configure input without a decoder.

2014-07-30 Thread Nicolas George
Le duodi 12 thermidor, an CCXXII, Carl Eugen Hoyos a écrit : Ping. Please add a comment about ticket #3779 to the commit message. Done and pushed to my tree for anyone to merge. (It did not seem urgent, I was waiting to bundle it with other patches to save time to the merger. Or are there

Re: [FFmpeg-devel] [PATCH] ffmpeg_filter: refuse to configure input without a decoder.

2014-07-30 Thread Michael Niedermayer
On Wed, Jul 30, 2014 at 03:12:19PM +0200, Nicolas George wrote: Le duodi 12 thermidor, an CCXXII, Carl Eugen Hoyos a écrit : Ping. Please add a comment about ticket #3779 to the commit message. Done and pushed to my tree for anyone to merge. merged (It did not seem urgent, I was

[FFmpeg-devel] [PATCH] ffmpeg_filter: refuse to configure input without a decoder.

2014-07-18 Thread Nicolas George
The decoder is necessary in order to filter frames. This makes the error message clearer in this case: currently, it will usually fail because the pixel or sample format is not defined and is converted into (null) (non-portable). Signed-off-by: Nicolas George geo...@nsup.org --- ffmpeg_filter.c

Re: [FFmpeg-devel] [PATCH] ffmpeg_filter: refuse to configure input without a decoder.

2014-07-18 Thread Michael Niedermayer
On Fri, Jul 18, 2014 at 10:43:22AM +0200, Nicolas George wrote: The decoder is necessary in order to filter frames. This makes the error message clearer in this case: currently, it will usually fail because the pixel or sample format is not defined and is converted into (null) (non-portable).