haskal
/
kicad
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

(Very) minor performance improvement.

This commit is contained in:
Jeff Young 2023-07-15 17:55:12 +01:00
parent 1218f61d0a
commit 0446152d15
1 changed files with 6 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
pcbnew/router/pns_kicad_iface.cpp
View File

@ -239,14 +239,18 @@ bool PNS_PCBNEW_RULE_RESOLVER::IsKeepout( const PNS::ITEM* aA, const PNS::ITEM*
return false; return false;
}; };
if( const ZONE* zoneA = dynamic_cast<ZONE*>( aA->Parent() ) ) if( aA->Parent() && aA->Parent()->Type() == PCB_ZONE_T )
{ {
const ZONE* zoneA = static_cast<ZONE*>( aA->Parent() );
if( zoneA->GetIsRuleArea() && aB->Parent() ) if( zoneA->GetIsRuleArea() && aB->Parent() )
return checkKeepout( zoneA, aB->Parent() ); return checkKeepout( zoneA, aB->Parent() );
} }
if( const ZONE* zoneB = dynamic_cast<ZONE*>( aB->Parent() ) ) if( aB->Parent() && aB->Parent()->Type() == PCB_ZONE_T )
{ {
const ZONE* zoneB = static_cast<ZONE*>( aB->Parent() );
if( zoneB->GetIsRuleArea() && aA->Parent() ) if( zoneB->GetIsRuleArea() && aA->Parent() )
return checkKeepout( zoneB, aA->Parent() ); return checkKeepout( zoneB, aA->Parent() );
} }
@ -915,14 +919,12 @@ public:
SetView( aView ); SetView( aView );
} }
~PNS_PCBNEW_DEBUG_DECORATOR() ~PNS_PCBNEW_DEBUG_DECORATOR()
{ {
PNS_PCBNEW_DEBUG_DECORATOR::Clear(); PNS_PCBNEW_DEBUG_DECORATOR::Clear();
delete m_items; delete m_items;
} }
void SetView( KIGFX::VIEW* aView ) void SetView( KIGFX::VIEW* aView )
{ {
Clear(); Clear();
@ -941,7 +943,6 @@ public:
m_view->Add( m_items ); m_view->Add( m_items );
} }
void AddPoint( const VECTOR2I& aP, const KIGFX::COLOR4D& aColor, int aSize, void AddPoint( const VECTOR2I& aP, const KIGFX::COLOR4D& aColor, int aSize,
const wxString& aName = wxT( "" ), const wxString& aName = wxT( "" ),
const SRC_LOCATION_INFO& aSrcLoc = SRC_LOCATION_INFO() ) override const SRC_LOCATION_INFO& aSrcLoc = SRC_LOCATION_INFO() ) override
@ -960,7 +961,6 @@ public:
AddShape( &sh, aColor, sh.Width(), aName, aSrcLoc ); AddShape( &sh, aColor, sh.Width(), aName, aSrcLoc );
} }
void AddItem( const PNS::ITEM* aItem, const KIGFX::COLOR4D& aColor, int aOverrideWidth = 0, void AddItem( const PNS::ITEM* aItem, const KIGFX::COLOR4D& aColor, int aOverrideWidth = 0,
const wxString& aName = wxT( "" ), const wxString& aName = wxT( "" ),
const SRC_LOCATION_INFO& aSrcLoc = SRC_LOCATION_INFO() ) override const SRC_LOCATION_INFO& aSrcLoc = SRC_LOCATION_INFO() ) override
@ -978,7 +978,6 @@ public:
m_view->Update( m_items ); m_view->Update( m_items );
} }
void AddShape( const BOX2I& aBox, const KIGFX::COLOR4D& aColor, int aOverrideWidth = 0, void AddShape( const BOX2I& aBox, const KIGFX::COLOR4D& aColor, int aOverrideWidth = 0,
const wxString& aName = wxT( "" ), const wxString& aName = wxT( "" ),
const SRC_LOCATION_INFO& aSrcLoc = SRC_LOCATION_INFO() ) override const SRC_LOCATION_INFO& aSrcLoc = SRC_LOCATION_INFO() ) override
@ -998,7 +997,6 @@ public:
AddShape( &l, aColor, aOverrideWidth, aName, aSrcLoc ); AddShape( &l, aColor, aOverrideWidth, aName, aSrcLoc );
} }
void AddShape( const SHAPE* aShape, const KIGFX::COLOR4D& aColor, int aOverrideWidth = 0, void AddShape( const SHAPE* aShape, const KIGFX::COLOR4D& aColor, int aOverrideWidth = 0,
const wxString& aName = wxT( "" ), const wxString& aName = wxT( "" ),
const SRC_LOCATION_INFO& aSrcLoc = SRC_LOCATION_INFO() ) override const SRC_LOCATION_INFO& aSrcLoc = SRC_LOCATION_INFO() ) override
@ -1016,7 +1014,6 @@ public:
m_view->Update( m_items ); m_view->Update( m_items );
} }
void Clear() override void Clear() override
{ {
if( m_view && m_items ) if( m_view && m_items )
@ -1864,7 +1861,6 @@ void PNS_KICAD_IFACE::UpdateItem( PNS::ITEM* aItem )
void PNS_KICAD_IFACE_BASE::AddItem( PNS::ITEM* aItem ) void PNS_KICAD_IFACE_BASE::AddItem( PNS::ITEM* aItem )
{ {
} }