diff --git a/common/dialogs/dialog_env_var_config.cpp b/common/dialogs/dialog_env_var_config.cpp index 2464932868..12d0657168 100644 --- a/common/dialogs/dialog_env_var_config.cpp +++ b/common/dialogs/dialog_env_var_config.cpp @@ -56,8 +56,6 @@ DIALOG_ENV_VAR_CONFIG::DIALOG_ENV_VAR_CONFIG( wxWindow* aParent, const ENV_VAR_M if( okButton ) SetDefaultItem( okButton ); - - wxLogDebug( wxT( "In DIALOG_ENV_VAR_CONFIG ctor." ) ); } @@ -174,6 +172,8 @@ bool DIALOG_ENV_VAR_CONFIG::TransferDataFromWindow() } } + std::vector< wxString > removeFromMap; + // Remove deleted entries from the map. for( ENV_VAR_MAP_ITER it = m_envVarMap.begin(); it != m_envVarMap.end(); ++it ) { @@ -189,13 +189,12 @@ bool DIALOG_ENV_VAR_CONFIG::TransferDataFromWindow() } if( !found ) - { - m_envVarMap.erase( it ); - it--; - } + removeFromMap.push_back( it->first ); } - wxLogDebug( wxT( "In DIALOG_ENV_VAR_CONFIG::TransferDataFromWindow()." ) ); + for( size_t i = 0; i < removeFromMap.size(); i++ ) + m_envVarMap.erase( removeFromMap[i] ); + return true; } diff --git a/common/pgm_base.cpp b/common/pgm_base.cpp index 981b54b7f9..e927327203 100644 --- a/common/pgm_base.cpp +++ b/common/pgm_base.cpp @@ -798,7 +798,7 @@ void PGM_BASE::ConfigurePaths( wxWindow* aParent ) for( ENV_VAR_MAP_ITER it = envVarMap.begin(); it != envVarMap.end(); ++it ) { - wxLogDebug( wxT( "Environment variable %s=%s defined externally = %d" ), + wxLogTrace( traceEnvVars, wxT( "Environment variable %s=%s defined externally = %d" ), GetChars( it->first ), GetChars( it->second.GetValue() ), it->second.GetDefinedExternally() ); }