Maciej Katafiasz wrote:
+ if(GetLastError()) + { + TRACE("Can't convert to WCHAR: %d\n", GetLastError()); + goto clean_wfn; + }This is not an appropriate way of testing for an API failure.What is inappropriate and how should I fix it?
Most APIs don't change GLE on success. Some do but most often the behaviour is undocumented or changed between 9x/NT. You should check the return value which usually is either 0 or -1 (depends on the API) on failure.
Felix