[MPlayer-dev-eng] MPlayer/MEncoder crash on using -vf scale and expand together. Fix attached.

[Reimar Döffinger]

On Mon, Oct 31, 2011 at 12:11:28AM +0200, Ivan Kalvachev wrote:
> On 10/29/11, Alex C. <alexc.xander at yahoo.in> wrote:
> > Seriously,  though, it  would  be better  to  replace libswscale  with
> > another, cleaner,  scaling engine, even  if a little  slower. Possible
> > alternatives  would include  y4mscale and  AVISynth scaling,  and I've
> > also heard good things about DScaler. Maybe we can cherry-pick those.
> 
> I suspend this and mine patch.
> I think I found mistake in my logic. The current flags should work for
> all types that are numerically bigger than EXPORT_, aka are not
> exported. So the reason for not aligning the stride may be exactly the
> reason you pointed out, using of vo that doesn't accept stride.

The case I was worrying about was direct rendering to a frame buffer
device for example.
With forced aligned stride we couldn't support direct rendering in some
cases at all, and I prefer to avoid the performance cost of that if
possible.