Re: [FFmpeg-devel] [PATCH]lswr/swresample: Mention actually supported formats when erroring out

2017-10-28 Thread Carl Eugen Hoyos
2017-10-27 4:12 GMT+02:00 Michael Niedermayer : > On Thu, Oct 26, 2017 at 05:03:37PM +0200, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch is supposed to fix ticket #6779. >> >> Please comment, Carl Eugen > >> swresample.c |4 ++-- >> 1 file changed, 2

Re: [FFmpeg-devel] [PATCH]lswr/swresample: Mention actually supported formats when erroring out

2017-10-26 Thread Michael Niedermayer
On Thu, Oct 26, 2017 at 05:03:37PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch is supposed to fix ticket #6779. > > Please comment, Carl Eugen > swresample.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > d66bcf39e9ce53dad4db82a5202c7494fbf2d46c >

Re: [FFmpeg-devel] [PATCH]lswr/swresample: Mention actually supported formats when erroring out

2017-10-26 Thread Carl Eugen Hoyos
2017-10-26 18:16 GMT+02:00 Derek Buitenhuis : > On 10/26/2017 5:13 PM, Carl Eugen Hoyos wrote: >> Not sure I understand: >> Do you mean that the filter option should not require "p" but add >> it always? >> >> Or do you mean it is obvious for users that only planar >>

Re: [FFmpeg-devel] [PATCH]lswr/swresample: Mention actually supported formats when erroring out

2017-10-26 Thread Derek Buitenhuis
On 10/26/2017 5:13 PM, Carl Eugen Hoyos wrote: > Not sure I understand: > Do you mean that the filter option should not require "p" but add > it always? > > Or do you mean it is obvious for users that only planar > formats are supported? If it lines up with the option names, then yes, leave the

Re: [FFmpeg-devel] [PATCH]lswr/swresample: Mention actually supported formats when erroring out

2017-10-26 Thread Carl Eugen Hoyos
2017-10-26 18:10 GMT+02:00 Derek Buitenhuis : > On 10/26/2017 4:03 PM, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch is supposed to fix ticket #6779. >> >> Please comment, Carl Eugen > > Is the 'p' suffix on each needed, since swr only supports 'planar' audio?

Re: [FFmpeg-devel] [PATCH]lswr/swresample: Mention actually supported formats when erroring out

2017-10-26 Thread Derek Buitenhuis
On 10/26/2017 4:03 PM, Carl Eugen Hoyos wrote: > Hi! > > Attached patch is supposed to fix ticket #6779. > > Please comment, Carl Eugen Is the 'p' suffix on each needed, since swr only supports 'planar' audio? - Derek ___ ffmpeg-devel mailing list

[FFmpeg-devel] [PATCH]lswr/swresample: Mention actually supported formats when erroring out

2017-10-26 Thread Carl Eugen Hoyos
Hi! Attached patch is supposed to fix ticket #6779. Please comment, Carl Eugen From 17ffc43ba32c83e0e1d26df06b7f4445adb1206b Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Thu, 26 Oct 2017 16:52:22 +0200 Subject: [PATCH] lswr/swresample: Mention the actually supported