kicad-developers team mailing list archive

[Chris Pavlina <pavlina.chris@xxxxxxxxx>]

Oh, wow. God, I'm a bloody idiot.

Third time's the charm?


On Mon, Jun 22, 2015 at 02:13:40PM -0400, Chris Pavlina wrote:
> Found the problem in the rescue update - left a couple unused variable 
> declarations behind, which were no longer valid. Not sure why clang was 
> okay with them, it shouldn't have been.
> 
> --
> Chris


commit 1ffd93a238adcec2ae485ab1eaf4fe0ffddf8d17
Author: Chris Pavlina <cpavlin1@xxxxxxxxxxxxxx>
Date:   Mon Jun 22 14:12:18 2015 -0400

    Remove unused variables which caused build error

diff --git a/eeschema/project_rescue.cpp b/eeschema/project_rescue.cpp
index 7150928..fe5dbfd 100644
--- a/eeschema/project_rescue.cpp
+++ b/eeschema/project_rescue.cpp
@@ -487,10 +487,6 @@ void RESCUER::UndoRescues()
 
 bool SCH_EDIT_FRAME::RescueProject( bool aRunningOnDemand )
 {
-    // Data that will be used throughout the operation
-    std::vector<RESCUE_CANDIDATE>   candidates;
-    wxString                part_name_suffix;
-
     RESCUER rescuer( *this, Prj() );
 
     rescuer.FindCandidates();