Subject: Re: [boost] [filesystem] temp_dir_path()
From: Jeff Flinn (TriumphSprint2000_at_[hidden])
Date: 2010-10-12 12:42:21
Stewart, Robert wrote:
> Jeff Flinn wrote:
>> I've updated ticket#4724 with temp_dir_path2.patch which
>> reflects the discussion in this thread.
>>
>> POSIX now uses the first environment variable from the list
>> TMPDIR, TMP, TEMP, TEMPDIR. Windows still uses GetTempPath.
>>
>> Error handling is consistent with the rest of filesystem.
>>
>> operations_test.hpp tests the existence of the path returned by
>> temp_dir_path, and the ability to create a file, write to it,
>> read from it, and remove it from the directory.
>
> Did you test the various combinations of e-vars being set?
Yes, manually(not within operations_test.cpp) on mac by
setting/unsetting the environment vars, not on windows though as I've
not been able to make GetTempPath to fail. Not sure if mucking with the
environ is acceptable to Beman's tests.
> How are you testing the error handling code?
Same as above. Any thoughts on how to proceed?
Thanks, Jeff