Fix over-zealous copy/paste and add a few more comments.

pcbnew/generators/pcb_tuning_pattern.cpp

@@ -406,7 +406,7 @@ public:
     void ShowPropertiesDialog( PCB_BASE_EDIT_FRAME* aEditFrame ) override;
 
     std::vector<EDA_ITEM*> GetPreviewItems( GENERATOR_TOOL* aTool, PCB_BASE_EDIT_FRAME* aFrame,
-                                            bool aStatusOnly = false ) override;
+                                            bool aStatusItemsOnly = false ) override;
 
     void GetMsgPanelInfo( EDA_DRAW_FRAME* aFrame, std::vector<MSG_PANEL_ITEM>& aList ) override;
 
@@ -1848,14 +1848,14 @@ void PCB_TUNING_PATTERN::ShowPropertiesDialog( PCB_BASE_EDIT_FRAME* aEditFrame )
 
 std::vector<EDA_ITEM*> PCB_TUNING_PATTERN::GetPreviewItems( GENERATOR_TOOL* aTool,
                                                             PCB_BASE_EDIT_FRAME* aFrame,
-                                                            bool aStatusOnly )
+                                                            bool aStatusItemsOnly )
 {
     std::vector<EDA_ITEM*> previewItems;
     KIGFX::VIEW*           view = aFrame->GetCanvas()->GetView();
 
     if( auto* placer = dynamic_cast<PNS::MEANDER_PLACER_BASE*>( aTool->Router()->Placer() ) )
     {
-        if( !aStatusOnly )
+        if( !aStatusItemsOnly )
         {
             PNS::ITEM_SET items = placer->TunedPath();
 

pcbnew/pcb_generator.cpp

@@ -73,7 +73,7 @@ bool PCB_GENERATOR::Update( GENERATOR_TOOL* aTool, BOARD* aBoard, PCB_BASE_EDIT_
 
 std::vector<EDA_ITEM*> PCB_GENERATOR::GetPreviewItems( GENERATOR_TOOL* aTool,
                                                        PCB_BASE_EDIT_FRAME* aFrame,
-                                                       bool aStatusOnly )
+                                                       bool aStatusItemsOnly )
 {
     return std::vector<EDA_ITEM*>();
 }
@@ -86,7 +86,7 @@ bool PCB_GENERATOR::MakeEditPoints( std::shared_ptr<EDIT_POINTS> aEditPoints ) c
 
 
 bool PCB_GENERATOR::UpdateFromEditPoints( std::shared_ptr<EDIT_POINTS> aEditPoints,
-                                          BOARD_COMMIT*                aCommit )
+                                          BOARD_COMMIT* aCommit )
 {
     return true;
 }

pcbnew/pcb_generator.h

@@ -63,14 +63,16 @@ public:
     virtual bool Update( GENERATOR_TOOL* aTool, BOARD* aBoard, PCB_BASE_EDIT_FRAME* aFrame,
                          BOARD_COMMIT* aCommit );
 
+#define STATUS_ITEMS_ONLY true
+
     virtual std::vector<EDA_ITEM*> GetPreviewItems( GENERATOR_TOOL* aTool,
                                                     PCB_BASE_EDIT_FRAME* aFrame,
-                                                    bool aStatusOnly = false );
+                                                    bool aStatusItemsOnly = false );
 
     virtual bool MakeEditPoints( std::shared_ptr<EDIT_POINTS> aEditPoints ) const;
 
     virtual bool UpdateFromEditPoints( std::shared_ptr<EDIT_POINTS> aEditPoints,
-                                       BOARD_COMMIT*                aCommit );
+                                       BOARD_COMMIT* aCommit );
 
     virtual bool UpdateEditPoints( std::shared_ptr<EDIT_POINTS> aEditPoints );
 

pcbnew/tools/pcb_point_editor.cpp

@@ -1506,12 +1506,16 @@ void PCB_POINT_EDITOR::updateItem( BOARD_COMMIT* aCommit )
         generatorItem->UpdateFromEditPoints( m_editPoints, aCommit );
         m_toolMgr->RunSynchronousAction( PCB_ACTIONS::genUpdateEdit, aCommit, generatorItem );
 
+        // Note: POINT_EDITOR::m_preview holds only the canvas-draw status "popup"; the meanders
+        // themselves (ROUTER_PREVIEW_ITEMs) are owned by the router.
+
         m_preview.FreeItems();
 
-        for( EDA_ITEM* previewItem : generatorItem->GetPreviewItems( generatorTool, frame(), true ) )
+        for( EDA_ITEM* previewItem : generatorItem->GetPreviewItems( generatorTool, frame(),
+                                                                     STATUS_ITEMS_ONLY ) )
+        {
             m_preview.Add( previewItem );
-
-        generatorTool->Router()->StopRouting();
+        }
 
         getView()->Update( &m_preview );
         break;