From 0d9373ca6c9ca9ec546eeeb8d7f92f52d1acb3a8 Mon Sep 17 00:00:00 2001
From: Maciej Suminski <maciej.suminski@cern.ch>
Date: Wed, 14 May 2014 18:45:59 +0200
Subject: [PATCH] Undo/report support for track dragging (PNS).

---
 pcbnew/router/pns_router.cpp  |  4 ----
 pcbnew/router/router_tool.cpp | 24 ++++++++++++++++++++++--
 2 files changed, 22 insertions(+), 6 deletions(-)

diff --git a/pcbnew/router/pns_router.cpp b/pcbnew/router/pns_router.cpp
index a08b0f0fc7..72e001801c 100644
--- a/pcbnew/router/pns_router.cpp
+++ b/pcbnew/router/pns_router.cpp
@@ -373,13 +373,9 @@ void PNS_ROUTER::ClearWorld()
     if( m_placer )
         delete m_placer;
 
-    if( m_previewItems )
-        delete m_previewItems;
-
     m_clearanceFunc = NULL;
     m_world = NULL;
     m_placer = NULL;
-    m_previewItems = NULL;
 }
 
 
diff --git a/pcbnew/router/router_tool.cpp b/pcbnew/router/router_tool.cpp
index db2c996a34..45e2d9d577 100644
--- a/pcbnew/router/router_tool.cpp
+++ b/pcbnew/router/router_tool.cpp
@@ -621,8 +621,6 @@ void ROUTER_TOOL::performRouting()
         m_needsSync = true;
     }
 
-    m_settings = m_router->Settings();
-
     ctls->SetAutoPan( false );
     ctls->ForceCursorPosition( false );
     highlightNet( false );
@@ -683,6 +681,8 @@ int ROUTER_TOOL::Main( TOOL_EVENT& aEvent )
     ctls->ShowCursor( false );
     getEditFrame<PCB_EDIT_FRAME>()->SetToolID( ID_NO_TOOL_SELECTED, wxCURSOR_DEFAULT, wxEmptyString );
 
+    // Store routing settings till the next invocation
+    m_settings = m_router->Settings();
     delete ctxMenu;
 
     return 0;
@@ -691,6 +691,8 @@ int ROUTER_TOOL::Main( TOOL_EVENT& aEvent )
 
 void ROUTER_TOOL::performDragging()
 {
+    PCB_EDIT_FRAME* frame = getEditFrame<PCB_EDIT_FRAME>();
+    bool saveUndoBuffer = true;
     VIEW_CONTROLS* ctls = getViewControls();
 
     bool dragStarted = m_router->StartDragging( m_startSnapPoint, m_startItem );
@@ -708,6 +710,11 @@ void ROUTER_TOOL::performDragging()
     {
         if( evt->IsCancel() )
             break;
+        else if( evt->Action() == TA_UNDO_REDO )
+        {
+            saveUndoBuffer = false;
+            break;
+        }
         else if( evt->IsMotion() )
         {
             updateEndItem( *evt );
@@ -726,6 +733,19 @@ void ROUTER_TOOL::performDragging()
     if( m_router->RoutingInProgress() )
         m_router->StopRouting();
 
+    if( saveUndoBuffer )
+    {
+        // Save the recent changes in the undo buffer
+        frame->SaveCopyInUndoList( m_router->GetUndoBuffer(), UR_UNSPECIFIED );
+        m_router->ClearUndoBuffer();
+        frame->OnModify();
+    }
+    else
+    {
+        // It was interrupted by TA_UNDO_REDO event, so we have to sync the world now
+        m_needsSync = true;
+    }
+
     ctls->SetAutoPan( false );
     ctls->ForceCursorPosition( false );
     highlightNet( false );