if ( !value )
{
#ifdef HAVE_UNSETENV
- return unsetenv(variable.mb_str()) == 0;
+ // don't test unsetenv() return value: it's void on some systems (at
+ // least Darwin)
+ unsetenv(variable.mb_str());
+ return true;
#else
value = ""; // we can't pass NULL to setenv()
#endif