On 4/24/07, Kirill K. Smirnov <[EMAIL PROTECTED]> wrote:
Hi,
I've sent the patch:
http://www.winehq.com/pipermail/wine-patches/2007-April/038205.html
about a week ago and it has not been applied. Please, explain, what is wrong
with it?
You haven't sent in any tests for this function. Is there some app
that needs it?
+ static const WCHAR env_name[] =
{'N','o','D','e','f','a','u','l','t','C','u','r','r','e','n','t','D','i','r','e','c','t','o','r','y','I','n','E','x','e','P','a','t','h',0};
That's a really long line. We try to keep the lines to a length of 80.
+ /* Check name first */
+ if (strchrW(name, '\\'))
+ return TRUE;
You should probably check name for NULL, but you'll have to test how
Windows handles this. That's a pretty bad comment too; check name for
what?
--
James Hawkins
