<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">2013/5/31 Detlef Riekenberg <span dir="ltr"><<a href="mailto:wine.dev@web.de" target="_blank">wine.dev@web.de</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Christian wrote at Wednesday, 29. Mai 2013 at 23:12 Uhr<br>
<div class="im"><br>
> > + */<br>
> > +HRESULT WINAPI SLGetWindowsInformationDWORD(PCWSTR name, DWORD *pvalue)<br>
> > +{<br>
> ><br>
</div><div class="im">> > -SLCAPI DWORD WINAPI SLGetWindowsInformationDWORD(LPCWSTR lpszValueName, LPDWORD pdwValue);<br>
> > +SLCAPI HRESULT WINAPI SLGetWindowsInformationDWORD(PCWSTR, DWORD *);<br>
> ><br>
</div><div class="im">> I think s/PCWSTR/const WCHAR */ and s/pvalue/value/ would be better. :)<br>
<br>
</div>I disagree for "const WCHAR *", which is really rare used in Wine<br>
(Please grep the headers yourself)<br>
<br></blockquote><div>�</div><div>There was a discussion about the LPxxx stuff removal (I guess Pxxx as well) some time ago.<br></div><div>There was also something about const that doesn't behave as expected in that case but I don't remember the details.<br>
</div><div>� <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
pvalue is a matter of taste.<br></blockquote><div><br></div><div>It's not particularily my taste. It's something preferred. It was just a guess since the patch was pending.<br></div></div><br></div><div class="gmail_extra">
Christian<br></div></div>