Re: mscoree: Print the debug string and not the pointer to it.

[Lauri Kenttä]

On 2012-03-22 00:36, Michael Stefaniuc wrote:
> HRESULT WINAPI GetRequestedRuntimeVersion(LPWSTR pExe, LPWSTR
> pVersion, DWORD cchBuffer, DWORD *dwlength)
>  {
> -    TRACE("(%s, %p, %d, %p)\n", debugstr_w(pExe), debugstr_w(pExe),
> cchBuffer, dwlength);
> +    TRACE("(%s, %s, %d, %p)\n", debugstr_w(pExe), debugstr_w(pExe),
> cchBuffer, dwlength);

Why is pExe printed two times? Maybe the second one should be pVersion?

-- 
Lauri Kenttä

mscoree: Print the debug string and not the pointer to it.

[Michael Stefaniuc]

On 03/23/2012 07:50 PM, Lauri Kenttä wrote:
> On 2012-03-22 00:36, Michael Stefaniuc wrote:
> > HRESULT WINAPI GetRequestedRuntimeVersion(LPWSTR pExe, LPWSTR
> > pVersion, DWORD cchBuffer, DWORD *dwlength)
> > {
> > - TRACE("(%s, %p, %d, %p)\n", debugstr_w(pExe), debugstr_w(pExe),
> > cchBuffer, dwlength);
> > + TRACE("(%s, %s, %d, %p)\n", debugstr_w(pExe), debugstr_w(pExe),
> > cchBuffer, dwlength);
>
> Why is pExe printed two times? Maybe the second one should be pVersion?
> Good catch. Please feel free to sent a patch that fixes that and 
> supersedes my patch.

thanks
bye
	michael