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?

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