[MPlayer-dev-eng] [PATCH] ggi osd support added
Rich Felker
dalias at aerifal.cx
Fri Jun 30 22:56:34 CEST 2006
On Fri, Jun 30, 2006 at 02:39:59PM -0400, spdepagn at ncsu.edu wrote:
> Hi, thanks for all the feedback.
>
> > 2006/6/30, spdepagn at ncsu.edu <spdepagn at ncsu.edu>:
> >> Hi, I have added support for the osd to the ggi output driver. I am
> >> wondering why the osd is implemented so each output driver must
> >> implement
> >> it.
> >
> > It must not.
> >
> > mplayer -vf expand=::::1
> >
>
> Would it make sense to make this a default for drivers that do no
> implement the osd? I think there are a lot of people who do not know
> about this option.
No, auto-loading filters to second-guess the user is lame and it will
also result in bad performance. The vo drivers should be fixed to
support osd if they don't.
> > You are storing osd internal planes from previous frame and draw them
> > on the next one, in draw_slice().
> >
> > draw_slice() could be called many times for sinlge picture when new
> > slice (16 lines) have been updated. You are currently redrawing all
> > osd elements on every slice.
> >
> > I must check the source in order to be sure that these planes are not
> > freed after they've been "drawn". Causing your driver to access freed
> > memory.
> >
> > Messing with osd internals looks quite horrible way to implement osd.
> > And I don't think it could be accepted.
> >
> > Rejected.
> > _______________________________________________
>
> I understand your reasons for rejecting this, especially since the osd can
> be added with another option. Thanks
OSD should be added, but the code you submitted is just incorrect and
will cause all kinds of corruption. A working patch would be very
welcome I'm sure.
Rich
More information about the MPlayer-dev-eng
mailing list