Hi Paul,
Thanks for the help with the tests. Here is a new set of tests. I'll send the patch that makes these tests succeed shortly. I would appreciate it if you could have a look at it.
Kind regards Alexandre
On Fri, Dec 11, 2009 at 12:25 PM, Alexandre Hardy alexandre.hardy@gmail.com wrote:
Hi Paul,
Don't think this memset makes sense as you do lstrcpyW after that (and yes it does not make sense in the A-test I did either ;) ).
Okay, that can be removed...
You're saying that length should be 0 but the test tests something else.
WIll correct that (copy and paste -> I was in too much of a hurry)
- memset(shortpath, 0, MAX_PATH * sizeof(WCHAR));
- memset(longpath, 0, MAX_PATH * sizeof(WCHAR));
I should remove that as well :)
You're setting longpath but it's not used anymore after that.
- lstrcpyW(shortpath, uncprefix);
- GetShortPathNameW(shortpath + 4, tempfile, MAX_PATH - 4);
- length = pGetLongPathNameW(shortpath,NULL,0);
This doesn't make sense to me. You are fill tempfile but not using it afterwards and then you call pGetLongPathNameW with NULL?
Yes, sorry, that should be GetShortPathNameW(tempfile, shortpath + 4, MAX_PATH - 4);
These are added tests but Wine crashes right now with these?
Yes, wine crashes right now with these.
Kind regards Alexandre
--
Alexandre Hardy http://www.ahardy.za.net