Minor cleanup and performance.

This commit is contained in:
Jeff Young 2020-01-09 22:55:46 +00:00
parent ae74815ff0
commit cc5a42ff41
1 changed files with 6 additions and 6 deletions

View File

@ -135,7 +135,7 @@ enum GROUP_TYPE
struct DATA_MODEL_ROW struct DATA_MODEL_ROW
{ {
DATA_MODEL_ROW( SCH_REFERENCE aFirstReference, GROUP_TYPE aType ) DATA_MODEL_ROW( const SCH_REFERENCE& aFirstReference, GROUP_TYPE aType )
{ {
m_Refs.push_back( aFirstReference ); m_Refs.push_back( aFirstReference );
m_Flag = aType; m_Flag = aType;
@ -215,7 +215,7 @@ public:
int GetNumberRows() override { return m_rows.size(); } int GetNumberRows() override { return m_rows.size(); }
// Columns are fieldNames + quantity column // Columns are fieldNames + quantity column
int GetNumberCols() override { return m_fieldNames.size() + 1; } int GetNumberCols() override { return (int) m_fieldNames.size() + 1; }
wxString GetColLabelValue( int aCol ) override wxString GetColLabelValue( int aCol ) override
@ -835,7 +835,7 @@ bool DIALOG_FIELDS_EDITOR_GLOBAL::TransferDataToWindow()
std::vector<SCH_REFERENCE> references = m_dataModel->GetRowReferences( row ); std::vector<SCH_REFERENCE> references = m_dataModel->GetRowReferences( row );
bool found = false; bool found = false;
for( SCH_REFERENCE ref : references ) for( const SCH_REFERENCE& ref : references )
{ {
if( ref.GetComp() == component ) if( ref.GetComp() == component )
{ {
@ -888,9 +888,9 @@ void DIALOG_FIELDS_EDITOR_GLOBAL::AddField( const wxString& aName,
m_config->Read( "SymbolFieldEditor/Show/" + aName, &defaultShow ); m_config->Read( "SymbolFieldEditor/Show/" + aName, &defaultShow );
m_config->Read( "SymbolFieldEditor/GroupBy/" + aName, &defaultSortBy ); m_config->Read( "SymbolFieldEditor/GroupBy/" + aName, &defaultSortBy );
fieldsCtrlRow.push_back( wxVariant( aName ) ); fieldsCtrlRow.emplace_back( wxVariant( aName ) );
fieldsCtrlRow.push_back( wxVariant( defaultShow ) ); fieldsCtrlRow.emplace_back( wxVariant( defaultShow ) );
fieldsCtrlRow.push_back( wxVariant( defaultSortBy ) ); fieldsCtrlRow.emplace_back( wxVariant( defaultSortBy ) );
m_fieldsCtrl->AppendItem( fieldsCtrlRow ); m_fieldsCtrl->AppendItem( fieldsCtrlRow );
} }