Hello Alexandre, I'm pretty sure that my patch contained only the second chunk of the patch commited. The first part seems to have been credited to me by mistake.
Alexandre Julliard wrote: > Module: wine > Branch: master > Commit: d6359edd3cbe366c71f22e09e27d3e427520f609 > URL: > http://source.winehq.org/git/wine.git/?a=commit;h=d6359edd3cbe366c71f22e09e27d3e427520f609 > > Author: Michael Stefaniuc <[EMAIL PROTECTED]> > Date: Tue Oct 14 16:32:33 2008 +0200 > > shlwapi: Remove break after return (Smatch). > > --- > > dlls/shlwapi/assoc.c | 27 +++++++++++++++++++++++++-- > 1 files changed, 25 insertions(+), 2 deletions(-) > > diff --git a/dlls/shlwapi/assoc.c b/dlls/shlwapi/assoc.c > index 7f71708..9e4b036 100644 > --- a/dlls/shlwapi/assoc.c > +++ b/dlls/shlwapi/assoc.c > @@ -627,7 +627,31 @@ static HRESULT > ASSOC_GetExecutable(IQueryAssociationsImpl *This, > if (!pszExtra) > { > hr = ASSOC_GetValue(hkeyShell, &pszExtraFromReg); > - if (FAILED(hr)) > + /* if no default action */ > + if (hr == E_FAIL || hr == HRESULT_FROM_WIN32(ERROR_FILE_NOT_FOUND)) > + { > + DWORD rlen; > + ret = RegQueryInfoKeyW(hkeyShell, 0, 0, 0, 0, &rlen, 0, 0, 0, 0, 0, 0); > + if (ret != ERROR_SUCCESS) > + { > + RegCloseKey(hkeyShell); > + return HRESULT_FROM_WIN32(ret); > + } > + rlen++; > + pszExtraFromReg = HeapAlloc(GetProcessHeap(), 0, rlen * sizeof(WCHAR)); > + if (!pszExtraFromReg) > + { > + RegCloseKey(hkeyShell); > + return E_OUTOFMEMORY; > + } > + ret = RegEnumKeyExW(hkeyShell, 0, pszExtraFromReg, &rlen, 0, NULL, > NULL, NULL); > + if (ret != ERROR_SUCCESS) > + { > + RegCloseKey(hkeyShell); > + return HRESULT_FROM_WIN32(ret); > + } > + } > + else if (FAILED(hr)) > { > RegCloseKey(hkeyShell); > return hr; > @@ -743,7 +767,6 @@ static HRESULT WINAPI IQueryAssociations_fnGetString( > return hr; > len++; > return ASSOC_ReturnData(pszOut, pcchOut, path, len); > - break; > } > > case ASSOCSTR_FRIENDLYAPPNAME: > > > bye michael
