[MPlayer-dev-eng] [PATCH] libass fixes and additions

[Evgeniy Stepanov]

On Sunday 01 March 2009 18:34:13 Grigori Goronzy wrote:
> Evgeniy Stepanov wrote:
> >> This patch makes positioned events retain their position when
> >> pan-and-scan is used -- even if this means clipping them.
> >> Regular events are still restricted to the visible area of the video.
> >
> > It was discussed on the ML, and we decided that it is better to fit every
> > positioned event to the screen even if it means moving them a little.
>
> How is this better or desirable?
> Positioned events are called like that for a reason. Not displaying them
>   at their designated position makes little sense.
> Maybe it can be made an option, but IMHO the default behavior should be
> to display positioned events at their correct location in any case.

You are right, it's the other way around :) Here is the link to that thread:
http://thread.gmane.org/gmane.comp.video.mplayer.devel/45600

Committed.

> >> * 0002-Composite-combine-overlapping-glyph-borders-and-shad.patch
> >>
> >> Like the old patch, but also combines shadows, if needed, and uses the
> >> cache for storing the composited bitmaps.
>
> No comment?

Looks ok, I'll commit it in a day or two if nothing better comes to mind.