[PATCH v2 1/6] wined3d: Add support for NV12 textures
Stefan Dösinger
stefandoesinger at gmail.com
Wed Feb 12 08:34:55 CST 2014
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Hi again!
I have some more format nitpicks below.
As far as the big picture is concerned there is also a software
conversion path in dlls/wined3d/surface.c. Search the file for
d3dfmt_converter_desc and you should find the right place. You don't
have to submit a patch for that unless you know an application that
needs it though; I'm just making you aware of it.
Am 2014-02-11 12:14, schrieb Martin Storsjo:
> + switch(textype)
You're missing a space here. As far as I can see the other places in
this patch are correct, but it doesn't hurt to double-check with grep
or something.
(I really hope the testbot can one day take care of this...)
> + case GL_TEXTURE_2D: tex = "2D"; break;
Please make this
case GL_TEXTURE_2D:
tex = "2D";
break;
etc
> + * When reading from rectangle textures, keep in mind that
> the input y coordinates + * go from 0 to d3d_height, whereas
> the opengl texture height is 1.5 * d3d_height + */
Comment closing should go next to the last line of text. Also add
proper interpunctuation at the last sentence of a comment.
This applies to a few more places in this patch, and most likely the
other patches.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iQIcBAEBAgAGBQJS+4aPAAoJEN0/YqbEcdMwCXMQAIPeaYbrI+zC2zf6KlOdOCbe
yDSbSvRS4vZWkukemQCb6jr2qFCjqoCJMHj1GpfWxlBxJ1+rOWyTdc+YG2DDzCqk
YE3K8PgOaID7BnopA/Ri0gYiwRz7K2WBpMWMwgeVnjTFOUWJ8tYt1z4VmCHXgEEi
JDphv1q9IDzCW/s3Pu68VtDrE4rsG/KLietRPTCgakdjX1oLvlKn1Fg5D7q78y87
szJg2SRaYOVWSXIK2D+s+A5Tvjr9gffSsDE6fgFgxeFyHsViGyq6QDJEBxCO+f+A
UZ1FhidcogIt/sqSphg7lCyEUw49WojlXSXhLpdofwjrR/4griYgp21h6Y8UhtqG
LzPUVGBfScvb208bIhL+LvCukQ00WnTI4LJA6khjL9AlCnrzh4nrhVo6RQDrL1A3
6V5gJLd1ZT4DepQc2XQLC0FupgYWafIJAzCcpZIpvQRjzH4MqJgeJC9m6alRzyWr
TkN+Jo6yKRk2t+x7LOzee+mMPAfIT2OcHoIlG60Z8ZxK3Apu6k7/RPoqi1BWdJZZ
gHA1CZLUDC+UXhPhr2pxNy4AQnD2EbIBp5GPKNHHvoqJJzXvbHPBx5TgOOaEyLnH
idubGx+GLf3zXEaK+ujbMg8Ae/poGh841gBVbMLDqZguSkuTsVGpt8hjCZgWq+5d
u177zpoEIMhOFIK17nSY
=IHTQ
-----END PGP SIGNATURE-----
More information about the wine-devel
mailing list