mirror of
https://gitlab.com/kicad/code/kicad.git
synced 2025-09-15 10:43:15 +02:00
performance efficiencies
This commit is contained in:
parent
68cbb820a7
commit
8251fca66a
@ -76,7 +76,7 @@ PANEL_SETUP_NETCLASSES::PANEL_SETUP_NETCLASSES( wxWindow* aParentWindow, EDA_DRA
|
|||||||
PANEL_SETUP_NETCLASSES_BASE( aParentWindow ),
|
PANEL_SETUP_NETCLASSES_BASE( aParentWindow ),
|
||||||
m_frame( aFrame ),
|
m_frame( aFrame ),
|
||||||
m_isEEschema( aIsEEschema ),
|
m_isEEschema( aIsEEschema ),
|
||||||
m_netSettings( aNetSettings ),
|
m_netSettings( std::move( aNetSettings ) ),
|
||||||
m_netNames( aNetNames ),
|
m_netNames( aNetNames ),
|
||||||
m_lastCheckedTicker( 0 ),
|
m_lastCheckedTicker( 0 ),
|
||||||
m_hoveredCol( -1 ),
|
m_hoveredCol( -1 ),
|
||||||
@ -788,7 +788,7 @@ void PANEL_SETUP_NETCLASSES::ImportSettingsFrom( const std::shared_ptr<NET_SETTI
|
|||||||
m_netclassGrid->ForceRefresh();
|
m_netclassGrid->ForceRefresh();
|
||||||
m_assignmentGrid->ForceRefresh();
|
m_assignmentGrid->ForceRefresh();
|
||||||
|
|
||||||
m_netSettings = savedSettings;
|
m_netSettings = std::move( savedSettings );
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@ -37,7 +37,7 @@ PANEL_SETUP_SEVERITIES::PANEL_SETUP_SEVERITIES( wxWindow* aParentWindow,
|
|||||||
RC_ITEM* aPinMapSpecialCase ) :
|
RC_ITEM* aPinMapSpecialCase ) :
|
||||||
wxPanel( aParentWindow ),
|
wxPanel( aParentWindow ),
|
||||||
m_severities( aSeverities ),
|
m_severities( aSeverities ),
|
||||||
m_items( aItems ),
|
m_items( std::move( aItems ) ),
|
||||||
m_pinMapSpecialCase( aPinMapSpecialCase )
|
m_pinMapSpecialCase( aPinMapSpecialCase )
|
||||||
{
|
{
|
||||||
wxString severities[] = { _( "Error" ), _( "Warning" ), _( "Ignore" ) };
|
wxString severities[] = { _( "Error" ), _( "Warning" ), _( "Ignore" ) };
|
||||||
|
@ -202,7 +202,7 @@ void PROPERTY_MANAGER::OverrideAvailability( TYPE_ID aDerived, TYPE_ID aBase,
|
|||||||
wxASSERT_MSG( aDerived != aBase, "Class cannot override from itself" );
|
wxASSERT_MSG( aDerived != aBase, "Class cannot override from itself" );
|
||||||
|
|
||||||
CLASS_DESC& derived = getClass( aDerived );
|
CLASS_DESC& derived = getClass( aDerived );
|
||||||
derived.m_availabilityOverrides[std::make_pair( aBase, aName )] = aFunc;
|
derived.m_availabilityOverrides[std::make_pair( aBase, aName )] = std::move( aFunc );
|
||||||
m_dirty = true;
|
m_dirty = true;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -214,7 +214,7 @@ void PROPERTY_MANAGER::OverrideWriteability( TYPE_ID aDerived, TYPE_ID aBase,
|
|||||||
wxASSERT_MSG( aDerived != aBase, "Class cannot override from itself" );
|
wxASSERT_MSG( aDerived != aBase, "Class cannot override from itself" );
|
||||||
|
|
||||||
CLASS_DESC& derived = getClass( aDerived );
|
CLASS_DESC& derived = getClass( aDerived );
|
||||||
derived.m_writeabilityOverrides[std::make_pair( aBase, aName )] = aFunc;
|
derived.m_writeabilityOverrides[std::make_pair( aBase, aName )] = std::move( aFunc );
|
||||||
m_dirty = true;
|
m_dirty = true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -248,7 +248,7 @@ PROPERTY_BASE( const wxString& aName, PROPERTY_DISPLAY aDisplay = PT_DEFAULT,
|
|||||||
*/
|
*/
|
||||||
PROPERTY_BASE& SetAvailableFunc( std::function<bool(INSPECTABLE*)> aFunc )
|
PROPERTY_BASE& SetAvailableFunc( std::function<bool(INSPECTABLE*)> aFunc )
|
||||||
{
|
{
|
||||||
m_availFunc = aFunc;
|
m_availFunc = std::move( aFunc );
|
||||||
return *this;
|
return *this;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -259,7 +259,7 @@ PROPERTY_BASE( const wxString& aName, PROPERTY_DISPLAY aDisplay = PT_DEFAULT,
|
|||||||
|
|
||||||
PROPERTY_BASE& SetWriteableFunc( std::function<bool(INSPECTABLE*)> aFunc )
|
PROPERTY_BASE& SetWriteableFunc( std::function<bool(INSPECTABLE*)> aFunc )
|
||||||
{
|
{
|
||||||
m_writeableFunc = aFunc;
|
m_writeableFunc = std::move( aFunc );
|
||||||
return *this;
|
return *this;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -44,7 +44,7 @@ public:
|
|||||||
void SetFromString( const std::string& aPath, ValueType aVal )
|
void SetFromString( const std::string& aPath, ValueType aVal )
|
||||||
{
|
{
|
||||||
// Calls the overload below, which will convert from dotted string to JSON pointer
|
// Calls the overload below, which will convert from dotted string to JSON pointer
|
||||||
( *this )[aPath] = aVal;
|
( *this )[aPath] = std::move( aVal );
|
||||||
}
|
}
|
||||||
|
|
||||||
template<typename ValueType>
|
template<typename ValueType>
|
||||||
|
@ -946,7 +946,7 @@ void SHAPE_POLY_SET::booleanOp( Clipper2Lib::ClipType aType, const SHAPE_POLY_SE
|
|||||||
//@todo amend X,Y values to true intersection between arcs or arc and segment
|
//@todo amend X,Y values to true intersection between arcs or arc and segment
|
||||||
};
|
};
|
||||||
|
|
||||||
c.SetZCallback( callback ); // register callback
|
c.SetZCallback( std::move( callback ) ); // register callback
|
||||||
|
|
||||||
c.Execute( aType, Clipper2Lib::FillRule::NonZero, solution );
|
c.Execute( aType, Clipper2Lib::FillRule::NonZero, solution );
|
||||||
|
|
||||||
|
@ -123,7 +123,7 @@ void PANEL_SETUP_TUNING_PATTERNS::ImportSettingsFrom( BOARD* aBoard )
|
|||||||
m_skewSettings = aBoard->GetDesignSettings().m_skewMeanderSettings;
|
m_skewSettings = aBoard->GetDesignSettings().m_skewMeanderSettings;
|
||||||
TransferDataToWindow();
|
TransferDataToWindow();
|
||||||
|
|
||||||
m_trackSettings = savedTrackSettings;
|
m_trackSettings = std::move( savedTrackSettings );
|
||||||
m_dpSettings = savedDPSettings;
|
m_dpSettings = std::move( savedDPSettings );
|
||||||
m_skewSettings = savedSkewSettings;
|
m_skewSettings = std::move( savedSkewSettings );
|
||||||
}
|
}
|
||||||
|
@ -36,9 +36,9 @@ DRC_REPORT::DRC_REPORT(BOARD* aBoard, EDA_UNITS aReportUnits,
|
|||||||
std::shared_ptr<RC_ITEMS_PROVIDER> aFpWarningsProvider) :
|
std::shared_ptr<RC_ITEMS_PROVIDER> aFpWarningsProvider) :
|
||||||
m_board( aBoard ),
|
m_board( aBoard ),
|
||||||
m_reportUnits( aReportUnits ),
|
m_reportUnits( aReportUnits ),
|
||||||
m_markersProvider( aMarkersProvider ),
|
m_markersProvider( std::move( aMarkersProvider ) ),
|
||||||
m_ratsnestProvider( aRatsnestProvider ),
|
m_ratsnestProvider( std::move( aRatsnestProvider ) ),
|
||||||
m_fpWarningsProvider( aFpWarningsProvider )
|
m_fpWarningsProvider( std::move( aFpWarningsProvider ) )
|
||||||
{
|
{
|
||||||
|
|
||||||
}
|
}
|
||||||
|
@ -1770,7 +1770,7 @@ int BOARD_INSPECTION_TOOL::HighlightNet( const TOOL_EVENT& aEvent )
|
|||||||
settings->SetHighlight( m_lastHighlighted );
|
settings->SetHighlight( m_lastHighlighted );
|
||||||
m_toolMgr->GetView()->UpdateAllLayersColor();
|
m_toolMgr->GetView()->UpdateAllLayersColor();
|
||||||
m_currentlyHighlighted = m_lastHighlighted;
|
m_currentlyHighlighted = m_lastHighlighted;
|
||||||
m_lastHighlighted = temp;
|
m_lastHighlighted = std::move( temp );
|
||||||
}
|
}
|
||||||
else if( aEvent.IsAction( &PCB_ACTIONS::toggleNetHighlight ) )
|
else if( aEvent.IsAction( &PCB_ACTIONS::toggleNetHighlight ) )
|
||||||
{
|
{
|
||||||
|
@ -117,8 +117,9 @@ public:
|
|||||||
CALLABLE_BASED_HANDLER( CREATION_HANDLER aCreationHandler,
|
CALLABLE_BASED_HANDLER( CREATION_HANDLER aCreationHandler,
|
||||||
MODIFICATION_HANDLER aModificationHandler,
|
MODIFICATION_HANDLER aModificationHandler,
|
||||||
DELETION_HANDLER aDeletionHandler ) :
|
DELETION_HANDLER aDeletionHandler ) :
|
||||||
m_creationHandler( aCreationHandler ),
|
m_creationHandler( std::move( aCreationHandler ) ),
|
||||||
m_modificationHandler( aModificationHandler ), m_deletionHandler( aDeletionHandler )
|
m_modificationHandler( std::move( aModificationHandler ) ),
|
||||||
|
m_deletionHandler( std::move( aDeletionHandler ) )
|
||||||
{
|
{
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -152,7 +153,10 @@ public:
|
|||||||
};
|
};
|
||||||
|
|
||||||
ITEM_MODIFICATION_ROUTINE( BOARD_ITEM* aBoard, CHANGE_HANDLER& aHandler ) :
|
ITEM_MODIFICATION_ROUTINE( BOARD_ITEM* aBoard, CHANGE_HANDLER& aHandler ) :
|
||||||
m_board( aBoard ), m_handler( aHandler ), m_numSuccesses( 0 ), m_numFailures( 0 )
|
m_board( aBoard ),
|
||||||
|
m_handler( aHandler ),
|
||||||
|
m_numSuccesses( 0 ),
|
||||||
|
m_numFailures( 0 )
|
||||||
{
|
{
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user