haskal
/
kicad
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix SCH_LINE::GetBoundingBox() 

Instead of returning a fixed line width that was rather large, we use
the actual line width
This commit is contained in:
Seth Hillbrand 2020-01-07 06:49:50 -08:00
parent 4378290094
commit 2321da702f
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
eeschema/sch_line.cpp
View File

@ -201,16 +201,16 @@ void SCH_LINE::ViewGetLayers( int aLayers[], int& aCount ) const
const EDA_RECT SCH_LINE::GetBoundingBox() const const EDA_RECT SCH_LINE::GetBoundingBox() const
{ {
int width = 25; int width = m_size / 2;
int extra = m_size & 0x1;
int xmin = std::min( m_start.x, m_end.x ) - width; int xmin = std::min( m_start.x, m_end.x ) - width;
int ymin = std::min( m_start.y, m_end.y ) - width; int ymin = std::min( m_start.y, m_end.y ) - width;
int xmax = std::max( m_start.x, m_end.x ) + width; int xmax = std::max( m_start.x, m_end.x ) + width + extra;
int ymax = std::max( m_start.y, m_end.y ) + width; int ymax = std::max( m_start.y, m_end.y ) + width + extra;
// return a rectangle which is [pos,dim) in nature. therefore the +1 EDA_RECT ret( wxPoint( xmin, ymin ), wxSize( xmax - xmin, ymax - ymin ) );
EDA_RECT ret( wxPoint( xmin, ymin ), wxSize( xmax - xmin + 1, ymax - ymin + 1 ) );
return ret; return ret;
} }
@ -815,4 +815,4 @@ void SCH_LINE::GetMsgPanelInfo( EDA_UNITS aUnits, MSG_PANEL_ITEMS& aList )
bool SCH_LINE::IsGraphicLine() const bool SCH_LINE::IsGraphicLine() const
{ {
return ( GetLayer() == LAYER_NOTES ); return ( GetLayer() == LAYER_NOTES );
} }