Dmitry Timoshkov wrote:
"Frank Schruefer" frank.schruefer@t-online.de wrote:
Was this patch forgotten/rejected?
Did you consider to provide a test suite as well to justify the change?
Not really: It looks like there's already an existing test in dlls/kernel/tests/environ.c. This test is broken BTW *without* my patch, it says: ... path.c:756: Test succeeded inside todo block: GetFullPathNameA returned 'C:\shortdir\pathtest.pth' instead of 'C:\shortdir\pathtest.pth' ... which doesn't make any sense anyways.
Justifications for the patch:
-Remark in dlls/kernel/environ.c. Quote: " * WARNING: there's a Windows incompatibility lurking here ! * Win32s always includes the full path of the program file, * whereas Windows NT only returns the full file path plus arguments * in case the program has been started with a full path. * Win9x seems to have inherited NT behaviour. " Although the comment is not accurate, you get what it says and it's exactly what the patch should fix.
-the patch clearly fixes an incompatibility.
-the patch is free ;-)
Shouldn't that be enough *justification*?
Another thing is: I'm neither familiar with C nor wine nor Windows and it took me more than a week of my spare time to make that fix. I'm not gonna spend another week to find out how to write a wine conform test case, sorry.
Well, I'm not mad in any way if you wanna reject it. Just say so.
What I'm doing now is tracking the case until it reached a final state, that's why I asked what happened to it.