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

Fix potential divide-by-zero (from Coverity scan).

This commit is contained in:
Jeff Young 2020-01-11 16:41:24 +00:00
parent 1591d13839
commit 715448611d
1 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
pcbnew/eagle_plugin.cpp
View File

@ -1820,15 +1820,19 @@ void EAGLE_PLUGIN::packagePolygon( MODULE* aModule, wxXmlNode* aTree ) const
double radius = sqrt( pow( center.x - kicad_x( v1.x ), 2 ) double radius = sqrt( pow( center.x - kicad_x( v1.x ), 2 )
+ pow( center.y - kicad_y( v1.y ), 2 ) ); + pow( center.y - kicad_y( v1.y ), 2 ) );
// If we are curving, we need at least 2 segments otherwise // Don't allow a zero-radius curve
// delta_angle == angle if( KiROUND( radius ) == 0 )
double delta_angle = angle / std::max( radius = 1.0;
2, GetArcToSegmentCount( KiROUND( radius ),
ARC_HIGH_DEF, *v1.curve ) - 1 );
for( double a = end_angle + angle; int segCount = GetArcToSegmentCount( KiROUND( radius ), ARC_HIGH_DEF, *v1.curve ) - 1;
fabs( a - end_angle ) > fabs( delta_angle );
a -= delta_angle ) // If we are curving, we need at least 2 segments otherwise delta == angle
if( segCount < 2 )
segCount = 2;
double delta = angle / segCount;
for( double a = end_angle + angle; fabs( a - end_angle ) > fabs( delta ); a -= delta )
{ {
pts.push_back( pts.push_back(
wxPoint( KiROUND( radius * cos( a ) ), wxPoint( KiROUND( radius * cos( a ) ),