James Hawkins wrote: > Hi, > > Anything wrong with these patches? > > Changelog: > * Add tests for GetPrivateProfileString. > > dlls/kernel32/tests/profile.c | 272 > +++++++++++++++++++++++++++++++++++++++++ > 1 files changed, 272 insertions(+), 0 deletions(-) >
There are lots of test crashes because of this patch (tested on W2K, XP-SP3, W2K3 and Vista). It looks like all tests with 'lpAppName is empty' and 'lpKeyName is empty' fail (haven't checked others). Marking them with an if(0) does seem overkill. Any idea? -- Cheers, Paul.
