Commit 60eb4b00 authored by Wayne Stambaugh's avatar Wayne Stambaugh

Pcbnew netlist read performance improvements.

* Do not search for footprint when no footprint is defined for a component
  in the netlist.
parent 2810f5a5
...@@ -185,6 +185,18 @@ void PCB_EDIT_FRAME::loadFootprints( NETLIST& aNetlist, REPORTER* aReporter ) ...@@ -185,6 +185,18 @@ void PCB_EDIT_FRAME::loadFootprints( NETLIST& aNetlist, REPORTER* aReporter )
{ {
component = aNetlist.GetComponent( ii ); component = aNetlist.GetComponent( ii );
if( component->GetFootprintName().IsEmpty() )
{
if( aReporter )
{
msg.Printf( _( "No footprint defined for component `%s`.\n" ),
GetChars( component->GetReference() ) );
aReporter->Report( msg );
}
continue;
}
// Check if component footprint is already on BOARD and only load the footprint from // Check if component footprint is already on BOARD and only load the footprint from
// the library if it's needed. // the library if it's needed.
if( aNetlist.IsFindByTimeStamp() ) if( aNetlist.IsFindByTimeStamp() )
...@@ -278,6 +290,18 @@ void PCB_EDIT_FRAME::loadFootprints( NETLIST& aNetlist, REPORTER* aReporter ) ...@@ -278,6 +290,18 @@ void PCB_EDIT_FRAME::loadFootprints( NETLIST& aNetlist, REPORTER* aReporter )
{ {
component = aNetlist.GetComponent( ii ); component = aNetlist.GetComponent( ii );
if( component->GetFootprintName().IsEmpty() )
{
if( aReporter )
{
msg.Printf( _( "No footprint defined for component `%s`.\n" ),
GetChars( component->GetReference() ) );
aReporter->Report( msg );
}
continue;
}
// Check if component footprint is already on BOARD and only load the footprint from // Check if component footprint is already on BOARD and only load the footprint from
// the library if it's needed. // the library if it's needed.
if( aNetlist.IsFindByTimeStamp() ) if( aNetlist.IsFindByTimeStamp() )
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment