Re: [FFmpeg-devel] ffplay: insertion point of the auto rotation filter - Github ticket #141

2015-12-28 Thread Marton Balint
On Mon, 28 Dec 2015, Michael Niedermayer wrote: On Sun, Dec 27, 2015 at 11:09:42PM +0100, Balint Marton wrote: Hi Michael, The patch you committed seems to break the cropping to even width / height as required by SDL overlay code. Also there can be a use case for inserting the auto

Re: [FFmpeg-devel] ffplay: insertion point of the auto rotation filter - Github ticket #141

2015-12-28 Thread Michael Niedermayer
On Sun, Dec 27, 2015 at 11:09:42PM +0100, Balint Marton wrote: > Hi Michael, > > The patch you committed seems to break the cropping to even width / > height as required by SDL overlay code. > > Also there can be a use case for inserting the auto rotation filter > after the user provided filter

Re: [FFmpeg-devel] ffplay: insertion point of the auto rotation filter - Github ticket #141

2015-12-28 Thread Michael Niedermayer
On Sun, Dec 27, 2015 at 11:09:42PM +0100, Balint Marton wrote: > Hi Michael, > > The patch you committed seems to break the cropping to even width / > height as required by SDL overlay code. > > Also there can be a use case for inserting the auto rotation filter > after the user provided filter

[FFmpeg-devel] ffplay: insertion point of the auto rotation filter - Github ticket #141

2015-12-27 Thread Balint Marton
Hi Michael, The patch you committed seems to break the cropping to even width / height as required by SDL overlay code. Also there can be a use case for inserting the auto rotation filter after the user provided filter chain as well. (e,g, deinterlacing). I don't think we can make everyone