kicad/pcbnew/ratsnest/ratsnest_view_item.cpp

254 lines
8.7 KiB
C++
Raw Normal View History

2013-11-25 15:50:03 +00:00
/*
* This program source code file is part of KICAD, a free EDA CAD application.
*
* Copyright (C) 2013 CERN
2021-09-07 20:34:10 +00:00
* Copyright (C) 2018-2021 KiCad Developers, see AUTHORS.txt for contributors.
2018-11-28 14:58:41 +00:00
*
2013-11-25 15:50:03 +00:00
* @author Maciej Suminski <maciej.suminski@cern.ch>
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, you may find one here:
* http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
* or you may search the http://www.gnu.org website for the version 2 license,
* or you may write to the Free Software Foundation, Inc.,
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA
*/
/**
* @brief Class that draws missing connections on a PCB.
*/
2021-02-22 23:47:17 +00:00
#include <ratsnest/ratsnest_view_item.h>
2021-09-07 20:34:10 +00:00
#include <kiface_base.h>
#include <pcbnew_settings.h>
#include <pcb_painter.h>
#include <ratsnest/ratsnest_data.h>
#include <layer_ids.h>
#include <pcb_base_frame.h>
2021-09-07 20:34:10 +00:00
#include <view/view.h>
2013-11-25 15:50:03 +00:00
2017-03-22 13:43:10 +00:00
#include <memory>
#include <utility>
2017-03-22 13:43:10 +00:00
2013-11-25 15:50:03 +00:00
2021-02-22 23:47:17 +00:00
RATSNEST_VIEW_ITEM::RATSNEST_VIEW_ITEM( std::shared_ptr<CONNECTIVITY_DATA> aData ) :
EDA_ITEM( NOT_USED ), m_data( std::move(aData) )
2013-11-25 15:50:03 +00:00
{
}
2021-02-22 23:47:17 +00:00
const BOX2I RATSNEST_VIEW_ITEM::ViewBBox() const
2013-11-25 15:50:03 +00:00
{
// Make it always visible
BOX2I bbox;
bbox.SetMaximum();
return bbox;
}
2018-11-28 14:58:41 +00:00
2021-02-22 23:47:17 +00:00
void RATSNEST_VIEW_ITEM::ViewDraw( int aLayer, KIGFX::VIEW* aView ) const
2013-11-25 15:50:03 +00:00
{
Fix issues with zone filling connectivity locking Two issues found with the locking system used to prevent access to stale connectivity data during the zone fill process: 1) a std::mutex has undefined behavior if you try to use it to guard against access from the same thread. Because of the use of wx event loops (and coroutines) it is entirely possible, and in some situations inevitable, that the same thread will try to redraw the ratsnest in the middle of zone refilling. 2) The mutex was only guarding the ZONE_FILLER::Fill method, but the callers of that method also do connectivity updates as part of the COMMIT::Push. Redrawing the ratsnest after the Fill but before the Push will result in stale connectivity pointers to zone filled areas. Fixed (1) by switching to a trivial spinlock implementation. Spinlocks would generally not be desirable if the contention for the connectivity data crossed thread boundaries, but at the moment I believe it's guaranteed that the reads and writes to connectivity that are guarded by this lock happen from the main UI thread. The writes are also quite rare compared to reads, and reads are generally fast, so I'm not really worried about the UI thread spinning for any real amount of time. Fixed (2) by moving the locking location up to the call sites of ZONE_FILLER::Fill. This issue was quite difficult to reproduce, but I found a fairly reliable way: It only happens (for me) on Windows, MSYS2 build, with wxWidgets 3.0 It also only happens if I restrict PcbNew to use 2 CPU cores. With those conditions, I can reproduce the issue described in #6471 by repeatedly editing a zone properties and changing its net. The crash is especially easy to trigger if you press some keys (such as 'e' for edit) while the progress dialog is displayed. It's easiest to do this in a debug build as the slower KiCad is running, the bigger the window is to trigger this bug. Fixes https://gitlab.com/kicad/code/kicad/-/issues/6471 Fixes https://gitlab.com/kicad/code/kicad/-/issues/7048
2021-01-18 17:24:07 +00:00
std::unique_lock<KISPINLOCK> lock( m_data->GetLock(), std::try_to_lock );
if( !lock )
return;
constexpr int CROSS_SIZE = 200000;
2017-03-22 13:43:10 +00:00
auto gal = aView->GetGAL();
2017-03-22 13:43:10 +00:00
gal->SetIsStroke( true );
gal->SetIsFill( false );
gal->SetLineWidth( 1.0 );
2021-09-07 20:34:10 +00:00
auto cfg = static_cast<PCBNEW_SETTINGS*>( Kiface().KifaceSettings() );
2021-02-22 23:47:17 +00:00
auto rs = static_cast<KIGFX::PCB_RENDER_SETTINGS*>( aView->GetPainter()->GetSettings() );
COLOR4D defaultColor = rs->GetColor( nullptr, LAYER_RATSNEST );
COLOR4D color = defaultColor;
const bool colorByNet = rs->GetNetColorMode() != NET_COLOR_MODE::OFF;
2013-11-25 15:50:03 +00:00
std::set<int> highlightedNets = rs->GetHighlightNetCodes();
const std::set<int>& hiddenNets = rs->GetHiddenNets();
2021-09-07 20:34:10 +00:00
std::map<int, KIGFX::COLOR4D>& netColors = rs->GetNetColorMap();
std::map<wxString, KIGFX::COLOR4D>& ncColors = rs->GetNetclassColorMap();
const std::map<int, wxString>& ncMap = m_data->GetNetclassMap();
2021-09-07 20:34:10 +00:00
const bool onlyVisibleLayers = cfg->m_Display.m_RatsnestMode == RATSNEST_MODE::VISIBLE;
LSET visibleLayers;
// If we are in "other layers off" mode, the active layer is the only visible layer
if( rs->m_ContrastModeDisplay == HIGH_CONTRAST_MODE::HIDDEN )
{
visibleLayers.set( rs->GetPrimaryHighContrastLayer() );
}
else
{
for( PCB_LAYER_ID layer : LSET::AllCuMask().Seq() )
2021-09-07 20:34:10 +00:00
{
if( aView->IsLayerVisible( layer ) )
visibleLayers.set( layer );
2021-09-07 20:34:10 +00:00
}
}
2021-09-07 20:34:10 +00:00
const bool curved_ratsnest = cfg->m_Display.m_DisplayRatsnestLinesCurved;
2017-03-22 13:43:10 +00:00
// Draw the "dynamic" ratsnest (i.e. for objects that may be currently being moved)
for( const RN_DYNAMIC_LINE& l : m_data->GetDynamicRatsnest() )
2017-03-22 13:43:10 +00:00
{
if( hiddenNets.count( l.netCode ) )
continue;
if( colorByNet && netColors.count( l.netCode ) )
color = netColors.at( l.netCode );
2021-09-07 20:34:10 +00:00
else if( colorByNet && ncMap.count( l.netCode ) && ncColors.count( ncMap.at( l.netCode ) ) )
color = ncColors.at( ncMap.at( l.netCode ) );
else
color = defaultColor;
if( color == COLOR4D::UNSPECIFIED )
color = defaultColor;
gal->SetStrokeColor( color.Brightened( 0.5 ) );
2021-11-30 14:19:39 +00:00
if( l.a == l.b )
2013-11-25 15:50:03 +00:00
{
2018-11-28 14:58:41 +00:00
gal->DrawLine( VECTOR2I( l.a.x - CROSS_SIZE, l.a.y - CROSS_SIZE ),
VECTOR2I( l.b.x + CROSS_SIZE, l.b.y + CROSS_SIZE ) );
gal->DrawLine( VECTOR2I( l.a.x - CROSS_SIZE, l.a.y + CROSS_SIZE ),
VECTOR2I( l.b.x + CROSS_SIZE, l.b.y - CROSS_SIZE ) );
}
else
{
if( curved_ratsnest )
{
2021-09-07 20:34:10 +00:00
int dx = l.b.x - l.a.x;
int dy = l.b.y - l.a.y;
const VECTOR2I center = VECTOR2I( l.a.x + 0.5 * dx - 0.1 * dy,
l.a.y + 0.5 * dy + 0.1 * dx );
gal->DrawCurve( l.a, center, center, l.b );
}
else
{
gal->DrawLine( l.a, l.b );
}
2017-03-22 13:43:10 +00:00
}
}
2013-11-25 15:50:03 +00:00
for( int i = 1 /* skip "No Net" at [0] */; i < m_data->GetNetCount(); ++i )
2017-03-22 13:43:10 +00:00
{
if( hiddenNets.count( i ) )
continue;
2017-03-22 13:43:10 +00:00
RN_NET* net = m_data->GetRatsnestForNet( i );
2013-11-25 15:50:03 +00:00
if( !net )
continue;
if( colorByNet && netColors.count( i ) )
color = netColors.at( i );
2021-09-07 20:34:10 +00:00
else if( colorByNet && ncMap.count( i ) && ncColors.count( ncMap.at( i ) ) )
color = ncColors.at( ncMap.at( i ) );
else
color = defaultColor;
if( color == COLOR4D::UNSPECIFIED )
color = defaultColor;
2013-11-25 15:50:03 +00:00
// Draw the "static" ratsnest
if( highlightedNets.count( i ) )
2021-11-30 14:19:39 +00:00
gal->SetStrokeColor( color.Brightened( 0.8 ) );
else
gal->SetStrokeColor( color ); // using the default ratsnest color for not highlighted
for( const CN_EDGE& edge : net->GetEdges() )
2013-11-25 15:50:03 +00:00
{
if( !edge.IsVisible() )
continue;
2021-11-30 14:19:39 +00:00
const CN_ANCHOR_PTR& sourceNode = edge.GetSourceNode();
const CN_ANCHOR_PTR& targetNode = edge.GetTargetNode();
const VECTOR2I source( sourceNode->Pos() );
const VECTOR2I target( targetNode->Pos() );
2013-11-25 15:50:03 +00:00
if( !sourceNode->Valid() || !targetNode->Valid() )
continue;
bool enable = !sourceNode->GetNoLine() && !targetNode->GetNoLine();
bool show;
2021-09-07 20:34:10 +00:00
// If the global ratsnest is currently enabled, the local ratsnest should be easy to
// turn off, so either element can disable it.
// If the global ratsnest is disabled, the local ratsnest should be easy to turn on
// so either element can enable it.
2021-09-07 20:34:10 +00:00
if( cfg->m_Display.m_ShowGlobalRatsnest )
{
show = sourceNode->Parent()->GetLocalRatsnestVisible() &&
targetNode->Parent()->GetLocalRatsnestVisible();
2021-09-07 20:34:10 +00:00
}
else
2021-09-07 20:34:10 +00:00
{
show = sourceNode->Parent()->GetLocalRatsnestVisible() ||
targetNode->Parent()->GetLocalRatsnestVisible();
2021-09-07 20:34:10 +00:00
}
if( onlyVisibleLayers && show )
{
LSET sourceLayers = sourceNode->Parent()->GetLayerSet();
LSET targetLayers = targetNode->Parent()->GetLayerSet();
if( !( sourceLayers & visibleLayers ).any() ||
!( targetLayers & visibleLayers ).any() )
2021-09-07 20:34:10 +00:00
{
show = false;
2021-09-07 20:34:10 +00:00
}
}
if ( enable && show )
2017-03-22 13:43:10 +00:00
{
if ( source == target )
{
2018-11-28 14:58:41 +00:00
gal->DrawLine( VECTOR2I( source.x - CROSS_SIZE, source.y - CROSS_SIZE ),
VECTOR2I( source.x + CROSS_SIZE, source.y + CROSS_SIZE ) );
gal->DrawLine( VECTOR2I( source.x - CROSS_SIZE, source.y + CROSS_SIZE ),
VECTOR2I( source.x + CROSS_SIZE, source.y - CROSS_SIZE ) );
2017-03-22 13:43:10 +00:00
}
else
{
if( curved_ratsnest )
{
2021-09-07 20:34:10 +00:00
int dx = target.x - source.x;
int dy = target.y - source.y;
const VECTOR2I center = VECTOR2I( source.x + 0.5 * dx - 0.1 * dy,
source.y + 0.5 * dy + 0.1 * dx );
gal->DrawCurve( source, center, center, target );
}
else
{
gal->DrawLine( source, target );
}
2017-03-22 13:43:10 +00:00
}
}
2013-11-25 15:50:03 +00:00
}
}
}
2021-02-22 23:47:17 +00:00
void RATSNEST_VIEW_ITEM::ViewGetLayers( int aLayers[], int& aCount ) const
2013-11-25 15:50:03 +00:00
{
aCount = 1;
aLayers[0] = LAYER_RATSNEST;
2013-11-25 15:50:03 +00:00
}