Don't use SPICE_MODEL_SUBCKT::CreatePins to crate a default pinMap.

(It creates a fully-unconnected map on the premise that it's easier
to add a pinMap than to edit one.  This is probably no longer true
now that you don't have to un-assign a pin before re-assigning it.)
This commit is contained in:
Jeff Young 2022-12-19 11:03:00 +00:00
parent abff7975f1
commit df8eb59854
1 changed files with 7 additions and 4 deletions

View File

@ -1437,7 +1437,7 @@ void SIM_MODEL::MigrateSimModel( T_symbol& aSymbol, const PROJECT* aProject )
if( pinMap.IsEmpty() )
{
// Generate a default pin map
model.model.CreatePins( sourcePins );
model.model.SIM_MODEL::CreatePins( sourcePins );
pinMap = wxString( model.model.Serde().GeneratePins() );
}
}
@ -1592,9 +1592,12 @@ void SIM_MODEL::MigrateSimModel( T_symbol& aSymbol, const PROJECT* aProject )
}
}
T_field pinsField( &aSymbol, -1, SIM_MODEL::PINS_FIELD );
pinsField.SetText( pinMap );
aSymbol.AddField( pinsField );
if( !pinMap.IsEmpty() )
{
T_field pinsField( &aSymbol, -1, SIM_MODEL::PINS_FIELD );
pinsField.SetText( pinMap );
aSymbol.AddField( pinsField );
}
}