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 android

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

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