Re: setupapi: Make sure machine name is non-empty before failing

2011-06-09 Thread Dmitry Timoshkov
Juan Lang juan.l...@gmail.com wrote: +static const WCHAR emptyW[] = { 0 }; ... -if (MachineName != NULL) +if (MachineName != NULL strcmpW(MachineName, emptyW)) What's wrong with 'if (MachineName *MachineName)' ? -- Dmitry.

Re: setupapi: Make sure machine name is non-empty before failing

2011-06-09 Thread Juan Lang
What's wrong with 'if (MachineName *MachineName)' ? Nothing, it's my brain that's lacking. Thanks. --Juan

Re: setupapi: Make sure machine name is non-empty before failing (try 2)

2011-06-09 Thread Marvin
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=11589 Your paranoid