On second thought, don't use the intermediate canvastype variable

This commit is contained in:
Marek Roszko 2020-10-31 16:57:54 -04:00
parent 7203a51fec
commit 98e07a77ef
1 changed files with 2 additions and 6 deletions

View File

@ -993,12 +993,12 @@ void EDA_DRAW_FRAME::GetUnitPair( EDA_UNITS& aPrimaryUnit, EDA_UNITS& aSecondary
void EDA_DRAW_FRAME::ResolveCanvasType() void EDA_DRAW_FRAME::ResolveCanvasType()
{ {
EDA_DRAW_PANEL_GAL::GAL_TYPE savedCanvasType = LoadCanvasTypeSetting(); m_canvasType = LoadCanvasTypeSetting();
// Nudge user to switch to OpenGL if they are on legacy or Cairo // Nudge user to switch to OpenGL if they are on legacy or Cairo
if( m_firstRunDialogSetting < 1 ) if( m_firstRunDialogSetting < 1 )
{ {
if( savedCanvasType != EDA_DRAW_PANEL_GAL::GAL_TYPE_OPENGL ) if( m_canvasType != EDA_DRAW_PANEL_GAL::GAL_TYPE_OPENGL )
{ {
wxString msg = wxString msg =
_( "KiCad can use your graphics card to give you a smoother " _( "KiCad can use your graphics card to give you a smoother "
@ -1034,8 +1034,4 @@ void EDA_DRAW_FRAME::ResolveCanvasType()
m_firstRunDialogSetting = 1; m_firstRunDialogSetting = 1;
SaveSettings( config() ); SaveSettings( config() );
} }
else
{
m_canvasType = savedCanvasType;
}
} }