<div dir="ltr"><div><div>I sent in the new patchset.<br><br></div>Total of 6 patches starting with CreateDeviceAndSwapChain as that was missed by the devs somehow.<br><br>I also followed this where possible <br><br><span class="im">"FLOAT" -> "float", "LPCWSTR" -> "const WCHAR *"<br>
  - "num_*" -> "*_count"</span><br><br></div>Hopefully we can get this much merged<br><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 9, 2015 at 8:13 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"><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">make: Entering directory '/home/jacek/wine/wine-git/<span class=""><div>include'<br>
../tools/widl/widl -o d3d11_1.h -m32 -I. -I../include -D__WINESRC__<br>
d3d11_1.idl<br>
dxgi1_2.idl:383: error: syntax error, unexpected aIDENTIFIER<br>
Makefile:933: recipe for target 'd3d11_1.h' failed<br>
<br>
You use DXGI_MODE_DESC1, which is undeclared. I guess that you could<br>
tested headers in different environment, where it was declared, but<br>
please test things in Wine while sending patches. It's an easy way to<br>
validate your changes. In this case:<br>
tools/make_makefiles && make -C include/<br>
is the very least to see the error. Obviously, full make afterwards is<br>
even better.</div></span></blockquote><div><br></div><div>Yes I fixed this already<br>I also had 2 variables with the same name and one or two other syntax errors<br>but I haven't updated the patch as I was waiting for feedback.<br><br></div><div>I will be doing a new series without any style changes or winapi family today.<br></div><div>The process seems quite slow when I have loads of changes.<br>So I decided to omit this.<br><br></div><div>I'm actually using the headers to built SDL2 and OBS-studio so I know they are correct.<br></div><div>Anyway I'll make the new series today :)<br></div><div><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Feb 8, 2015 at 6:13 PM, Jacek Caban <span dir="ltr"><<a href="mailto:jacek@codeweavers.com" target="_blank">jacek@codeweavers.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Martell,<br>
<span><br>
On 02/07/15 01:02, Martell Malone wrote:<br>
> Initially I would have only done the dx stuff because mingw-w64 shares<br>
> those headers.<br>
> Essentially they Jacek Caban pulls in the d3d11 headers and idls from<br>
> wine in to mingw-w64 to update.<br>
><br>
> I could update the mingw-w64 project with the changes but I didn't<br>
> want to leave wine out of the loop.<br>
> I do understand that all headers would need to be partitioned to<br>
> support this eventually.<br>
> I was hoping the start would be with directx<br>
<br>
</span>Yes, I reviewed them and patches seemed OK for me. FWIW, the amount of<br>
style comments that you received surprises me. We generally leave style<br>
of new files up to developer, unless it's really ugly, and I don't think<br>
it was.<br>
<br>
I warned you that style changes of existing code better avoided (patch<br>
2), but you change arguments names at the same time, which makes it more<br>
acceptable.<br>
<br>
The real problem with your patches is something I didn't catch in the<br>
earlier review. I just tried to compile your code and got an error:<br>
<br>
make: Entering directory '/home/jacek/wine/wine-git/include'<br>
../tools/widl/widl -o d3d11_1.h -m32 -I. -I../include -D__WINESRC__<br>
d3d11_1.idl<br>
dxgi1_2.idl:383: error: syntax error, unexpected aIDENTIFIER<br>
Makefile:933: recipe for target 'd3d11_1.h' failed<br>
<br>
You use DXGI_MODE_DESC1, which is undeclared. I guess that you could<br>
tested headers in different environment, where it was declared, but<br>
please test things in Wine while sending patches. It's an easy way to<br>
validate your changes. In this case:<br>
tools/make_makefiles && make -C include/<br>
is the very least to see the error. Obviously, full make afterwards is<br>
even better.<br>
<br>
As for winapifamily.h header, I don't see a problem with having it in<br>
Wine. Alexandre, if there is any reason that this particular patch is<br>
not accepted (other than being part of questionable series), please comment.<br>
<br>
Thanks,<br>
Jacek<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>