electron-rare/kicad-source-mirror
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't overwrite explicit sizes unless needed

Browse Source 
We already did this for single-ended, this extends to dp routes

Fixes https://gitlab.com/kicad/code/kicad/-/issues/20889

(cherry picked from commit c06ddf966d)
This commit is contained in:
Seth Hillbrand
2025-05-09 13:32:55 -07:00
parent 4b009b0698
commit e387635edf
1 changed files with 14 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pcbnew/router/router_tool.cpp
+14 -2
View File
@@ -798,7 +798,13 @@ void ROUTER_TOOL::updateSizesAfterRouterEvent( int aTargetLayer, const VECTOR2I&
if( !constraint.IsNull() )
{
sizes.SetDiffPairWidth( std::max( bds.m_TrackMinWidth, constraint.m_Value.Opt() ) );
if( bds.UseNetClassDiffPair()
|| ( sizes.DiffPairWidth() < bds.m_TrackMinWidth )
|| ( sizes.DiffPairWidth() < constraint.m_Value.Min() )
|| ( sizes.DiffPairWidth() > constraint.m_Value.Max() ) )
{
sizes.SetDiffPairWidth( std::max( bds.m_TrackMinWidth, constraint.m_Value.Opt() ) );
}
if( sizes.DiffPairWidth() == constraint.m_Value.Opt() )
sizes.SetDiffPairWidthSource( constraint.GetName() );
@@ -811,7 +817,13 @@ void ROUTER_TOOL::updateSizesAfterRouterEvent( int aTargetLayer, const VECTOR2I&
if( !constraint.IsNull() )
{
sizes.SetDiffPairGap( std::max( bds.m_MinClearance, constraint.m_Value.Opt() ) );
if( bds.UseNetClassDiffPair()
|| ( sizes.DiffPairGap() < bds.m_MinClearance )
|| ( sizes.DiffPairGap() < constraint.m_Value.Min() )
|| ( sizes.DiffPairGap() > constraint.m_Value.Max() ) )
{
sizes.SetDiffPairGap( std::max( bds.m_MinClearance, constraint.m_Value.Opt() ) );
}
if( sizes.DiffPairGap() == constraint.m_Value.Opt() )
sizes.SetDiffPairGapSource( constraint.GetName() );