From 8d71bdc3e10e340c2245e4eb924d609ebc5bf8c6 Mon Sep 17 00:00:00 2001 From: Ian McInerney Date: Fri, 8 Oct 2021 21:37:39 +0100 Subject: [PATCH] Cleanup unit handling in the stackup drawing tool --- pcbnew/tools/drawing_stackup_table_tool.cpp | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/pcbnew/tools/drawing_stackup_table_tool.cpp b/pcbnew/tools/drawing_stackup_table_tool.cpp index 652b540d2d..28a82e5e71 100644 --- a/pcbnew/tools/drawing_stackup_table_tool.cpp +++ b/pcbnew/tools/drawing_stackup_table_tool.cpp @@ -268,14 +268,13 @@ std::vector DRAWING_TOOL::DrawSpecificationStackup( const wxPoint& t = static_cast( headStyle->Duplicate() ); - if( m_frame->GetUserUnits() == EDA_UNITS::MILLIMETRES ) - t->SetText( _( "Thickness (mm)" ) ); - - else if( m_frame->GetUserUnits() == EDA_UNITS::INCHES ) - t->SetText( _( "Thickness (inches)" ) ); - - else if( m_frame->GetUserUnits() == EDA_UNITS::MILS ) - t->SetText( _( "Thickness (mils)" ) ); + switch( m_frame->GetUserUnits() ) + { + case EDA_UNITS::MILLIMETRES: t->SetText( _( "Thickness (mm)" ) ); break; + case EDA_UNITS::INCHES: t->SetText( _( "Thickness (inches)" ) ); break; + case EDA_UNITS::MILS: t->SetText( _( "Thickness (mils)" ) ); break; + default: wxFAIL_MSG( "Unhandled unit type" ); + } colThickness.push_back( t );