<div dir="ltr"><br><div class="gmail_extra">Hi,<br><br></div><div class="gmail_extra">You're right. This can be simplified. Will send a patch. Thanks.<br><br></div><div class="gmail_extra">Christian<br></div><div class="gmail_extra">
<br><div class="gmail_quote">2013/5/29 gurketsky <span dir="ltr"><<a href="mailto:gurketsky@googlemail.com" target="_blank">gurketsky@googlemail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
just saw that a bit late. Is there a reason for the rewrite of hr?<br>
<br>
Could any call return anything other than D3D_OK and succeed (maybe S_FALSE)? It may be possible to drop "if (FAILED(hr)) goto end;" (for both cases) and "hr = D3D_OK"... Also the function (parse_mesh) seems to overwrite the result anyway to D3D_OK on success, so I see no reason to change the hr to D3D_OK in parse_skin_mesh_info.<br>

<br>
Cheers<br>
Rico<div class="HOEnZb"><div class="h5"><br>
<br>
On 28.05.2013 22:16, Alexandre Julliard wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+ � � � �if (SUCCEEDED(hr))<br>
+ � � � � � �hr = mesh_data->skin_info->lpVtbl-><u></u>SetBoneOffsetMatrix(mesh_data-<u></u>>skin_info, index,<br>
+ � � � � � � � � � � (const D3DMATRIX*)(data + nb_influences * (sizeof(DWORD) + sizeof(FLOAT))));<br>
+ � � � �if (FAILED(hr))<br>
+ � � � � � �goto end;<br>
+ � �}<br>
+<br>
+ � �hr = D3D_OK;<br>
</blockquote>
<br>
</div></div></blockquote></div><br></div></div>