<div class="gmail_quote">On Tue, May 29, 2012 at 7:07 PM, Alexandre Julliard <span dir="ltr"><<a href="mailto:julliard@winehq.org" target="_blank">julliard@winehq.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">Alexandre Goujon <<a href="mailto:ale.goujon@gmail.com">ale.goujon@gmail.com</a>> writes:<br>
<br>
> @@ -729,6 +731,8 @@ BOOL WINAPI GetVolumeInformationW( LPCWSTR root, LPWSTR label, DWORD label_len,<br>
>          }<br>
>          else<br>
>          {<br>
> +            /* At this point, last error is ERROR_NOT_SUPPORTED. Restoring the previous one */<br>
> +            SetLastError(lastError);<br>
<br>
</div>You should never need to do that sort of thing, there are no guarantees<br>
about last error on success.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Alexandre Julliard<br>
<a href="mailto:julliard@winehq.org">julliard@winehq.org</a><br>
</font></span></blockquote></div>I know but The Sims 3 calls GetLastError very often (in the top 10), nearly between each call, maybe to check if a specific error does (or doesn't) occur.<br>It troubled me when I investigated a bug and I thought the cost was so low that we could do it anyway.<br>

But I know you are right, so just forget about it.<br><br>PS: FYI I tested it on WIndows and the last error is not changed (but it doesn't matter as you said)<br>