[2/6] dxgi: Add interface for IDXGISurface1

Stefan Dösinger stefandoesinger at gmail.com
Tue Feb 10 04:53:11 CST 2015


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

Hi Martell,

There are a few things in Henri's list of fixes that are still missing here:

Am 2015-02-09 um 10:28 schrieb Martell Malone:
> Subject: [PATCH 2/6] dxgi: Add interface for IDXGISurface1
"include:" instead of "dxgi".

> +    uuid( 4ae63092-6327-4c1b-80ae-bfe12ea32b86),
You have an extra space here.

> +        [out] HDC* hdc
HDC *hdc, or even better HDC *dc.

We avoid the Hungarian notification Microsoft once used, e.g. things like DWORD dwNumber. HDC hdc is the same kind of thing, although it's a bit of a special case.

Patch 3:

> +typedef struct DXGI_OUTDUPL_MOVE_RECT {
> +    POINT SourcePoint;
> +    RECT DestinationRect;
> +} DXGI_OUTDUPL_MOVE_RECT;
{ placement .

You also have FOO* bar; vs FOO *bar; issues in this patch.

> +        [in] IUnknown* pDevice,
> +        [in] IUnknown* pWindow,
> +        [in] const DXGI_SWAP_CHAIN_DESC1* pDesc,
pStuff

The same things also occur in patches 4, 5, and 6.

Patch 4:
> +        [size_is(count_levels)] const D3D_FEATURE_LEVEL* levels,
> +        UINT count_levels,
You are mixing count_* and *_count in patches 4, 5 and 6. For consistency's sake please make this *_count consistently.

Patch 6:
> +}
> \ No newline at end of file
Is this intentional?

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBAgAGBQJU2eMXAAoJEN0/YqbEcdMwUmYP/A8I/1UBVqcynhIJ5qIvVTl4
sf3XQcWTmgI40JurL4D1mIaQMH+VeulQ7Jeroo9L9/WhIw1RRJ7R0jFULxZIu9tP
VX5AyTVjd555jbt9DvlbmkaAicb0UuX7xSO5OelkkdwtdfuDCKnDq2+LxzXA2fXw
y98vlT98meapIP5BzERMyAAyM0AkvpAMaxkBabE9w9Mj6IJUWteHQEwa2iavu0HR
wBr/KotHXu/nqVD41PTYnAsyQoWeBNPBghg4RyqI42YKiOnDxuApOT9nAgewwHPA
VghOW/hdlZahCZc+/Vst01pD7Sa7vS/Nuq4iDfJJVofSsXgwIdl5KT63eAOzR4Dl
MefDKrEBmdTh44K/JeyCey1Rb4o6fNtPCQJV6lqXQkiHVeE/8IW8qPLectwuGPVd
UtC7Fy38iR4UpDJBmPGypdZaFJzS2ZF1hO+mA/jJhOnefyL31vCeFWU+bf5ISuwo
TDEGu7G/8yBpY+HKHr+++IwJlJKkEngf98ksjnyRC1PpIner05XHUDeu1PIll/FS
T3aveyW5lWsyVGCyyovHLQIxfkaV6JPqR/AsuTuhDmhZu4ZQPoGLH5kpfrAhgVMA
TeLGoR6RtTMOyFLob4sRI9nBR89H/UiEQ++fGbp/YXsOLnOZBJvn6y8AYh6zaeD5
6smvq4QiqsuQQS3tSC+f
=CmBk
-----END PGP SIGNATURE-----



More information about the wine-devel mailing list