On Fri Oct 28 19:30:58 2022 +0000, **** wrote:
Zebediah Figura replied on the mailing list:
On 10/26/22 20:34, Mohamad Al-Jaf wrote: > + if (ppidlLast) > + *ppidlLast = NULL; > + > if (!pidl || !ppv) > return E_INVALIDARG; > - > + > *ppv = NULL; > - if (ppidlLast) > - *ppidlLast = NULL; Why are these changes being made? At least they shouldn't be part of this patch.
Yes, you're right. Those changes were made in order to conform to the behavior of Windows and pass the tests, specifically the last test which checks if ppidlLast is NULL after the function returns E_INVALIDARG. Though, this only applied to SHBindToFolderIDListParent. There was no test for SHBindToParent for this behavior. I've added one now.
I'm still not really sure what Alexandre meant by merging the two code change patches. Merging the commits just made it unclear IMO. I've added a separate commit for reimplementing SHBindToParent as well as a separate test commit. Please let me know if this is okay. I can always just start a new merge request for reimplementing SHBindToParent if necessary.