<div dir="ltr"><div><div><div><div><div><div><div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">"include:" instead of "dxgi"</blockquote><div>He told me to prefix the ones involving dxgi with dxgi: and not include:<br></div></div></div>He also told me to prefix the ones involving d3d11 with d3d11:<br></div>I had intended to use include but this is what his email said to do.<br></div>I need to clarify this before I resubmit ??<br><br><span class="im">Patch 3:<br>
> +typedef struct DXGI_OUTDUPL_MOVE_RECT {<br>
> +    POINT SourcePoint;<br>
> +    RECT DestinationRect;<br>
> +} DXGI_OUTDUPL_MOVE_RECT;<br>
{ placement</span><br></div>He didn't really specify this btw.<br></div>The rest of the file has this format for the structs already.<br></div>How do you want me to do this ?<br><div><div><div><div><br></div></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 10, 2015 at 11:41 AM, Martell Malone <span dir="ltr"><<a href="mailto:martellmalone@gmail.com" target="_blank">martellmalone@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">"include:" instead of "dxgi"He told me to prefix the ones involving dxgi with dxgi: and not include:<br></blockquote></div></div>He also told me to prefix the ones involving d3d11 with d3d11:<br></div>I had intended to use include but this is what his email said to do.<br></div><div><span class=""><br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">Patch 3:<br>
> +typedef struct DXGI_OUTDUPL_MOVE_RECT {<br>
> +    POINT SourcePoint;<br>
> +    RECT DestinationRect;<br>
> +} DXGI_OUTDUPL_MOVE_RECT;<br>
{ placement .<br>
You also have FOO* bar; vs FOO *bar; issues in this patch.<br>
> +        [in] IUnknown* pDevice,<br>
> +        [in] IUnknown* pWindow,<br>
> +        [in] const DXGI_SWAP_CHAIN_DESC1* pDesc,<br>
pStuff<br>
The same things also occur in patches 4, 5, and 6<br></blockquote><br></span>I originally had the * at the right side in the first series of patches so when he told me to change that I assumed he meant move it to the left. I will move them back.<span class=""><br><br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">Patch 4:<br>
> +        [size_is(count_levels)] const D3D_FEATURE_LEVEL* levels,<br>
> +        UINT count_levels,<br>
You are mixing count_* and *_count in patches 4, 5 and 6. For consistency's sake please make this *_count consistently.<br></blockquote>
<br></span>Can you give me an example of this please. Originally I had num_levels but he did say change to size.<br></div><div>Does _countlevels suffice ? I'm not quite sure here<span class=""><br><br>Patch 6:<br>
> +}<br>
> \ No newline at end of file<br>
Is this intentional?<br></span></div><div>No I meant to have a newline at the end of the file.<br></div><div>It appears I made the patches worse rather than better.<br></div><div><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 10, 2015 at 10:53 AM, Stefan Dösinger <span dir="ltr"><<a href="mailto:stefandoesinger@gmail.com" target="_blank">stefandoesinger@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
Hi Martell,<br>
<br>
There are a few things in Henri's list of fixes that are still missing here:<br>
<br>
Am 2015-02-09 um 10:28 schrieb Martell Malone:<br>
> Subject: [PATCH 2/6] dxgi: Add interface for IDXGISurface1<br>
"include:" instead of "dxgi".<br>
<br>
> +    uuid( 4ae63092-6327-4c1b-80ae-bfe12ea32b86),<br>
You have an extra space here.<br>
<br>
> +        [out] HDC* hdc<br>
HDC *hdc, or even better HDC *dc.<br>
<br>
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.<br>
<br>
Patch 3:<br>
<br>
> +typedef struct DXGI_OUTDUPL_MOVE_RECT {<br>
> +    POINT SourcePoint;<br>
> +    RECT DestinationRect;<br>
> +} DXGI_OUTDUPL_MOVE_RECT;<br>
{ placement .<br>
<br>
You also have FOO* bar; vs FOO *bar; issues in this patch.<br>
<br>
> +        [in] IUnknown* pDevice,<br>
> +        [in] IUnknown* pWindow,<br>
> +        [in] const DXGI_SWAP_CHAIN_DESC1* pDesc,<br>
pStuff<br>
<br>
The same things also occur in patches 4, 5, and 6.<br>
<br>
Patch 4:<br>
> +        [size_is(count_levels)] const D3D_FEATURE_LEVEL* levels,<br>
> +        UINT count_levels,<br>
You are mixing count_* and *_count in patches 4, 5 and 6. For consistency's sake please make this *_count consistently.<br>
<br>
Patch 6:<br>
> +}<br>
> \ No newline at end of file<br>
Is this intentional?<br>
<br>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v2<br>
<br>
iQIcBAEBAgAGBQJU2eMXAAoJEN0/YqbEcdMwUmYP/A8I/1UBVqcynhIJ5qIvVTl4<br>
sf3XQcWTmgI40JurL4D1mIaQMH+VeulQ7Jeroo9L9/WhIw1RRJ7R0jFULxZIu9tP<br>
VX5AyTVjd555jbt9DvlbmkaAicb0UuX7xSO5OelkkdwtdfuDCKnDq2+LxzXA2fXw<br>
y98vlT98meapIP5BzERMyAAyM0AkvpAMaxkBabE9w9Mj6IJUWteHQEwa2iavu0HR<br>
wBr/KotHXu/nqVD41PTYnAsyQoWeBNPBghg4RyqI42YKiOnDxuApOT9nAgewwHPA<br>
VghOW/hdlZahCZc+/Vst01pD7Sa7vS/Nuq4iDfJJVofSsXgwIdl5KT63eAOzR4Dl<br>
MefDKrEBmdTh44K/JeyCey1Rb4o6fNtPCQJV6lqXQkiHVeE/8IW8qPLectwuGPVd<br>
UtC7Fy38iR4UpDJBmPGypdZaFJzS2ZF1hO+mA/jJhOnefyL31vCeFWU+bf5ISuwo<br>
TDEGu7G/8yBpY+HKHr+++IwJlJKkEngf98ksjnyRC1PpIner05XHUDeu1PIll/FS<br>
T3aveyW5lWsyVGCyyovHLQIxfkaV6JPqR/AsuTuhDmhZu4ZQPoGLH5kpfrAhgVMA<br>
TeLGoR6RtTMOyFLob4sRI9nBR89H/UiEQ++fGbp/YXsOLnOZBJvn6y8AYh6zaeD5<br>
6smvq4QiqsuQQS3tSC+f<br>
=CmBk<br>
-----END PGP SIGNATURE-----<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>