[MPlayer-dev-eng] Re: [BUG] various weirdness in vf_expand when expanded width > frame width
Jason Tackaberry
tack at urandom.ca
Sun Dec 31 16:35:41 CET 2006
On Sun, 2006-12-31 at 15:40 +0100, Tomas Janousek wrote:
> The vf_expand lacks the code to clear the left/right sides, having a
> //TODO instead. It does that for top/bottom so it should certainly be
> there. I posted a patch addressing this a while ago, please look into
> the archives for a thread called "[PATCH] vf_expand - clear left and
> right side of the image if needed" and check whether it solves the issue
> for you.
I did indeed see the TODO in the code, but given that when vf_expand
expands width-wise if the expand osd parameter is set I don't even see
the OSD at all, I didn't bother looking further there.
Unlike when the height gets expanded, when the width is expanded the
default OSD seems to render in the bands. I suppose it should be this
way.
My primary issue was that the left/right bands were gray, but later
discovered that was mainly a property of the vo, and all VOs behaved
differently. My patch yesterday addresses this in vo_xv (but it just
occurred to me that I did it the wrong way).
I'll try fixing vf_expand's osd in this case (if it's just a matter of
wrong offsets it should be easy enough) and retry your patch. If it
works I'll push a bit to get it merged. :)
Thanks,
Jason.
More information about the MPlayer-dev-eng
mailing list