On 4/23/2010 20:46, Paul Vriens wrote:
On 04/23/2010 06:30 PM, André Hentschel wrote:
Juan Lang schrieb:
My point of view is that its not an error on windows, with this patch it passes in windows and gets a todo_wine, which it actually is (like you say).
Any test that allows all possibilities, as your patch does on Windows, is worthless. That implies that it should be investigated, and, if it's truly a meaningless test, then remove it. --Juan
It still has two benefits applying the patch: it states its a todo on wine and it documents the old behavior of windows as broken.
But who guarantees that the old behavior is broken and not the new one?
I'm with Juan and Nikolay on this one, we need more investigation.
An update here for anyone who cares. I dumped returned pidls (results are here https://testbot.winehq.org/JobDetails.pl?Key=1725&log_101=1#k101). Vista failure caused by SHParseDisplayName returning pidl with a root at UsersFiles (alias for user home directory in Users). Parsing from desktop returns pidl with root at My Computer.
Looks like SHParseDisplayName now uses this virtual folder (user home alias) as a root, that's all, and there's no way to switch to pre-vista behavior.