[MPlayer-dev-eng] Direct3D OSD discussion

Jim Hauxwell james at dattrax.co.uk
Mon Dec 8 13:00:52 CET 2008


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Jim Hauxwell wrote:
> Reimar Döffinger wrote:
>> Hello,
>> On Mon, Dec 08, 2008 at 11:22:41AM +0100, Reimar Döffinger wrote:
>>> On Mon, Dec 08, 2008 at 09:14:59AM +0200, Georgi Petrov wrote:
>>>> I vote for inclusion now. If there are some indent / spacing issues,
>>>> I'll fix them after the patch is commited. We need the OSD.
>>> Have you tested it?
>>> Also there is still almost the same buffer flow problem: the allocated
>>> texture may be smaller than the arguments to vo_draw_text, allowing for
>>> a buffer overflow.
>>> There are also minor issues like the expression to select
>>> non-power-of-two texture is extremely overcomplicated (e.g. !(a ^ b) is
>>> the same as a == b) and also wrong.
>>> I guess I can go over it myself and fix it when I got a bit of time.
>> This one should be better, but I could not test since wine is currently
>> broken on my system.
> 
> I think it will suffer from the same problem that the original patch
> has.  The texture will be stretched when the aspect is changed.
> 
> I'm also confused as to why the intrinsics were dropped, doesn't MPlayer
> stipulate to compile on gcc only?  This is not an issue, just I would
> like to know for future.
> 
> I would also like to know why there is a second draw in the
> render_d3d_frame which only has enough vertices for a single triangle?
> Are you trying to redraw the OSD?  If so, why?
> 

Here is another patch, with the two issues fixed.  It should never go
outside the texture area when generating the OSD and the logic for the
power2 has been simplified as discussed.

Jim


-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkk9DHQACgkQhrNWoHjgI1CxfwCffw+bU6VpwmoF4nYPlogvZc4n
sskAnREFxMaUKydvrsf1L3X2lL+61Xqi
=m1W/
-----END PGP SIGNATURE-----
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: diff20.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20081208/59c4a93c/attachment.txt>


More information about the MPlayer-dev-eng mailing list