Avoid printing superfluous bus messages
If we don't have a bus selected, we don't need the bus-relevant data and the net might be separate Fixes https://gitlab.com/kicad/code/kicad/-/issues/18350
This commit is contained in:
+13
-10
@@ -450,19 +450,22 @@ void SCH_CONNECTION::AppendInfoToMsgPanel( std::vector<MSG_PANEL_ITEM>& aList )
|
||||
|
||||
aList.emplace_back( _( "Connection Name" ), UnescapeString( Name() ) );
|
||||
|
||||
if( std::shared_ptr<BUS_ALIAS> alias = m_graph->GetBusAlias( m_name ) )
|
||||
if( IsBus() )
|
||||
{
|
||||
msg.Printf( _( "Bus Alias %s Members" ), m_name );
|
||||
aList.emplace_back( msg, boost::algorithm::join( alias->Members(), " " ) );
|
||||
}
|
||||
else if( NET_SETTINGS::ParseBusGroup( m_name, &group_name, &group_members ) )
|
||||
{
|
||||
for( const wxString& group_member : group_members )
|
||||
if( std::shared_ptr<BUS_ALIAS> alias = m_graph->GetBusAlias( m_name ) )
|
||||
{
|
||||
if( std::shared_ptr<BUS_ALIAS> group_alias = m_graph->GetBusAlias( group_member ) )
|
||||
msg.Printf( _( "Bus Alias %s Members" ), m_name );
|
||||
aList.emplace_back( msg, boost::algorithm::join( alias->Members(), " " ) );
|
||||
}
|
||||
else if( NET_SETTINGS::ParseBusGroup( m_name, &group_name, &group_members ) )
|
||||
{
|
||||
for( const wxString& group_member : group_members )
|
||||
{
|
||||
msg.Printf( _( "Bus Alias %s Members" ), group_alias->GetName() );
|
||||
aList.emplace_back( msg, boost::algorithm::join( group_alias->Members(), " " ) );
|
||||
if( std::shared_ptr<BUS_ALIAS> group_alias = m_graph->GetBusAlias( group_member ) )
|
||||
{
|
||||
msg.Printf( _( "Bus Alias %s Members" ), group_alias->GetName() );
|
||||
aList.emplace_back( msg, boost::algorithm::join( group_alias->Members(), " " ) );
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user