[MPlayer-dev-eng] [PATCH] VF Overlay
Benjamin Zores
ben at geexbox.org
Tue Jul 28 22:30:00 CEST 2009
Diego Biurrun wrote:
> On Mon, Jul 27, 2009 at 09:26:21PM +0200, Benjamin Zores wrote:
>> I intend to apply it within the week unless someone's complain about it,
>> as it shouldn't harm any other part of MPlayer.
>
> This has lots of issues, apart from ugly formatting. Don't apply it
> as-is.
>
> I'm in a hurry now, more substantial comments later. But you could
> really review it yourself first, I'm convinced you'll find a few things.
> At least this could be formatted as proper K&R.
Attached an updated version with format changes, doc update and a test
tool as a bonus.
Ben
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vf-overlay.diff
Type: text/x-diff
Size: 66632 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20090728/229cbb51/attachment.diff>
More information about the MPlayer-dev-eng
mailing list