diff --git a/common/drawing_sheet/ds_draw_item.cpp b/common/drawing_sheet/ds_draw_item.cpp index 12386e1460..982d782b04 100644 --- a/common/drawing_sheet/ds_draw_item.cpp +++ b/common/drawing_sheet/ds_draw_item.cpp @@ -111,24 +111,23 @@ void DS_DRAW_ITEM_BASE::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, switch( dataItem->GetType() ) { case DS_DATA_ITEM::DS_SEGMENT: - aList.push_back( MSG_PANEL_ITEM( _( "Line" ), msg ) ); + aList.emplace_back( _( "Line" ), wxEmptyString ); break; case DS_DATA_ITEM::DS_RECT: - aList.push_back( MSG_PANEL_ITEM( _( "Rectangle" ), msg ) ); + aList.emplace_back( _( "Rectangle" ), wxEmptyString ); break; case DS_DATA_ITEM::DS_TEXT: - msg = static_cast( this )->GetShownText(); - aList.push_back( MSG_PANEL_ITEM( _( "Text" ), msg ) ); + aList.emplace_back( _( "Text" ), static_cast( this )->GetShownText() ); break; case DS_DATA_ITEM::DS_POLYPOLYGON: - aList.push_back( MSG_PANEL_ITEM( _( "Imported Shape" ), msg ) ); + aList.emplace_back( _( "Imported Shape" ), wxEmptyString ); break; case DS_DATA_ITEM::DS_BITMAP: - aList.push_back( MSG_PANEL_ITEM( _( "Image" ), msg ) ); + aList.emplace_back( _( "Image" ), wxEmptyString ); break; } @@ -139,21 +138,21 @@ void DS_DRAW_ITEM_BASE::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, default: msg = _( "All Pages" ); break; } - aList.push_back( MSG_PANEL_ITEM( _( "First Page Option" ), msg ) ); + aList.emplace_back( _( "First Page Option" ), msg ); msg = MessageTextFromValue( EDA_UNITS::UNSCALED, dataItem->m_RepeatCount ); - aList.push_back( MSG_PANEL_ITEM( _( "Repeat Count" ), msg ) ); + aList.emplace_back( _( "Repeat Count" ), msg ); msg = MessageTextFromValue( EDA_UNITS::UNSCALED, dataItem->m_IncrementLabel ); - aList.push_back( MSG_PANEL_ITEM( _( "Repeat Label Increment" ), msg ) ); + aList.emplace_back( _( "Repeat Label Increment" ), msg ); msg.Printf( wxT( "(%s, %s)" ), MessageTextFromValue( aFrame->GetUserUnits(), dataItem->m_IncrementVector.x ), MessageTextFromValue( aFrame->GetUserUnits(), dataItem->m_IncrementVector.y ) ); - aList.push_back( MSG_PANEL_ITEM( _( "Repeat Position Increment" ), msg ) ); + aList.emplace_back( _( "Repeat Position Increment" ), msg ); - aList.push_back( MSG_PANEL_ITEM( _( "Comment" ), dataItem->m_Info ) ); + aList.emplace_back( _( "Comment" ), dataItem->m_Info ); } diff --git a/eeschema/lib_circle.cpp b/eeschema/lib_circle.cpp index 9eb6e31670..36cd1bf056 100644 --- a/eeschema/lib_circle.cpp +++ b/eeschema/lib_circle.cpp @@ -236,25 +236,21 @@ const EDA_RECT LIB_CIRCLE::GetBoundingBox() const void LIB_CIRCLE::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vector& aList ) { - wxString msg; EDA_RECT bBox = GetBoundingBox(); LIB_ITEM::GetMsgPanelInfo( aFrame, aList ); - msg = MessageTextFromValue( aFrame->GetUserUnits(), m_Width ); + aList.emplace_back( _( "Line Width" ), MessageTextFromValue( aFrame->GetUserUnits(), + m_Width ) ); - aList.push_back( MSG_PANEL_ITEM( _( "Line Width" ), msg ) ); + aList.emplace_back( _( "Radius" ), MessageTextFromValue( aFrame->GetUserUnits(), + GetRadius() ) ); - msg = MessageTextFromValue( aFrame->GetUserUnits(), GetRadius() ); - aList.push_back( MSG_PANEL_ITEM( _( "Radius" ), msg ) ); - - msg.Printf( wxT( "(%d, %d, %d, %d)" ), - bBox.GetOrigin().x, - bBox.GetOrigin().y, - bBox.GetEnd().x, - bBox.GetEnd().y ); - - aList.push_back( MSG_PANEL_ITEM( _( "Bounding Box" ), msg ) ); + aList.emplace_back( _( "Bounding Box" ), wxString::Format( wxT( "(%d, %d, %d, %d)" ), + bBox.GetOrigin().x, + bBox.GetOrigin().y, + bBox.GetEnd().x, + bBox.GetEnd().y ) ); } diff --git a/eeschema/lib_field.cpp b/eeschema/lib_field.cpp index 1409345635..ef82613516 100644 --- a/eeschema/lib_field.cpp +++ b/eeschema/lib_field.cpp @@ -423,19 +423,17 @@ void LIB_FIELD::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vectorGetUserUnits(), GetTextWidth() ); - aList.push_back( MSG_PANEL_ITEM( _( "Text Size" ), msg ) ); + aList.emplace_back( _( "Text Size" ), MessageTextFromValue( aFrame->GetUserUnits(), + GetTextWidth() ) ); switch ( GetHorizJustify() ) { @@ -444,7 +442,7 @@ void LIB_FIELD::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vectorGetUserUnits(), m_length ); - aList.push_back( MSG_PANEL_ITEM( _( "Length" ), text ) ); + aList.emplace_back( _( "Length" ), StringFromValue( aFrame->GetUserUnits(), m_length ) ); - text = PinOrientationName( (unsigned) PinOrientationIndex( m_orientation ) ); - aList.push_back( MSG_PANEL_ITEM( _( "Orientation" ), text ) ); + int i = PinOrientationIndex( m_orientation ); + aList.emplace_back( _( "Orientation" ), PinOrientationName( (unsigned) i ) ); wxPoint pinpos = GetPosition(); - pinpos.y = -pinpos.y; // Display coord are top to bottom - // lib items coord are bottom to top + pinpos.y = -pinpos.y; // Display coords are top to bottom; lib item coords are bottom to top - text = MessageTextFromValue( aFrame->GetUserUnits(), pinpos.x ); - aList.push_back( MSG_PANEL_ITEM( _( "Pos X" ), text ) ); - - text = MessageTextFromValue( aFrame->GetUserUnits(), pinpos.y ); - aList.push_back( MSG_PANEL_ITEM( _( "Pos Y" ), text ) ); + aList.emplace_back( _( "Pos X" ), MessageTextFromValue( aFrame->GetUserUnits(), pinpos.x ) ); + aList.emplace_back( _( "Pos Y" ), MessageTextFromValue( aFrame->GetUserUnits(), pinpos.y ) ); } diff --git a/eeschema/lib_polyline.cpp b/eeschema/lib_polyline.cpp index b3d9409aca..0b3faa3669 100644 --- a/eeschema/lib_polyline.cpp +++ b/eeschema/lib_polyline.cpp @@ -330,19 +330,17 @@ void LIB_POLYLINE::DeleteSegment( const wxPoint aPosition ) void LIB_POLYLINE::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vector& aList ) { - wxString msg; EDA_RECT bBox = GetBoundingBox(); LIB_ITEM::GetMsgPanelInfo( aFrame, aList ); - msg = MessageTextFromValue( aFrame->GetUserUnits(), m_Width ); + aList.emplace_back( _( "Line Width" ), MessageTextFromValue( aFrame->GetUserUnits(), m_Width ) ); - aList.push_back( MSG_PANEL_ITEM( _( "Line Width" ), msg ) ); - - msg.Printf( wxT( "(%d, %d, %d, %d)" ), bBox.GetOrigin().x, - bBox.GetOrigin().y, bBox.GetEnd().x, bBox.GetEnd().y ); - - aList.push_back( MSG_PANEL_ITEM( _( "Bounding Box" ), msg ) ); + aList.emplace_back( _( "Bounding Box" ), wxString::Format( wxT( "(%d, %d, %d, %d)" ), + bBox.GetOrigin().x, + bBox.GetOrigin().y, + bBox.GetEnd().x, + bBox.GetEnd().y ) ); } diff --git a/eeschema/lib_rectangle.cpp b/eeschema/lib_rectangle.cpp index 44b5148bd1..14e03cc259 100644 --- a/eeschema/lib_rectangle.cpp +++ b/eeschema/lib_rectangle.cpp @@ -185,9 +185,7 @@ void LIB_RECTANGLE::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vectorGetUserUnits(), m_Width ); - - aList.push_back( MSG_PANEL_ITEM( _( "Line Width" ), msg ) ); + aList.emplace_back( _( "Line Width" ), MessageTextFromValue( aFrame->GetUserUnits(), m_Width ) ); } diff --git a/eeschema/lib_text.cpp b/eeschema/lib_text.cpp index 904c997f77..cfc51b5247 100644 --- a/eeschema/lib_text.cpp +++ b/eeschema/lib_text.cpp @@ -346,13 +346,12 @@ void LIB_TEXT::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vectorGetUserUnits(), GetTextWidth() ); - aList.push_back( MSG_PANEL_ITEM( _( "Text Size" ), msg ) ); + aList.emplace_back( _( "Text Size" ), MessageTextFromValue( aFrame->GetUserUnits(), + GetTextWidth() ) ); switch ( GetHorizJustify() ) { @@ -361,7 +360,7 @@ void LIB_TEXT::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vector& aList ) { - aList.push_back( MSG_PANEL_ITEM( _( "Bitmap" ), wxEmptyString ) ); + aList.emplace_back( _( "Bitmap" ), wxEmptyString ); - aList.push_back( MSG_PANEL_ITEM( _( "Width" ), MessageTextFromValue( aFrame->GetUserUnits(), - GetSize().x ) ) ); - aList.push_back( MSG_PANEL_ITEM( _( "Height" ), MessageTextFromValue( aFrame->GetUserUnits(), - GetSize().y ) ) ); + aList.emplace_back( _( "Width" ), MessageTextFromValue( aFrame->GetUserUnits(), GetSize().x ) ); + aList.emplace_back( _( "Height" ), MessageTextFromValue( aFrame->GetUserUnits(), GetSize().y ) ); } diff --git a/eeschema/sch_bus_entry.cpp b/eeschema/sch_bus_entry.cpp index f718f0dfe6..3ac592e415 100644 --- a/eeschema/sch_bus_entry.cpp +++ b/eeschema/sch_bus_entry.cpp @@ -490,7 +490,7 @@ void SCH_BUS_ENTRY_BASE::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, case LAYER_BUS: msg = _( "Bus" ); break; } - aList.push_back( MSG_PANEL_ITEM( _( "Bus Entry Type" ), msg ) ); + aList.emplace_back( _( "Bus Entry Type" ), msg ); SCH_CONNECTION* conn = dynamic_cast( aFrame ) ? Connection() : nullptr; @@ -507,7 +507,7 @@ void SCH_BUS_ENTRY_BASE::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, if( netSettings.m_NetClassAssignments.count( netname ) ) netclassName = netSettings.m_NetClassAssignments[ netname ]; - aList.push_back( MSG_PANEL_ITEM( _( "Assigned Netclass" ), netclassName ) ); + aList.emplace_back( _( "Assigned Netclass" ), netclassName ); } } } diff --git a/eeschema/sch_connection.cpp b/eeschema/sch_connection.cpp index 35005a6fc7..b1a6ba4925 100644 --- a/eeschema/sch_connection.cpp +++ b/eeschema/sch_connection.cpp @@ -398,47 +398,40 @@ void SCH_CONNECTION::SetSuffix( const wxString& aSuffix ) void SCH_CONNECTION::AppendInfoToMsgPanel( std::vector& aList ) const { - wxString msg, group_name; + wxString msg, group_name, members; std::vector group_members; - aList.push_back( MSG_PANEL_ITEM( _( "Connection Name" ), UnescapeString( Name() ) ) ); + aList.emplace_back( _( "Connection Name" ), UnescapeString( Name() ) ); // NOTE(JE) Disabling this for now, because net codes are generated in the netlist exporter // in order to avoid sort costs. It may make sense to just tear out net codes from the // CONNECTION_GRAPH entirely in the future, as they are mostly only useful for netlist exports. #if 0 if( !IsBus() ) - { - msg.Printf( "%d", m_net_code ); - aList.push_back( MSG_PANEL_ITEM( _( "Net Code" ), msg ) ); - } + aList.emplace_back( _( "Net Code" ), wxString::Format( "%d", m_net_code ) ); #endif if( auto alias = m_graph->GetBusAlias( m_name ) ) { msg.Printf( _( "Bus Alias %s Members" ), m_name ); - wxString members; - - for( const auto& member : alias->Members() ) + for( const wxString& member : alias->Members() ) members << member << " "; - aList.push_back( MSG_PANEL_ITEM( msg, members ) ); + aList.emplace_back( msg, members ); } else if( NET_SETTINGS::ParseBusGroup( m_name, &group_name, &group_members ) ) { - for( const auto& group_member : group_members ) + for( const wxString& group_member : group_members ) { - if( auto group_alias = m_graph->GetBusAlias( group_member ) ) + if( std::shared_ptr group_alias = m_graph->GetBusAlias( group_member ) ) { msg.Printf( _( "Bus Alias %s Members" ), group_alias->GetName() ); - wxString members; - - for( const auto& member : group_alias->Members() ) + for( const wxString& member : group_alias->Members() ) members << member << " "; - aList.push_back( MSG_PANEL_ITEM( msg, members ) ); + aList.emplace_back( msg, members ); } } } @@ -450,18 +443,14 @@ void SCH_CONNECTION::AppendInfoToMsgPanel( std::vector& aList ) return; if( IsBus() ) - { - msg.Printf( "%d", m_bus_code ); - aList.push_back( MSG_PANEL_ITEM( "Bus Code", msg ) ); - } + aList.emplace_back( "Bus Code", wxString::Format( "%d", m_bus_code ) ); - msg.Printf( "%d", m_subgraph_code ); - aList.push_back( MSG_PANEL_ITEM( "Subgraph Code", msg ) ); + aList.emplace_back( "Subgraph Code", wxString::Format( "%d", m_subgraph_code ) ); - if( auto driver = Driver() ) + if( SCH_ITEM* driver = Driver() ) { msg.Printf( "%s at %p", driver->GetSelectMenuText( EDA_UNITS::MILLIMETRES ), driver ); - aList.push_back( MSG_PANEL_ITEM( "Connection Source", msg ) ); + aList.emplace_back( "Connection Source", msg ); } #endif } diff --git a/eeschema/sch_field.cpp b/eeschema/sch_field.cpp index f195215207..2745cf2c08 100644 --- a/eeschema/sch_field.cpp +++ b/eeschema/sch_field.cpp @@ -481,19 +481,17 @@ void SCH_FIELD::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vectorGetUserUnits(), GetTextWidth() ); - aList.push_back( MSG_PANEL_ITEM( _( "Text Size" ), msg ) ); + aList.emplace_back( _( "Text Size" ), MessageTextFromValue( aFrame->GetUserUnits(), + GetTextWidth() ) ); switch ( GetHorizJustify() ) { @@ -502,7 +500,7 @@ void SCH_FIELD::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vector( aFrame ) ? Connection() : nullptr; @@ -905,7 +905,7 @@ void SCH_LINE::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vector& aList ) { - aList.push_back( MSG_PANEL_ITEM( _( "Electrical Rule Check Error" ), - m_rcItem->GetErrorMessage() ) ); + aList.emplace_back( _( "Electrical Rule Check Error" ), m_rcItem->GetErrorMessage() ); } diff --git a/eeschema/sch_pin.cpp b/eeschema/sch_pin.cpp index 3d15f2199f..ee87f61b92 100644 --- a/eeschema/sch_pin.cpp +++ b/eeschema/sch_pin.cpp @@ -181,14 +181,14 @@ void SCH_PIN::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vectorGetUnit() == 0 ) msg = _( "All" ); else msg.Printf( wxT( "%d" ), m_libPin->GetUnit() ); - aList.push_back( MSG_PANEL_ITEM( _( "Unit" ), msg ) ); + aList.emplace_back( _( "Unit" ), msg ); if( m_libPin->GetConvert() == LIB_ITEM::LIB_CONVERT::BASE ) msg = _( "no" ); @@ -197,24 +197,19 @@ void SCH_PIN::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vectorGetUserUnits(), GetLength() ) ); - // Display pin length - msg = StringFromValue( aFrame->GetUserUnits(), GetLength() ); - aList.push_back( MSG_PANEL_ITEM( _( "Length" ), msg ) ); - - msg = PinOrientationName( (unsigned) PinOrientationIndex( GetOrientation() ) ); - aList.push_back( MSG_PANEL_ITEM( _( "Orientation" ), msg ) ); + int i = PinOrientationIndex( GetOrientation() ); + aList.emplace_back( _( "Orientation" ), PinOrientationName( (unsigned) i ) ); SCH_EDIT_FRAME* schframe = dynamic_cast( aFrame ); SCH_SHEET_PATH* currentSheet = schframe ? &schframe->GetCurrentSheet() : nullptr; @@ -223,17 +218,13 @@ void SCH_PIN::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vectorGetRef( currentSheet ), symbol->GetValue( currentSheet, true ) ); #if defined(DEBUG) + if( dynamic_cast( aFrame ) ) + { + SCH_CONNECTION* conn = Connection(); - SCH_EDIT_FRAME* frame = dynamic_cast( aFrame ); - - if( !frame ) - return; - - SCH_CONNECTION* conn = Connection(); - - if( conn ) - conn->AppendInfoToMsgPanel( aList ); - + if( conn ) + conn->AppendInfoToMsgPanel( aList ); + } #endif } diff --git a/eeschema/sch_symbol.cpp b/eeschema/sch_symbol.cpp index 7b51f72cf0..d51d66b61b 100644 --- a/eeschema/sch_symbol.cpp +++ b/eeschema/sch_symbol.cpp @@ -1373,20 +1373,17 @@ void SCH_SYMBOL::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vectorIsPower() ? _( "Power symbol" ) : _( "Value" ); - - aList.push_back( MSG_PANEL_ITEM( msg, GetValue( currentSheet, true ) ) ); + aList.emplace_back( msg, GetValue( currentSheet, true ) ); #if 0 // Display symbol flags, for debug only - aList.push_back( MSG_PANEL_ITEM( _( "flags" ), - wxString::Format( "%X", GetEditFlags() ) ) ); + aList.emplace_back( _( "flags" ), wxString::Format( "%X", GetEditFlags() ) ); #endif // Display symbol reference in library and library - aList.push_back( MSG_PANEL_ITEM( _( "Name" ), - UnescapeString( GetLibId().GetLibItemName() ) ) ); + aList.emplace_back( _( "Name" ), UnescapeString( GetLibId().GetLibItemName() ) ); if( !m_part->IsRoot() ) { @@ -1397,15 +1394,15 @@ void SCH_SYMBOL::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vectorGetName(); - aList.push_back( MSG_PANEL_ITEM( _( "Alias of" ), UnescapeString( msg ) ) ); + aList.emplace_back( _( "Alias of" ), UnescapeString( msg ) ); } else if( !m_lib_id.GetLibNickname().empty() ) { - aList.push_back( MSG_PANEL_ITEM( _( "Library" ), m_lib_id.GetLibNickname() ) ); + aList.emplace_back( _( "Library" ), m_lib_id.GetLibNickname() ); } else { - aList.push_back( MSG_PANEL_ITEM( _( "Library" ), _( "Undefined!!!" ) ) ); + aList.emplace_back( _( "Library" ), _( "Undefined!!!" ) ); } // Display the current associated footprint, if exists. @@ -1414,32 +1411,28 @@ void SCH_SYMBOL::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vector" ); - aList.push_back( MSG_PANEL_ITEM( _( "Footprint" ), msg ) ); + aList.emplace_back( _( "Footprint" ), msg ); // Display description of the symbol, and keywords found in lib - aList.push_back( MSG_PANEL_ITEM( _( "Description" ), m_part->GetDescription(), - DARKCYAN ) ); - aList.push_back( MSG_PANEL_ITEM( _( "Keywords" ), m_part->GetKeyWords() ) ); + aList.emplace_back( _( "Description" ), m_part->GetDescription() ); + aList.emplace_back( _( "Keywords" ), m_part->GetKeyWords() ); } } else { - aList.push_back( MSG_PANEL_ITEM( _( "Reference" ), GetRef( currentSheet ) ) ); + aList.emplace_back( _( "Reference" ), GetRef( currentSheet ) ); - aList.push_back( MSG_PANEL_ITEM( _( "Value" ), GetValue( currentSheet, true ) ) ); - aList.push_back( MSG_PANEL_ITEM( _( "Name" ), GetLibId().GetLibItemName() ) ); + aList.emplace_back( _( "Value" ), GetValue( currentSheet, true ) ); + aList.emplace_back( _( "Name" ), GetLibId().GetLibItemName() ); wxString libNickname = GetLibId().GetLibNickname(); if( libNickname.empty() ) - { - aList.push_back( MSG_PANEL_ITEM( _( "Library" ), _( "No library defined!" ) ) ); - } + msg = _( "No library defined!" ); else - { msg.Printf( _( "Symbol not found in %s!" ), libNickname ); - aList.push_back( MSG_PANEL_ITEM( _( "Library" ), msg ) ); - } + + aList.emplace_back( _( "Library" ), msg ); } } diff --git a/eeschema/sch_text.cpp b/eeschema/sch_text.cpp index 1a712b14ec..a1e1c782b0 100644 --- a/eeschema/sch_text.cpp +++ b/eeschema/sch_text.cpp @@ -750,14 +750,11 @@ void SCH_TEXT::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vectorGetUserUnits(), GetTextWidth() ); - aList.push_back( MSG_PANEL_ITEM( _( "Text Size" ), msg ) ); + aList.emplace_back( _( "Text Size" ), MessageTextFromValue( aFrame->GetUserUnits(), + GetTextWidth() ) ); switch( GetLabelSpinStyle() ) { @@ -783,7 +779,7 @@ void SCH_TEXT::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vector( aFrame ) ? Connection() : nullptr; @@ -799,7 +795,7 @@ void SCH_TEXT::GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vectorGetName() ) ); - #if 0 // Enable for debugging +#if 0 // Enable for debugging if( GetBoard() ) - { - // Display net code: - msg.Printf( wxT( "%d" ), GetNetCode() ); - aList.emplace_back( _( "NetCode" ), msg ); - } + aList.emplace_back( _( "NetCode" ), wxString::Format( wxT( "%d" ), GetNetCode() ) ); - // Display the flags: - msg.Printf( wxT( "0x%08X" ), m_flags ); - aList.emplace_back( wxT( "Flags" ), msg ); + aList.emplace_back( wxT( "Flags" ), wxString::Format( wxT( "0x%08X" ), m_flags ) ); - // Display start and end positions: - msg.Printf( wxT( "%d %d" ), m_Start.x, m_Start.y ); - aList.push_back( MSG_PANEL_ITEM( wxT( "Start pos" ), msg ) ); - msg.Printf( wxT( "%d %d" ), m_End.x, m_End.y ); - aList.push_back( MSG_PANEL_ITEM( wxT( "End pos" ), msg ) ); + aList.emplace_back( wxT( "Start pos" ), wxString::Format( wxT( "%d %d" ), + m_Start.x, + m_Start.y ) ); + aList.emplace_back( wxT( "End pos" ), wxString::Format( wxT( "%d %d" ), + m_End.x, + m_End.y ) ); #endif // Display the State member