Re: [PATCH] scrrun: Add MoveFile().

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] scrrun: Add MoveFile().

Fabian Maurer

> + TRACE("%p %s %s\n", iface, debugstr_w(src), debugstr_w(dest));

> +

> + if (MoveFileW(src, dest))

> + return S_OK;

> +

> + error = GetLastError();

> + switch (error)

> + {

> + case ERROR_ALREADY_EXISTS:

> + return CTL_E_FILEALREADYEXISTS;

> + case ERROR_FILE_NOT_FOUND:

> + return CTL_E_FILENOTFOUND;

> + default:

> + return HRESULT_FROM_WIN32(error);

> + }

 

That doesn't seem quite correct.

As the docs I linked from https://bugs.winehq.org/show_bug.cgi?id=45839 state, this function also accepts wildcards. On my win7 machine that works, but MoveFileW can't handle this.

 

Regards,

Fabian Maurer