From 76853dfeb2edd1f24e884fe89eaac39686460ef2 Mon Sep 17 00:00:00 2001 From: Seth Hillbrand Date: Fri, 2 Aug 2019 13:12:53 -0700 Subject: [PATCH] pcbnew: Fix copy-paste error from 161dc9ce7 create_vrml_shell() needed the idxSide check but create_vrml_plane only had an extra variable that wasn't used and so shouldn't be checked. (cherry picked from commit c9956def36fa0350a84e5154e89f415552846981) --- pcbnew/exporters/export_vrml.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/pcbnew/exporters/export_vrml.cpp b/pcbnew/exporters/export_vrml.cpp index 8ec682f094..7f0b8e5666 100644 --- a/pcbnew/exporters/export_vrml.cpp +++ b/pcbnew/exporters/export_vrml.cpp @@ -1688,10 +1688,8 @@ static void create_vrml_plane( IFSG_TRANSFORM& PcbOutput, VRML_COLOR_INDEX color { std::vector< double > vertices; std::vector< int > idxPlane; - std::vector< int > idxSide; - if( !( *layer ).Get2DTriangles( vertices, idxPlane, top_z, aTopPlane ) || idxPlane.empty() - || idxSide.empty() ) + if( !( *layer ).Get2DTriangles( vertices, idxPlane, top_z, aTopPlane ) ) { #ifdef DEBUG do { @@ -1705,7 +1703,7 @@ static void create_vrml_plane( IFSG_TRANSFORM& PcbOutput, VRML_COLOR_INDEX color return; } - if( ( idxPlane.size() % 3 ) || ( idxSide.size() % 3 ) ) + if( ( idxPlane.size() % 3 ) ) { #ifdef DEBUG do {