zones work

change_log.txt

@@ -16,9 +16,14 @@ email address.
 + all
   * Added Doxygen configuration file, whose standard name is Doxyfile.  Output
     is set to go to ./doxygen directory just off the project tree.
-  * added a note to todo.txt which asks folks to start using "Doxygen compatible"
+  * Added a note to todo.txt which asks folks to start using "Doxygen compatible"
     comments in member functions and classes.  Run Doxygen on the project, then
     look at the documentation for class INSPECTOR as an example.
++ pcbnew
+    zones.cpp, Trace_Pcb(), & EDGE_ZONE class: 
+    Reversed the usage of EDGE_ZONE::Pnext and Pback in the list management to be 
+    consistent with other classes and with INSPECTOR::Inspect() and 
+    BOARD::~BOARD().
 
 
 2007-Oct-31 UPDATE  Jean-Pierre Charras <jean-pierre.charras@inpg.fr>

gerbview/trpiste.cpp

@@ -247,14 +247,16 @@ void Trace_DrawSegmentPcb( WinEDA_DrawPanel* panel, wxDC* DC,
     /* coord de depart */
     ux0 = PtDrawSegment->m_Start.x;
     uy0 = PtDrawSegment->m_Start.y;
+    
     /* coord d'arrivee */
     dx = PtDrawSegment->m_End.x;
     dy = PtDrawSegment->m_End.y;
 
-
     mode = DisplayOpt.DisplayPcbTrackFill ? FILLED : SKETCH;
+    
     if( PtDrawSegment->m_Flags & FORCE_SKETCH )
         mode = SKETCH;
+    
     if( l_piste < (L_MIN_DESSIN * zoom) )
         mode = FILAIRE;
 

include/pcbstruct.h

@@ -433,6 +433,8 @@ public:
 
     EDGE_ZONE* Next() { return (EDGE_ZONE*) Pnext; }
 
+    EDGE_ZONE* Back() { return (EDGE_ZONE*) Pback; }
+    
     
     /**
      * Function Save

pcbnew/collectors.h

@@ -37,9 +37,9 @@
 
 /**
  * Class COLLECTORS_GUIDE
- * is an abstract base class whose derivatives may be passed to a GENERALCOLLECTOR, 
- * telling GENERALCOLLECTOR what should be collected (aside from HitTest()ing 
- * and KICAD_T scanTypes[], information which are provided to the GENERALCOLLECTOR 
+ * is an abstract base class whose derivatives may be passed to a GENERAL_COLLECTOR, 
+ * telling GENERAL_COLLECTOR what should be collected (aside from HitTest()ing 
+ * and KICAD_T scanTypes[], information which are provided to the GENERAL_COLLECTOR 
  * through attributes or arguments separately). 
  * <p>
  * A justification for this class is to keep the structural storage details of
@@ -113,7 +113,7 @@ public:
 
     /**
      * Function IgnoreMTextsMarkedNoShow
-     * @return bool -true if MTexts marked as "no show" should be ignored.
+     * @return bool - true if MTexts marked as "no show" should be ignored.
      */
     virtual     bool IgnoreMTextsMarkedNoShow() const = 0;
     
@@ -302,7 +302,7 @@ public:
     
     /**
      * Function Collect
-     * scans a BOARD using this class's Inspector method, which does the collection.
+     * scans a BOARD_ITEM using this class's Inspector method, which does the collection.
      * @param aItem A BOARD_ITEM to scan, may be a BOARD or MODULE, or whatever.
      * @param aScanList A list of KICAD_Ts with a terminating EOT, that specs 
      *  what is to be collected and the priority order of the resultant 
@@ -460,7 +460,7 @@ public:
     
     /**
      * Function IgnoreMTextsMarkedNoShow
-     * @return bool -true if MTexts marked as "no show" should be ignored.
+     * @return bool - true if MTexts marked as "no show" should be ignored.
      */
     bool IgnoreMTextsMarkedNoShow() const { return m_IgnoreMTextsMarkedNoShow; }
     void SetIgnoreMTextsMarkedNoShow( bool ignore ) { m_IgnoreMTextsMarkedNoShow = ignore; }

pcbnew/tracepcb.cpp

@@ -176,8 +176,8 @@ void WinEDA_PcbFrame::Trace_Pcb( wxDC* DC, int mode )
     if( g_HightLigt_Status )
         DrawHightLight( DC, g_HightLigth_NetCode );
 
-    EDGE_ZONE* segment = m_Pcb->m_CurrentLimitZone;
-    for( ; segment != NULL;   segment = (EDGE_ZONE*) segment->Pback )
+    EDGE_ZONE* segment;
+    for( segment = m_Pcb->m_CurrentLimitZone; segment; segment = segment->Next() )
     {
         if( segment->m_Flags & IS_MOVED )
             continue;