try2 [1/3] kernel32/tests: Add tests for GetVolumeInformationA and QueryDosDevice
Guy Albertelli
galberte at neo.rr.com
Fri Apr 10 22:06:30 CDT 2009
On Fri, 2009-04-10 at 08:31 +0200, Paul Vriens wrote:
> Guy Albertelli wrote:
> > Enhance QueryDosDevice test for attempting to return all devices.
> > Add tests for GetVolumeInformationA testing the different root dir formats.
> > ---
> > dlls/kernel32/tests/volume.c | 134 +++++++++++++++++++++++++++++++++++++++++-
> > 1 files changed, 131 insertions(+), 3 deletions(-)
> >
> >
> >
> > ------------------------------------------------------------------------
> >
> >
> + char Root_Dir3[]="C:\\Windows\\System\\";
> ..
> + Root_Dir3[0] = windowsdir[0];
>
> This won't work on all systems as the windows directory can be different
> (and can be changed during installation).
>
> You could use GetSystemDirectory() instead of GetWindowsDirectory() to
> find out the drive letter and in the same go use the output from
> GetSystemDirectory() to fill Root_Dir3.
>
> As you're actually interested in an existing directory you could even
> use the output from GetWindowsDirectory() to fill Root_Dir3.
>
You are 100% correct. Will resubmit.
More information about the wine-devel
mailing list