kernel32: Tiny improvement to the GetVolumePathNameW stub (try 2)

Louis. Lenders xerox_xerox2000 at yahoo.co.uk
Sat Aug 16 14:09:06 CDT 2008



Hi Michael, thanks for your comments. I've added some simple sanity checks from your first suggestions.
As for the other suggestions, i'm sure that if this function ever gets implemented all these checks and tests are needed, but as i said, a bit of google for fixme:GetVolumePath shows the common way (the very few) apps call this function is with filename="c:\blabla\bla", so for now , in my opinion, i don't think it's worth the effort.




Hi,  this fixes Adobe Lightroom 2.0 start up bug, mentioned in 
http://bugs.winehq.org/show_bug.cgi?id=8224#c4

as long wine doesn't handle with volume mount points, it's probably safe to satisfy most apps.

(A simple google for 
"fixme:volume:GetVolumePathNameW"
revealed only about 3 apps , and "fixme:volume:GetVolumePathNameA" revealed zero hits, so for now the use of this api seems not really wide-spread, and no need to touch GetVolumePathNameA i guess)

Send instant messages to your online friends http://uk.messenger.yahoo.com 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.winehq.org/pipermail/wine-patches/attachments/20080816/979e8562/attachment.htm 
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: c.txt
Url: http://www.winehq.org/pipermail/wine-patches/attachments/20080816/979e8562/attachment.txt 


More information about the wine-patches mailing list