Removed ambiguity between deprecated and new constructor.

[wxWidgets.git] / contrib / src / ogl / basic.cpp

diff --git a/contrib/src/ogl/basic.cpp b/contrib/src/ogl/basic.cpp
index 29e50b9c9320bf882ee6c1c19f2914825459b1bb..b4b1b3e125f53ff90cf512b3e9dd153aa7e3c9ff 100644 (file)
--- a/contrib/src/ogl/basic.cpp
+++ b/contrib/src/ogl/basic.cpp
@@ -6,7 +6,7 @@
 // Created:     12/07/98
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart
 // Created:     12/07/98
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart

-// Licence:    wxWindows licence
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__


@@ -24,29 +24,19 @@
 #include <wx/wx.h>
 #endif
 
 #include <wx/wx.h>
 #endif
 

-#include <wx/wxexpr.h>
+#if wxUSE_PROLOGIO
+#include <wx/deprecated/wxexpr.h>
+#endif

 
 #ifdef new
 #undef new
 #endif
 
 
 #ifdef new
 #undef new
 #endif
 

-#if wxUSE_IOSTREAMH
-#include <iostream.h>
-#else
-#include <iostream>
-#endif
-

 #include <stdio.h>
 #include <ctype.h>
 #include <stdio.h>
 #include <ctype.h>

-#include <math.h>

 
 

-#include <wx/ogl/basic.h>
-#include <wx/ogl/basicp.h>
-#include <wx/ogl/composit.h>
-#include <wx/ogl/lines.h>
-#include <wx/ogl/canvas.h>
-#include <wx/ogl/divided.h>
-#include <wx/ogl/misc.h>
+#include "wx/ogl/ogl.h"
+

 
 // Control point types
 // Rectangle and most other shapes
 
 // Control point types
 // Rectangle and most other shapes


@@ -146,7 +136,7 @@ bool wxShapeEvtHandler::OnMovePre(wxDC& dc, double x, double y, double old_x, do
   if (m_previousHandler)
     return m_previousHandler->OnMovePre(dc, x, y, old_x, old_y, display);
   else
   if (m_previousHandler)
     return m_previousHandler->OnMovePre(dc, x, y, old_x, old_y, display);
   else

-    return TRUE;
+    return true;

 }
 
 void wxShapeEvtHandler::OnMovePost(wxDC& dc, double x, double y, double old_x, double old_y, bool display)
 }
 
 void wxShapeEvtHandler::OnMovePost(wxDC& dc, double x, double y, double old_x, double old_y, bool display)


@@ -279,25 +269,24 @@ wxShape::wxShape(wxShapeCanvas *can)
   m_eventHandler = this;
   SetShape(this);
   m_id = 0;
   m_eventHandler = this;
   SetShape(this);
   m_id = 0;

-  m_formatted = FALSE;
+  m_formatted = false;

   m_canvas = can;
   m_xpos = 0.0; m_ypos = 0.0;
   m_pen = g_oglBlackPen;
   m_brush = wxWHITE_BRUSH;
   m_font = g_oglNormalFont;
   m_canvas = can;
   m_xpos = 0.0; m_ypos = 0.0;
   m_pen = g_oglBlackPen;
   m_brush = wxWHITE_BRUSH;
   m_font = g_oglNormalFont;

-  m_textColour = wxBLACK;
-  m_textColourName = "BLACK";
-  m_visible = FALSE;
-  m_clientData = NULL;
-  m_selected = FALSE;
+  m_textColour = wxT("BLACK");
+  m_textColourName = wxT("BLACK");
+  m_visible = false;
+  m_selected = false;

   m_attachmentMode = ATTACHMENT_MODE_NONE;
   m_attachmentMode = ATTACHMENT_MODE_NONE;

-  m_spaceAttachments = TRUE;
-  m_disableLabel = FALSE;
-  m_fixedWidth = FALSE;
-  m_fixedHeight = FALSE;
-  m_drawHandles = TRUE;
+  m_spaceAttachments = true;
+  m_disableLabel = false;
+  m_fixedWidth = false;
+  m_fixedHeight = false;
+  m_drawHandles = true;

   m_sensitivity = OP_ALL;
   m_sensitivity = OP_ALL;

-  m_draggable = TRUE;
+  m_draggable = true;

   m_parent = NULL;
   m_formatMode = FORMAT_CENTRE_HORIZ | FORMAT_CENTRE_VERT;
   m_shadowMode = SHADOW_NONE;
   m_parent = NULL;
   m_formatMode = FORMAT_CENTRE_HORIZ | FORMAT_CENTRE_VERT;
   m_shadowMode = SHADOW_NONE;


@@ -306,10 +295,10 @@ wxShape::wxShape(wxShapeCanvas *can)
   m_shadowBrush = wxBLACK_BRUSH;
   m_textMarginX = 5;
   m_textMarginY = 5;
   m_shadowBrush = wxBLACK_BRUSH;
   m_textMarginX = 5;
   m_textMarginY = 5;

-  m_regionName = "0";
-  m_centreResize = TRUE;
-  m_maintainAspectRatio = FALSE;
-  m_highlighted = FALSE;
+  m_regionName = wxT("0");
+  m_centreResize = true;
+  m_maintainAspectRatio = false;
+  m_highlighted = false;

   m_rotation = 0.0;
   m_branchNeckLength = 10;
   m_branchStemLength = 10;
   m_rotation = 0.0;
   m_branchNeckLength = 10;
   m_branchStemLength = 10;


@@ -320,10 +309,10 @@ wxShape::wxShape(wxShapeCanvas *can)
   // the region eventually (the duplication is for compatibility)
   wxShapeRegion *region = new wxShapeRegion;
   m_regions.Append(region);
   // the region eventually (the duplication is for compatibility)
   wxShapeRegion *region = new wxShapeRegion;
   m_regions.Append(region);

-  region->SetName("0");
+  region->SetName(wxT("0"));

   region->SetFont(g_oglNormalFont);
   region->SetFormatMode(FORMAT_CENTRE_HORIZ | FORMAT_CENTRE_VERT);
   region->SetFont(g_oglNormalFont);
   region->SetFormatMode(FORMAT_CENTRE_HORIZ | FORMAT_CENTRE_VERT);

-  region->SetColour("BLACK");
+  region->SetColour(wxT("BLACK"));

 }
 
 wxShape::~wxShape()
 }
 
 wxShape::~wxShape()


@@ -338,11 +327,6 @@ wxShape::~wxShape()
   if (m_canvas)
     m_canvas->RemoveShape(this);
 
   if (m_canvas)
     m_canvas->RemoveShape(this);
 

-  if (m_clientData) {
-      delete m_clientData;
-      m_clientData = NULL;
-  }
-

   GetEventHandler()->OnDelete();
 }
 
   GetEventHandler()->OnDelete();
 }
 


@@ -351,12 +335,12 @@ void wxShape::SetHighlight(bool hi, bool recurse)
   m_highlighted = hi;
   if (recurse)
   {
   m_highlighted = hi;
   if (recurse)
   {

-    wxNode *node = m_children.First();
+    wxNode *node = m_children.GetFirst();

     while (node)
     {
     while (node)
     {

-      wxShape *child = (wxShape *)node->Data();
+      wxShape *child = (wxShape *)node->GetData();

       child->SetHighlight(hi, recurse);
       child->SetHighlight(hi, recurse);

-      node = node->Next();
+      node = node->GetNext();

     }
   }
 }
     }
   }
 }


@@ -364,19 +348,19 @@ void wxShape::SetHighlight(bool hi, bool recurse)
 void wxShape::SetSensitivityFilter(int sens, bool recursive)
 {
   if (sens & OP_DRAG_LEFT)
 void wxShape::SetSensitivityFilter(int sens, bool recursive)
 {
   if (sens & OP_DRAG_LEFT)

-    m_draggable = TRUE;
+    m_draggable = true;

   else
   else

-    m_draggable = FALSE;
+    m_draggable = false;

 
   m_sensitivity = sens;
   if (recursive)
   {
 
   m_sensitivity = sens;
   if (recursive)
   {

-    wxNode *node = m_children.First();
+    wxNode *node = m_children.GetFirst();

     while (node)
     {
     while (node)
     {

-      wxShape *obj = (wxShape *)node->Data();
-      obj->SetSensitivityFilter(sens, TRUE);
-      node = node->Next();
+      wxShape *obj = (wxShape *)node->GetData();
+      obj->SetSensitivityFilter(sens, true);
+      node = node->GetNext();

     }
   }
 }
     }
   }
 }


@@ -392,12 +376,12 @@ void wxShape::SetDraggable(bool drag, bool recursive)
 
   if (recursive)
   {
 
   if (recursive)
   {

-    wxNode *node = m_children.First();
+    wxNode *node = m_children.GetFirst();

     while (node)
     {
     while (node)
     {

-      wxShape *obj = (wxShape *)node->Data();
-      obj->SetDraggable(drag, TRUE);
-      node = node->Next();
+      wxShape *obj = (wxShape *)node->GetData();
+      obj->SetDraggable(drag, true);
+      node = node->GetNext();

     }
   }
 }
     }
   }
 }


@@ -405,12 +389,12 @@ void wxShape::SetDraggable(bool drag, bool recursive)
 void wxShape::SetDrawHandles(bool drawH)
 {
   m_drawHandles = drawH;
 void wxShape::SetDrawHandles(bool drawH)
 {
   m_drawHandles = drawH;

-  wxNode *node = m_children.First();
+  wxNode *node = m_children.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShape *obj = (wxShape *)node->Data();
+    wxShape *obj = (wxShape *)node->GetData();

     obj->SetDrawHandles(drawH);
     obj->SetDrawHandles(drawH);

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
   }
 }
 


@@ -435,27 +419,27 @@ void wxShape::SetShadowMode(int mode, bool redraw)
 void wxShape::SetCanvas(wxShapeCanvas *theCanvas)
 {
   m_canvas = theCanvas;
 void wxShape::SetCanvas(wxShapeCanvas *theCanvas)
 {
   m_canvas = theCanvas;

-  wxNode *node = m_children.First();
+  wxNode *node = m_children.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShape *child = (wxShape *)node->Data();
+    wxShape *child = (wxShape *)node->GetData();

     child->SetCanvas(theCanvas);
     child->SetCanvas(theCanvas);

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
 void wxShape::AddToCanvas(wxShapeCanvas *theCanvas, wxShape *addAfter)
 {
   theCanvas->AddShape(this, addAfter);
   }
 }
 
 void wxShape::AddToCanvas(wxShapeCanvas *theCanvas, wxShape *addAfter)
 {
   theCanvas->AddShape(this, addAfter);

-  wxNode *node = m_children.First();
+  wxNode *node = m_children.GetFirst();

   wxShape *lastImage = this;
   while (node)
   {
   wxShape *lastImage = this;
   while (node)
   {

-    wxShape *object = (wxShape *)node->Data();
+    wxShape *object = (wxShape *)node->GetData();

     object->AddToCanvas(theCanvas, lastImage);
     lastImage = object;
 
     object->AddToCanvas(theCanvas, lastImage);
     lastImage = object;
 

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
   }
 }
 


@@ -463,41 +447,41 @@ void wxShape::AddToCanvas(wxShapeCanvas *theCanvas, wxShape *addAfter)
 void wxShape::InsertInCanvas(wxShapeCanvas *theCanvas)
 {
   theCanvas->InsertShape(this);
 void wxShape::InsertInCanvas(wxShapeCanvas *theCanvas)
 {
   theCanvas->InsertShape(this);

-  wxNode *node = m_children.First();
+  wxNode *node = m_children.GetFirst();

   wxShape *lastImage = this;
   while (node)
   {
   wxShape *lastImage = this;
   while (node)
   {

-    wxShape *object = (wxShape *)node->Data();
+    wxShape *object = (wxShape *)node->GetData();

     object->AddToCanvas(theCanvas, lastImage);
     lastImage = object;
 
     object->AddToCanvas(theCanvas, lastImage);
     lastImage = object;
 

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
 void wxShape::RemoveFromCanvas(wxShapeCanvas *theCanvas)
 {
   if (Selected())
   }
 }
 
 void wxShape::RemoveFromCanvas(wxShapeCanvas *theCanvas)
 {
   if (Selected())

-    Select(FALSE);
+    Select(false);

   theCanvas->RemoveShape(this);
   theCanvas->RemoveShape(this);

-  wxNode *node = m_children.First();
+  wxNode *node = m_children.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShape *object = (wxShape *)node->Data();
+    wxShape *object = (wxShape *)node->GetData();

     object->RemoveFromCanvas(theCanvas);
 
     object->RemoveFromCanvas(theCanvas);
 

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
 void wxShape::ClearAttachments()
 {
   }
 }
 
 void wxShape::ClearAttachments()
 {

-  wxNode *node = m_attachmentPoints.First();
+  wxNode *node = m_attachmentPoints.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxAttachmentPoint *point = (wxAttachmentPoint *)node->Data();
+    wxAttachmentPoint *point = (wxAttachmentPoint *)node->GetData();

     delete point;
     delete point;

-    node = node->Next();
+    node = node->GetNext();

   }
   m_attachmentPoints.Clear();
 }
   }
   m_attachmentPoints.Clear();
 }


@@ -506,24 +490,24 @@ void wxShape::ClearText(int regionId)
 {
   if (regionId == 0)
   {
 {
   if (regionId == 0)
   {

-    m_text.DeleteContents(TRUE);
+    m_text.DeleteContents(true);

     m_text.Clear();
     m_text.Clear();

-    m_text.DeleteContents(FALSE);
+    m_text.DeleteContents(false);

   }
   }

-  wxNode *node = m_regions.Nth(regionId);
+  wxNode *node = m_regions.Item(regionId);

   if (!node)
     return;
   if (!node)
     return;

-  wxShapeRegion *region = (wxShapeRegion *)node->Data();
+  wxShapeRegion *region = (wxShapeRegion *)node->GetData();

   region->ClearText();
 }
 
 void wxShape::ClearRegions()
 {
   region->ClearText();
 }
 
 void wxShape::ClearRegions()
 {

-  wxNode *node = m_regions.First();
+  wxNode *node = m_regions.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShapeRegion *region = (wxShapeRegion *)node->Data();
-    wxNode *next = node->Next();
+    wxShapeRegion *region = (wxShapeRegion *)node->GetData();
+    wxNode *next = node->GetNext();

     delete region;
     delete node;
     node = next;
     delete region;
     delete node;
     node = next;


@@ -537,9 +521,9 @@ void wxShape::AddRegion(wxShapeRegion *region)
 
 void wxShape::SetDefaultRegionSize()
 {
 
 void wxShape::SetDefaultRegionSize()
 {

-  wxNode *node = m_regions.First();
+  wxNode *node = m_regions.GetFirst();

   if (!node) return;
   if (!node) return;

-  wxShapeRegion *region = (wxShapeRegion *)node->Data();
+  wxShapeRegion *region = (wxShapeRegion *)node->GetData();

   double w, h;
   GetBoundingBoxMin(&w, &h);
   region->SetSize(w, h);
   double w, h;
   GetBoundingBoxMin(&w, &h);
   region->SetSize(w, h);


@@ -548,7 +532,7 @@ void wxShape::SetDefaultRegionSize()
 bool wxShape::HitTest(double x, double y, int *attachment, double *distance)
 {
 //  if (!sensitive)
 bool wxShape::HitTest(double x, double y, int *attachment, double *distance)
 {
 //  if (!sensitive)

-//    return FALSE;
+//    return false;

 
   double width = 0.0, height = 0.0;
   GetBoundingBoxMin(&width, &height);
 
   double width = 0.0, height = 0.0;
   GetBoundingBoxMin(&width, &height);


@@ -592,40 +576,41 @@ bool wxShape::HitTest(double x, double y, int *attachment, double *distance)
     }
     *attachment = nearest_attachment;
     *distance = nearest;
     }
     *attachment = nearest_attachment;
     *distance = nearest;

-    return TRUE;
+    return true;

   }
   }

-  else return FALSE;
+  else return false;

 }
 
 // Format a text string according to the region size, adding
 // strings with positions to region text list
 
 }
 
 // Format a text string according to the region size, adding
 // strings with positions to region text list
 

-static bool GraphicsInSizeToContents = FALSE; // Infinite recursion elimination
+static bool GraphicsInSizeToContents = false; // Infinite recursion elimination

 void wxShape::FormatText(wxDC& dc, const wxString& s, int i)
 {
   double w, h;
   ClearText(i);
 
 void wxShape::FormatText(wxDC& dc, const wxString& s, int i)
 {
   double w, h;
   ClearText(i);
 

-  if (m_regions.Number() < 1)
+  if (m_regions.GetCount() < 1)

     return;
     return;

-  wxNode *node = m_regions.Nth(i);
+  wxNode *node = m_regions.Item(i);

   if (!node)
     return;
 
   if (!node)
     return;
 

-  wxShapeRegion *region = (wxShapeRegion *)node->Data();
-  region->SetText(s);
+  wxShapeRegion *region = (wxShapeRegion *)node->GetData();
+  // region->SetText(s);  // don't set the formatted text yet, it will be done below
+  region->m_regionText = s;

   dc.SetFont(* region->GetFont());
 
   region->GetSize(&w, &h);
 
   dc.SetFont(* region->GetFont());
 
   region->GetSize(&w, &h);
 

-  wxStringList *stringList = oglFormatText(dc, s, (w-5), (h-5), region->GetFormatMode());
-  node = stringList->First();
+  wxStringList *stringList = oglFormatText(dc, s, (w-2*m_textMarginX), (h-2*m_textMarginY), region->GetFormatMode());
+  node = (wxNode*)stringList->GetFirst();

   while (node)
   {
   while (node)
   {

-    char *s = (char *)node->Data();
+    wxChar *s = (wxChar *)node->GetData();

     wxShapeTextLine *line = new wxShapeTextLine(0.0, 0.0, s);
     region->GetFormattedText().Append((wxObject *)line);
     wxShapeTextLine *line = new wxShapeTextLine(0.0, 0.0, s);
     region->GetFormattedText().Append((wxObject *)line);

-    node = node->Next();
+    node = node->GetNext();

   }
   delete stringList;
   double actualW = w;
   }
   delete stringList;
   double actualW = w;


@@ -633,11 +618,11 @@ void wxShape::FormatText(wxDC& dc, const wxString& s, int i)
   // Don't try to resize an object with more than one image (this case should be dealt
   // with by overriden handlers)
   if ((region->GetFormatMode() & FORMAT_SIZE_TO_CONTENTS) &&
   // Don't try to resize an object with more than one image (this case should be dealt
   // with by overriden handlers)
   if ((region->GetFormatMode() & FORMAT_SIZE_TO_CONTENTS) &&

-      (region->GetFormattedText().Number() > 0) &&
-      (m_regions.Number() == 1) && !GraphicsInSizeToContents)
+      (region->GetFormattedText().GetCount() > 0) &&
+      (m_regions.GetCount() == 1) && !GraphicsInSizeToContents)

   {
     oglGetCentredTextExtent(dc, &(region->GetFormattedText()), m_xpos, m_ypos, w, h, &actualW, &actualH);
   {
     oglGetCentredTextExtent(dc, &(region->GetFormattedText()), m_xpos, m_ypos, w, h, &actualW, &actualH);

-    if ((actualW+m_textMarginX != w ) || (actualH+m_textMarginY != h))
+    if ((actualW+2*m_textMarginX != w ) || (actualH+2*m_textMarginY != h))

     {
       // If we are a descendant of a composite, must make sure the composite gets
       // resized properly
     {
       // If we are a descendant of a composite, must make sure the composite gets
       // resized properly


@@ -646,11 +631,11 @@ void wxShape::FormatText(wxDC& dc, const wxString& s, int i)
       if (topAncestor != this)
       {
         // Make sure we don't recurse infinitely
       if (topAncestor != this)
       {
         // Make sure we don't recurse infinitely

-        GraphicsInSizeToContents = TRUE;
+        GraphicsInSizeToContents = true;

 
         wxCompositeShape *composite = (wxCompositeShape *)topAncestor;
         composite->Erase(dc);
 
         wxCompositeShape *composite = (wxCompositeShape *)topAncestor;
         composite->Erase(dc);

-        SetSize(actualW+m_textMarginX, actualH+m_textMarginY);
+        SetSize(actualW+2*m_textMarginX, actualH+2*m_textMarginY);

         Move(dc, m_xpos, m_ypos);
         composite->CalculateSize();
         if (composite->Selected())
         Move(dc, m_xpos, m_ypos);
         composite->CalculateSize();
         if (composite->Selected())


@@ -662,21 +647,21 @@ void wxShape::FormatText(wxDC& dc, const wxString& s, int i)
         // Where infinite recursion might happen if we didn't stop it
         composite->Draw(dc);
 
         // Where infinite recursion might happen if we didn't stop it
         composite->Draw(dc);
 

-        GraphicsInSizeToContents = FALSE;
+        GraphicsInSizeToContents = false;

       }
       else
       {
         Erase(dc);
       }
       else
       {
         Erase(dc);

-        SetSize(actualW+m_textMarginX, actualH+m_textMarginY);
+        SetSize(actualW+2*m_textMarginX, actualH+2*m_textMarginY);

         Move(dc, m_xpos, m_ypos);
       }
         Move(dc, m_xpos, m_ypos);
       }

-      SetSize(actualW+m_textMarginX, actualH+m_textMarginY);
+      SetSize(actualW+2*m_textMarginX, actualH+2*m_textMarginY);

       Move(dc, m_xpos, m_ypos);
       EraseContents(dc);
     }
   }
       Move(dc, m_xpos, m_ypos);
       EraseContents(dc);
     }
   }

-  oglCentreText(dc, &(region->GetFormattedText()), m_xpos, m_ypos, actualW, actualH, region->GetFormatMode());
-  m_formatted = TRUE;
+  oglCentreText(dc, &(region->GetFormattedText()), m_xpos, m_ypos, actualW-2*m_textMarginX, actualH-2*m_textMarginY, region->GetFormatMode());
+  m_formatted = true;

 }
 
 void wxShape::Recentre(wxDC& dc)
 }
 
 void wxShape::Recentre(wxDC& dc)


@@ -684,23 +669,23 @@ void wxShape::Recentre(wxDC& dc)
   double w, h;
   GetBoundingBoxMin(&w, &h);
 
   double w, h;
   GetBoundingBoxMin(&w, &h);
 

-  int noRegions = m_regions.Number();
+  int noRegions = m_regions.GetCount();

   for (int i = 0; i < noRegions; i++)
   {
   for (int i = 0; i < noRegions; i++)
   {

-    wxNode *node = m_regions.Nth(i);
+    wxNode *node = m_regions.Item(i);

     if (node)
     {
     if (node)
     {

-      wxShapeRegion *region = (wxShapeRegion *)node->Data();
-      oglCentreText(dc, &(region->GetFormattedText()), m_xpos, m_ypos, w, h, region->GetFormatMode());
+      wxShapeRegion *region = (wxShapeRegion *)node->GetData();
+      oglCentreText(dc, &(region->GetFormattedText()), m_xpos, m_ypos, w-2*m_textMarginX, h-2*m_textMarginY, region->GetFormatMode());

     }
   }
 }
 
     }
   }
 }
 

-bool wxShape::GetPerimeterPoint(double x1, double y1,
-                                     double x2, double y2,
-                                     double *x3, double *y3)
+bool wxShape::GetPerimeterPoint(double WXUNUSED(x1), double WXUNUSED(y1),
+                                     double WXUNUSED(x2), double WXUNUSED(y2),
+                                     double *WXUNUSED(x3), double *WXUNUSED(y3))

 {
 {

-  return FALSE;
+  return false;

 }
 
 void wxShape::SetPen(wxPen *the_pen)
 }
 
 void wxShape::SetPen(wxPen *the_pen)


@@ -731,90 +716,89 @@ wxShape *wxShape::GetTopAncestor()
 void wxShape::SetFont(wxFont *the_font, int regionId)
 {
   m_font = the_font;
 void wxShape::SetFont(wxFont *the_font, int regionId)
 {
   m_font = the_font;

-  wxNode *node = m_regions.Nth(regionId);
+  wxNode *node = m_regions.Item(regionId);

   if (!node)
     return;
   if (!node)
     return;

-  wxShapeRegion *region = (wxShapeRegion *)node->Data();
+  wxShapeRegion *region = (wxShapeRegion *)node->GetData();

   region->SetFont(the_font);
 }
 
 wxFont *wxShape::GetFont(int n) const
 {
   region->SetFont(the_font);
 }
 
 wxFont *wxShape::GetFont(int n) const
 {

-  wxNode *node = m_regions.Nth(n);
+  wxNode *node = m_regions.Item(n);

   if (!node)
     return NULL;
   if (!node)
     return NULL;

-  wxShapeRegion *region = (wxShapeRegion *)node->Data();
+  wxShapeRegion *region = (wxShapeRegion *)node->GetData();

   return region->GetFont();
 }
 
 void wxShape::SetFormatMode(int mode, int regionId)
 {
   return region->GetFont();
 }
 
 void wxShape::SetFormatMode(int mode, int regionId)
 {

-  wxNode *node = m_regions.Nth(regionId);
+  wxNode *node = m_regions.Item(regionId);

   if (!node)
     return;
   if (!node)
     return;

-  wxShapeRegion *region = (wxShapeRegion *)node->Data();
+  wxShapeRegion *region = (wxShapeRegion *)node->GetData();

   region->SetFormatMode(mode);
 }
 
 int wxShape::GetFormatMode(int regionId) const
 {
   region->SetFormatMode(mode);
 }
 
 int wxShape::GetFormatMode(int regionId) const
 {

-  wxNode *node = m_regions.Nth(regionId);
+  wxNode *node = m_regions.Item(regionId);

   if (!node)
     return 0;
   if (!node)
     return 0;

-  wxShapeRegion *region = (wxShapeRegion *)node->Data();
+  wxShapeRegion *region = (wxShapeRegion *)node->GetData();

   return region->GetFormatMode();
 }
 
 void wxShape::SetTextColour(const wxString& the_colour, int regionId)
 {
   return region->GetFormatMode();
 }
 
 void wxShape::SetTextColour(const wxString& the_colour, int regionId)
 {

-  wxColour *wxcolour = wxTheColourDatabase->FindColour(the_colour);
-  m_textColour = wxcolour;
+  m_textColour = wxTheColourDatabase->Find(the_colour);

   m_textColourName = the_colour;
 
   m_textColourName = the_colour;
 

-  wxNode *node = m_regions.Nth(regionId);
+  wxNode *node = m_regions.Item(regionId);

   if (!node)
     return;
   if (!node)
     return;

-  wxShapeRegion *region = (wxShapeRegion *)node->Data();
+  wxShapeRegion *region = (wxShapeRegion *)node->GetData();

   region->SetColour(the_colour);
 }
 
 wxString wxShape::GetTextColour(int regionId) const
 {
   region->SetColour(the_colour);
 }
 
 wxString wxShape::GetTextColour(int regionId) const
 {

-  wxNode *node = m_regions.Nth(regionId);
+  wxNode *node = m_regions.Item(regionId);

   if (!node)
   if (!node)

-    return wxString("");
-  wxShapeRegion *region = (wxShapeRegion *)node->Data();
+    return wxEmptyString;
+  wxShapeRegion *region = (wxShapeRegion *)node->GetData();

   return region->GetColour();
 }
 
 void wxShape::SetRegionName(const wxString& name, int regionId)
 {
   return region->GetColour();
 }
 
 void wxShape::SetRegionName(const wxString& name, int regionId)
 {

-  wxNode *node = m_regions.Nth(regionId);
+  wxNode *node = m_regions.Item(regionId);

   if (!node)
     return;
   if (!node)
     return;

-  wxShapeRegion *region = (wxShapeRegion *)node->Data();
+  wxShapeRegion *region = (wxShapeRegion *)node->GetData();

   region->SetName(name);
 }
 
 wxString wxShape::GetRegionName(int regionId)
 {
   region->SetName(name);
 }
 
 wxString wxShape::GetRegionName(int regionId)
 {

-  wxNode *node = m_regions.Nth(regionId);
+  wxNode *node = m_regions.Item(regionId);

   if (!node)
   if (!node)

-    return wxString("");
-  wxShapeRegion *region = (wxShapeRegion *)node->Data();
+    return wxEmptyString;
+  wxShapeRegion *region = (wxShapeRegion *)node->GetData();

   return region->GetName();
 }
 
 int wxShape::GetRegionId(const wxString& name)
 {
   return region->GetName();
 }
 
 int wxShape::GetRegionId(const wxString& name)
 {

-  wxNode *node = m_regions.First();
+  wxNode *node = m_regions.GetFirst();

   int i = 0;
   while (node)
   {
   int i = 0;
   while (node)
   {

-    wxShapeRegion *region = (wxShapeRegion *)node->Data();
+    wxShapeRegion *region = (wxShapeRegion *)node->GetData();

     if (region->GetName() == name)
       return i;
     if (region->GetName() == name)
       return i;

-    node = node->Next();
+    node = node->GetNext();

     i ++;
   }
   return -1;
     i ++;
   }
   return -1;


@@ -824,26 +808,27 @@ int wxShape::GetRegionId(const wxString& name)
 void wxShape::NameRegions(const wxString& parentName)
 {
   int n = GetNumberOfTextRegions();
 void wxShape::NameRegions(const wxString& parentName)
 {
   int n = GetNumberOfTextRegions();

-  char buf[100];
+  wxString buff;

   for (int i = 0; i < n; i++)
   {
     if (parentName.Length() > 0)
   for (int i = 0; i < n; i++)
   {
     if (parentName.Length() > 0)

-      sprintf(buf, "%s.%d", (const char*) parentName, i);
+      buff << parentName << wxT(".") << i;

     else
     else

-      sprintf(buf, "%d", i);
-    SetRegionName(buf, i);
+      buff << i;
+    SetRegionName(buff, i);

   }
   }

-  wxNode *node = m_children.First();
+  wxNode *node = m_children.GetFirst();

   int j = 0;
   while (node)
   {
   int j = 0;
   while (node)
   {

-    wxShape *child = (wxShape *)node->Data();
+    buff.Empty();
+    wxShape *child = (wxShape *)node->GetData();

     if (parentName.Length() > 0)
     if (parentName.Length() > 0)

-      sprintf(buf, "%s.%d", (const char*) parentName, j);
+      buff << parentName << wxT(".") << j;

     else
     else

-      sprintf(buf, "%d", j);
-    child->NameRegions(buf);
-    node = node->Next();
+      buff << j;
+    child->NameRegions(buff);
+    node = node->GetNext();

     j ++;
   }
 }
     j ++;
   }
 }


@@ -858,14 +843,14 @@ wxShape *wxShape::FindRegion(const wxString& name, int *regionId)
     return this;
   }
 
     return this;
   }
 

-  wxNode *node = m_children.First();
+  wxNode *node = m_children.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShape *child = (wxShape *)node->Data();
+    wxShape *child = (wxShape *)node->GetData();

     wxShape *actualImage = child->FindRegion(name, regionId);
     if (actualImage)
       return actualImage;
     wxShape *actualImage = child->FindRegion(name, regionId);
     if (actualImage)
       return actualImage;

-    node = node->Next();
+    node = node->GetNext();

   }
   return NULL;
 }
   }
   return NULL;
 }


@@ -878,15 +863,15 @@ void wxShape::FindRegionNames(wxStringList& list)
   for (int i = 0; i < n; i++)
   {
     wxString name(GetRegionName(i));
   for (int i = 0; i < n; i++)
   {
     wxString name(GetRegionName(i));

-    list.Add((const char*) name);
+    list.Add(name);

   }
 
   }
 

-  wxNode *node = m_children.First();
+  wxNode *node = m_children.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShape *child = (wxShape *)node->Data();
+    wxShape *child = (wxShape *)node->GetData();

     child->FindRegionNames(list);
     child->FindRegionNames(list);

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
   }
 }
 


@@ -894,16 +879,16 @@ void wxShape::AssignNewIds()
 {
 //  if (m_id == 0)
   m_id = wxNewId();
 {
 //  if (m_id == 0)
   m_id = wxNewId();

-  wxNode *node = m_children.First();
+  wxNode *node = m_children.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShape *child = (wxShape *)node->Data();
+    wxShape *child = (wxShape *)node->GetData();

     child->AssignNewIds();
     child->AssignNewIds();

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
   }
 }
 

-void wxShape::OnDraw(wxDC& dc)
+void wxShape::OnDraw(wxDC& WXUNUSED(dc))

 {
 }
 
 {
 }
 


@@ -912,12 +897,12 @@ void wxShape::OnMoveLinks(wxDC& dc)
   // Want to set the ends of all attached links
   // to point to/from this object
 
   // Want to set the ends of all attached links
   // to point to/from this object
 

-  wxNode *current = m_lines.First();
+  wxNode *current = m_lines.GetFirst();

   while (current)
   {
   while (current)
   {

-    wxLineShape *line = (wxLineShape *)current->Data();
+    wxLineShape *line = (wxLineShape *)current->GetData();

     line->GetEventHandler()->OnMoveLink(dc);
     line->GetEventHandler()->OnMoveLink(dc);

-    current = current->Next();
+    current = current->GetNext();

   }
 }
 
   }
 }
 


@@ -926,23 +911,23 @@ void wxShape::OnDrawContents(wxDC& dc)
 {
   double bound_x, bound_y;
   GetBoundingBoxMin(&bound_x, &bound_y);
 {
   double bound_x, bound_y;
   GetBoundingBoxMin(&bound_x, &bound_y);

-    if (m_regions.Number() < 1) return;
+    if (m_regions.GetCount() < 1) return;

 
     if (m_pen) dc.SetPen(* m_pen);
 
 
     if (m_pen) dc.SetPen(* m_pen);
 

-    wxShapeRegion *region = (wxShapeRegion *)m_regions.First()->Data();
+    wxShapeRegion *region = (wxShapeRegion *)m_regions.GetFirst()->GetData();

     if (region->GetFont()) dc.SetFont(* region->GetFont());
 
     if (region->GetFont()) dc.SetFont(* region->GetFont());
 

-    dc.SetTextForeground(* (region->GetActualColourObject()));
+    dc.SetTextForeground(region->GetActualColourObject());

     dc.SetBackgroundMode(wxTRANSPARENT);
     if (!m_formatted)
     {
     dc.SetBackgroundMode(wxTRANSPARENT);
     if (!m_formatted)
     {

-      oglCentreText(dc, &(region->GetFormattedText()), m_xpos, m_ypos, bound_x, bound_y, region->GetFormatMode());
-      m_formatted = TRUE;
+      oglCentreText(dc, &(region->GetFormattedText()), m_xpos, m_ypos, bound_x-2*m_textMarginX, bound_y-2*m_textMarginY, region->GetFormatMode());
+      m_formatted = true;

     }
     if (!GetDisableLabel())
     {
     }
     if (!GetDisableLabel())
     {

-      oglDrawFormattedText(dc, &(region->GetFormattedText()), m_xpos, m_ypos, bound_x, bound_y, region->GetFormatMode());
+      oglDrawFormattedText(dc, &(region->GetFormattedText()), m_xpos, m_ypos, bound_x-2*m_textMarginX, bound_y-2*m_textMarginY, region->GetFormatMode());

     }
 }
 
     }
 }
 


@@ -951,16 +936,16 @@ void wxShape::DrawContents(wxDC& dc)
   GetEventHandler()->OnDrawContents(dc);
 }
 
   GetEventHandler()->OnDrawContents(dc);
 }
 

-void wxShape::OnSize(double x, double y)
+void wxShape::OnSize(double WXUNUSED(x), double WXUNUSED(y))

 {
 }
 
 {
 }
 

-bool wxShape::OnMovePre(wxDC& dc, double x, double y, double old_x, double old_y, bool display)
+bool wxShape::OnMovePre(wxDC& WXUNUSED(dc), double WXUNUSED(x), double WXUNUSED(y), double WXUNUSED(old_x), double WXUNUSED(old_y), bool WXUNUSED(display))

 {
 {

-  return TRUE;
+  return true;

 }
 
 }
 

-void wxShape::OnMovePost(wxDC& dc, double x, double y, double old_x, double old_y, bool display)
+void wxShape::OnMovePost(wxDC& WXUNUSED(dc), double WXUNUSED(x), double WXUNUSED(y), double WXUNUSED(old_x), double WXUNUSED(old_y), bool WXUNUSED(display))

 {
 }
 
 {
 }
 


@@ -970,12 +955,12 @@ void wxShape::OnErase(wxDC& dc)
     return;
 
   // Erase links
     return;
 
   // Erase links

-  wxNode *current = m_lines.First();
+  wxNode *current = m_lines.GetFirst();

   while (current)
   {
   while (current)
   {

-    wxLineShape *line = (wxLineShape *)current->Data();
+    wxLineShape *line = (wxLineShape *)current->GetData();

     line->GetEventHandler()->OnErase(dc);
     line->GetEventHandler()->OnErase(dc);

-    current = current->Next();
+    current = current->GetNext();

   }
   GetEventHandler()->OnEraseContents(dc);
 }
   }
   GetEventHandler()->OnEraseContents(dc);
 }


@@ -1009,23 +994,23 @@ void wxShape::EraseLinks(wxDC& dc, int attachment, bool recurse)
   if (!m_visible)
     return;
 
   if (!m_visible)
     return;
 

-  wxNode *current = m_lines.First();
+  wxNode *current = m_lines.GetFirst();

   while (current)
   {
   while (current)
   {

-    wxLineShape *line = (wxLineShape *)current->Data();
+    wxLineShape *line = (wxLineShape *)current->GetData();

     if (attachment == -1 || ((line->GetTo() == this && line->GetAttachmentTo() == attachment) ||
                              (line->GetFrom() == this && line->GetAttachmentFrom() == attachment)))
       line->GetEventHandler()->OnErase(dc);
     if (attachment == -1 || ((line->GetTo() == this && line->GetAttachmentTo() == attachment) ||
                              (line->GetFrom() == this && line->GetAttachmentFrom() == attachment)))
       line->GetEventHandler()->OnErase(dc);

-    current = current->Next();
+    current = current->GetNext();

   }
   if (recurse)
   {
   }
   if (recurse)
   {

-    wxNode *node = m_children.First();
+    wxNode *node = m_children.GetFirst();

     while (node)
     {
     while (node)
     {

-      wxShape *child = (wxShape *)node->Data();
+      wxShape *child = (wxShape *)node->GetData();

       child->EraseLinks(dc, attachment, recurse);
       child->EraseLinks(dc, attachment, recurse);

-      node = node->Next();
+      node = node->GetNext();

     }
   }
 }
     }
   }
 }


@@ -1035,29 +1020,29 @@ void wxShape::DrawLinks(wxDC& dc, int attachment, bool recurse)
   if (!m_visible)
     return;
 
   if (!m_visible)
     return;
 

-  wxNode *current = m_lines.First();
+  wxNode *current = m_lines.GetFirst();

   while (current)
   {
   while (current)
   {

-    wxLineShape *line = (wxLineShape *)current->Data();
+    wxLineShape *line = (wxLineShape *)current->GetData();

     if (attachment == -1 ||
         (line->GetTo() == this && line->GetAttachmentTo() == attachment) ||
         (line->GetFrom() == this && line->GetAttachmentFrom() == attachment))
       line->Draw(dc);
     if (attachment == -1 ||
         (line->GetTo() == this && line->GetAttachmentTo() == attachment) ||
         (line->GetFrom() == this && line->GetAttachmentFrom() == attachment))
       line->Draw(dc);

-    current = current->Next();
+    current = current->GetNext();

   }
   if (recurse)
   {
   }
   if (recurse)
   {

-    wxNode *node = m_children.First();
+    wxNode *node = m_children.GetFirst();

     while (node)
     {
     while (node)
     {

-      wxShape *child = (wxShape *)node->Data();
+      wxShape *child = (wxShape *)node->GetData();

       child->DrawLinks(dc, attachment, recurse);
       child->DrawLinks(dc, attachment, recurse);

-      node = node->Next();
+      node = node->GetNext();

     }
   }
 }
 
     }
   }
 }
 

-// Returns TRUE if pt1 <= pt2 in the sense that one point comes before another on an
+// Returns true if pt1 <= pt2 in the sense that one point comes before another on an

 // edge of the shape.
 // attachmentPoint is the attachment point (= side) in question.
 
 // edge of the shape.
 // attachmentPoint is the attachment point (= side) in question.
 


@@ -1071,24 +1056,22 @@ bool wxShape::AttachmentSortTest(int attachmentPoint, const wxRealPoint& pt1, co
         case 2:
         {
           return (pt1.x <= pt2.x) ;
         case 2:
         {
           return (pt1.x <= pt2.x) ;

-          break;

         }
         case 1:
         case 3:
         {
           return (pt1.y <= pt2.y) ;
         }
         case 1:
         case 3:
         {
           return (pt1.y <= pt2.y) ;

-          break;

         }
     }
 
         }
     }
 

-    return FALSE;
+    return false;

 }
 
 bool wxShape::MoveLineToNewAttachment(wxDC& dc, wxLineShape *to_move,
                                        double x, double y)
 {
   if (GetAttachmentMode() == ATTACHMENT_MODE_NONE)
 }
 
 bool wxShape::MoveLineToNewAttachment(wxDC& dc, wxLineShape *to_move,
                                        double x, double y)
 {
   if (GetAttachmentMode() == ATTACHMENT_MODE_NONE)

-      return FALSE;
+      return false;

 
   int newAttachment, oldAttachment;
   double distance;
 
   int newAttachment, oldAttachment;
   double distance;


@@ -1097,7 +1080,7 @@ bool wxShape::MoveLineToNewAttachment(wxDC& dc, wxLineShape *to_move,
   // the user has moved the point to
   bool hit = HitTest(x, y, &newAttachment, &distance);
   if (!hit)
   // the user has moved the point to
   bool hit = HitTest(x, y, &newAttachment, &distance);
   if (!hit)

-    return FALSE;
+    return false;

 
   EraseLinks(dc);
 
 
   EraseLinks(dc);
 


@@ -1110,11 +1093,11 @@ bool wxShape::MoveLineToNewAttachment(wxDC& dc, wxLineShape *to_move,
   wxList newOrdering;
 
   // First, add all links to the new list.
   wxList newOrdering;
 
   // First, add all links to the new list.

-  wxNode *node = m_lines.First();
+  wxNode *node = m_lines.GetFirst();

   while (node)
   {
   while (node)
   {

-    newOrdering.Append(node->Data());
-    node = node->Next();
+    newOrdering.Append(node->GetData());
+    node = node->GetNext();

   }
 
   // Delete the line object from the list of links; we're going to move
   }
 
   // Delete the line object from the list of links; we're going to move


@@ -1124,12 +1107,12 @@ bool wxShape::MoveLineToNewAttachment(wxDC& dc, wxLineShape *to_move,
   double old_x = (double) -99999.9;
   double old_y = (double) -99999.9;
 
   double old_x = (double) -99999.9;
   double old_y = (double) -99999.9;
 

-  node = newOrdering.First();
-  bool found = FALSE;
+  node = newOrdering.GetFirst();
+  bool found = false;

 
   while (!found && node)
   {
 
   while (!found && node)
   {

-    wxLineShape *line = (wxLineShape *)node->Data();
+    wxLineShape *line = (wxLineShape *)node->GetData();

     if ((line->GetTo() == this && oldAttachment == line->GetAttachmentTo()) ||
         (line->GetFrom() == this && oldAttachment == line->GetAttachmentFrom()))
     {
     if ((line->GetTo() == this && oldAttachment == line->GetAttachmentTo()) ||
         (line->GetFrom() == this && oldAttachment == line->GetAttachmentFrom()))
     {


@@ -1152,14 +1135,14 @@ bool wxShape::MoveLineToNewAttachment(wxDC& dc, wxLineShape *to_move,
 
       if (AttachmentSortTest(newAttachment, newPoint, thisPoint) && AttachmentSortTest(newAttachment, lastPoint, newPoint))
       {
 
       if (AttachmentSortTest(newAttachment, newPoint, thisPoint) && AttachmentSortTest(newAttachment, lastPoint, newPoint))
       {

-        found = TRUE;
+        found = true;

         newOrdering.Insert(node, to_move);
       }
 
       old_x = xp;
       old_y = yp;
     }
         newOrdering.Insert(node, to_move);
       }
 
       old_x = xp;
       old_y = yp;
     }

-    node = node->Next();
+    node = node->GetNext();

   }
 
   if (!found)
   }
 
   if (!found)


@@ -1167,7 +1150,7 @@ bool wxShape::MoveLineToNewAttachment(wxDC& dc, wxLineShape *to_move,
 
   GetEventHandler()->OnChangeAttachment(newAttachment, to_move, newOrdering);
 
 
   GetEventHandler()->OnChangeAttachment(newAttachment, to_move, newOrdering);
 

-  return TRUE;
+  return true;

 }
 
 void wxShape::OnChangeAttachment(int attachment, wxLineShape* line, wxList& ordering)
 }
 
 void wxShape::OnChangeAttachment(int attachment, wxLineShape* line, wxList& ordering)


@@ -1193,36 +1176,36 @@ void wxShape::ApplyAttachmentOrdering(wxList& linesToSort)
   // This is a temporary store of all the lines.
   wxList linesStore;
 
   // This is a temporary store of all the lines.
   wxList linesStore;
 

-  wxNode *node = m_lines.First();
+  wxNode *node = m_lines.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxLineShape *line = (wxLineShape *)node->Data();
+    wxLineShape *line = (wxLineShape *)node->GetData();

     linesStore.Append(line);
     linesStore.Append(line);

-    node = node->Next();;
+    node = node->GetNext();;

   }
 
   m_lines.Clear();
 
   }
 
   m_lines.Clear();
 

-  node = linesToSort.First();
+  node = linesToSort.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxLineShape *line = (wxLineShape *)node->Data();
+    wxLineShape *line = (wxLineShape *)node->GetData();

     if (linesStore.Member(line))
     {
       // Done this one
       linesStore.DeleteObject(line);
       m_lines.Append(line);
     }
     if (linesStore.Member(line))
     {
       // Done this one
       linesStore.DeleteObject(line);
       m_lines.Append(line);
     }

-    node = node->Next();
+    node = node->GetNext();

   }
 
   // Now add any lines that haven't been listed in linesToSort.
   }
 
   // Now add any lines that haven't been listed in linesToSort.

-  node = linesStore.First();
+  node = linesStore.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxLineShape *line = (wxLineShape *)node->Data();
+    wxLineShape *line = (wxLineShape *)node->GetData();

     m_lines.Append(line);
     m_lines.Append(line);

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
   }
 }
 


@@ -1235,11 +1218,11 @@ void wxShape::SortLines(int attachment, wxList& linesToSort)
   // point. We'll tick them off as we've processed them.
   wxList linesAtThisAttachment;
 
   // point. We'll tick them off as we've processed them.
   wxList linesAtThisAttachment;
 

-  wxNode *node = m_lines.First();
+  wxNode *node = m_lines.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxLineShape *line = (wxLineShape *)node->Data();
-    wxNode *next = node->Next();
+    wxLineShape *line = (wxLineShape *)node->GetData();
+    wxNode *next = node->GetNext();

     if ((line->GetTo() == this && line->GetAttachmentTo() == attachment) ||
         (line->GetFrom() == this && line->GetAttachmentFrom() == attachment))
     {
     if ((line->GetTo() == this && line->GetAttachmentTo() == attachment) ||
         (line->GetFrom() == this && line->GetAttachmentFrom() == attachment))
     {


@@ -1247,33 +1230,33 @@ void wxShape::SortLines(int attachment, wxList& linesToSort)
       delete node;
       node = next;
     }
       delete node;
       node = next;
     }

-    else node = node->Next();
+    else node = node->GetNext();

   }
 
   }
 

-  node = linesToSort.First();
+  node = linesToSort.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxLineShape *line = (wxLineShape *)node->Data();
+    wxLineShape *line = (wxLineShape *)node->GetData();

     if (linesAtThisAttachment.Member(line))
     {
       // Done this one
       linesAtThisAttachment.DeleteObject(line);
       m_lines.Append(line);
     }
     if (linesAtThisAttachment.Member(line))
     {
       // Done this one
       linesAtThisAttachment.DeleteObject(line);
       m_lines.Append(line);
     }

-    node = node->Next();
+    node = node->GetNext();

   }
 
   // Now add any lines that haven't been listed in linesToSort.
   }
 
   // Now add any lines that haven't been listed in linesToSort.

-  node = linesAtThisAttachment.First();
+  node = linesAtThisAttachment.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxLineShape *line = (wxLineShape *)node->Data();
+    wxLineShape *line = (wxLineShape *)node->GetData();

     m_lines.Append(line);
     m_lines.Append(line);

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
   }
 }
 

-void wxShape::OnHighlight(wxDC& dc)
+void wxShape::OnHighlight(wxDC& WXUNUSED(dc))

 {
 }
 
 {
 }
 


@@ -1329,7 +1312,7 @@ void wxShape::OnDragLeft(bool draw, double x, double y, int keys, int attachment
 
   dc.SetLogicalFunction(OGLRBLF);
 
 
   dc.SetLogicalFunction(OGLRBLF);
 

-  wxPen dottedPen(wxColour(0, 0, 0), 1, wxDOT);
+  wxPen dottedPen(*wxBLACK, 1, wxDOT);

   dc.SetPen(dottedPen);
   dc.SetBrush(* wxTRANSPARENT_BRUSH);
 
   dc.SetPen(dottedPen);
   dc.SetBrush(* wxTRANSPARENT_BRUSH);
 


@@ -1374,7 +1357,7 @@ void wxShape::OnBeginDragLeft(double x, double y, int keys, int attachment)
 //  m_xpos = xx; m_ypos = yy;
   dc.SetLogicalFunction(OGLRBLF);
 
 //  m_xpos = xx; m_ypos = yy;
   dc.SetLogicalFunction(OGLRBLF);
 

-  wxPen dottedPen(wxColour(0, 0, 0), 1, wxDOT);
+  wxPen dottedPen(*wxBLACK, 1, wxDOT);

   dc.SetPen(dottedPen);
   dc.SetBrush((* wxTRANSPARENT_BRUSH));
 
   dc.SetPen(dottedPen);
   dc.SetBrush((* wxTRANSPARENT_BRUSH));
 


@@ -1550,12 +1533,12 @@ void wxShape::Flash()
 void wxShape::Show(bool show)
 {
   m_visible = show;
 void wxShape::Show(bool show)
 {
   m_visible = show;

-  wxNode *node = m_children.First();
+  wxNode *node = m_children.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShape *image = (wxShape *)node->Data();
+    wxShape *image = (wxShape *)node->GetData();

     image->Show(show);
     image->Show(show);

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
   }
 }
 


@@ -1563,7 +1546,7 @@ void wxShape::Erase(wxDC& dc)
 {
   GetEventHandler()->OnErase(dc);
   GetEventHandler()->OnEraseControlPoints(dc);
 {
   GetEventHandler()->OnErase(dc);
   GetEventHandler()->OnEraseControlPoints(dc);

-  GetEventHandler()->OnDrawBranches(dc, TRUE);
+  GetEventHandler()->OnDrawBranches(dc, true);

 }
 
 void wxShape::EraseContents(wxDC& dc)
 }
 
 void wxShape::EraseContents(wxDC& dc)


@@ -1573,19 +1556,19 @@ void wxShape::EraseContents(wxDC& dc)
 
 void wxShape::AddText(const wxString& string)
 {
 
 void wxShape::AddText(const wxString& string)
 {

-  wxNode *node = m_regions.First();
+  wxNode *node = m_regions.GetFirst();

   if (!node)
     return;
   if (!node)
     return;

-  wxShapeRegion *region = (wxShapeRegion *)node->Data();
+  wxShapeRegion *region = (wxShapeRegion *)node->GetData();

   region->ClearText();
   wxShapeTextLine *new_line =
       new wxShapeTextLine(0.0, 0.0, string);
   region->GetFormattedText().Append(new_line);
 
   region->ClearText();
   wxShapeTextLine *new_line =
       new wxShapeTextLine(0.0, 0.0, string);
   region->GetFormattedText().Append(new_line);
 

-  m_formatted = FALSE;
+  m_formatted = false;

 }
 
 }
 

-void wxShape::SetSize(double x, double y, bool recursive)
+void wxShape::SetSize(double x, double y, bool WXUNUSED(recursive))

 {
   SetAttachmentSize(x, y);
   SetDefaultRegionSize();
 {
   SetAttachmentSize(x, y);
   SetDefaultRegionSize();


@@ -1604,13 +1587,13 @@ void wxShape::SetAttachmentSize(double w, double h)
     scaleY = 1.0;
   else scaleY = h/height;
 
     scaleY = 1.0;
   else scaleY = h/height;
 

-  wxNode *node = m_attachmentPoints.First();
+  wxNode *node = m_attachmentPoints.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxAttachmentPoint *point = (wxAttachmentPoint *)node->Data();
+    wxAttachmentPoint *point = (wxAttachmentPoint *)node->GetData();

     point->m_x = (double)(point->m_x * scaleX);
     point->m_y = (double)(point->m_y * scaleY);
     point->m_x = (double)(point->m_x * scaleX);
     point->m_y = (double)(point->m_y * scaleY);

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
   }
 }
 


@@ -1629,9 +1612,9 @@ void wxShape::AddLine(wxLineShape *line, wxShape *other,
     {
         // Don't preserve old ordering if we have new ordering instructions
         m_lines.DeleteObject(line);
     {
         // Don't preserve old ordering if we have new ordering instructions
         m_lines.DeleteObject(line);

-        if (positionFrom < m_lines.Number())
+        if (positionFrom < (int) m_lines.GetCount())

         {
         {

-            wxNode* node = m_lines.Nth(positionFrom);
+            wxNode* node = m_lines.Item(positionFrom);

             m_lines.Insert(node, line);
         }
         else
             m_lines.Insert(node, line);
         }
         else


@@ -1647,9 +1630,9 @@ void wxShape::AddLine(wxLineShape *line, wxShape *other,
     {
         // Don't preserve old ordering if we have new ordering instructions
         other->m_lines.DeleteObject(line);
     {
         // Don't preserve old ordering if we have new ordering instructions
         other->m_lines.DeleteObject(line);

-        if (positionTo < other->m_lines.Number())
+        if (positionTo < (int) other->m_lines.GetCount())

         {
         {

-            wxNode* node = other->m_lines.Nth(positionTo);
+            wxNode* node = other->m_lines.Item(positionTo);

             other->m_lines.Insert(node, line);
         }
         else
             other->m_lines.Insert(node, line);
         }
         else


@@ -1664,9 +1647,9 @@ void wxShape::AddLine(wxLineShape *line, wxShape *other,
         m_lines.Append(line);
     else
     {
         m_lines.Append(line);
     else
     {

-        if (positionFrom < m_lines.Number())
+        if (positionFrom < m_lines.GetCount())

         {
         {

-            wxNode* node = m_lines.Nth(positionFrom);
+            wxNode* node = m_lines.Item(positionFrom);

             m_lines.Insert(node, line);
         }
         else
             m_lines.Insert(node, line);
         }
         else


@@ -1677,9 +1660,9 @@ void wxShape::AddLine(wxLineShape *line, wxShape *other,
         other->m_lines.Append(line);
     else
     {
         other->m_lines.Append(line);
     else
     {

-        if (positionTo < other->m_lines.Number())
+        if (positionTo < other->m_lines.GetCount())

         {
         {

-            wxNode* node = other->m_lines.Nth(positionTo);
+            wxNode* node = other->m_lines.Item(positionTo);

             other->m_lines.Insert(node, line);
         }
         else
             other->m_lines.Insert(node, line);
         }
         else


@@ -1702,115 +1685,115 @@ void wxShape::RemoveLine(wxLineShape *line)
   m_lines.DeleteObject(line);
 }
 
   m_lines.DeleteObject(line);
 }
 

-#ifdef PROLOGIO
+#if wxUSE_PROLOGIO

 void wxShape::WriteAttributes(wxExpr *clause)
 {
 void wxShape::WriteAttributes(wxExpr *clause)
 {

-  clause->AddAttributeValueString("type", GetClassInfo()->GetClassName());
-  clause->AddAttributeValue("id", m_id);
+  clause->AddAttributeValueString(_T("type"), GetClassInfo()->GetClassName());
+  clause->AddAttributeValue(_T("id"), m_id);

 
   if (m_pen)
   {
     int penWidth = m_pen->GetWidth();
     int penStyle = m_pen->GetStyle();
     if (penWidth != 1)
 
   if (m_pen)
   {
     int penWidth = m_pen->GetWidth();
     int penStyle = m_pen->GetStyle();
     if (penWidth != 1)

-      clause->AddAttributeValue("pen_width", (long)penWidth);
+      clause->AddAttributeValue(_T("pen_width"), (long)penWidth);

     if (penStyle != wxSOLID)
     if (penStyle != wxSOLID)

-      clause->AddAttributeValue("pen_style", (long)penStyle);
+      clause->AddAttributeValue(_T("pen_style"), (long)penStyle);

 
     wxString penColour = wxTheColourDatabase->FindName(m_pen->GetColour());
 
     wxString penColour = wxTheColourDatabase->FindName(m_pen->GetColour());

-    if (penColour == "")
+    if (penColour == wxEmptyString)

     {
       wxString hex(oglColourToHex(m_pen->GetColour()));
     {
       wxString hex(oglColourToHex(m_pen->GetColour()));

-      hex = wxString("#") + hex;
-      clause->AddAttributeValueString("pen_colour", hex);
+      hex = wxString(_T("#")) + hex;
+      clause->AddAttributeValueString(_T("pen_colour"), hex);

     }
     }

-    else if (penColour != "BLACK")
-      clause->AddAttributeValueString("pen_colour", penColour);
+    else if (penColour != _T("BLACK"))
+      clause->AddAttributeValueString(_T("pen_colour"), penColour);

   }
 
   if (m_brush)
   {
     wxString brushColour = wxTheColourDatabase->FindName(m_brush->GetColour());
 
   }
 
   if (m_brush)
   {
     wxString brushColour = wxTheColourDatabase->FindName(m_brush->GetColour());
 

-    if (brushColour == "")
+    if (brushColour == wxEmptyString)

     {
       wxString hex(oglColourToHex(m_brush->GetColour()));
     {
       wxString hex(oglColourToHex(m_brush->GetColour()));

-      hex = wxString("#") + hex;
-      clause->AddAttributeValueString("brush_colour", hex);
+      hex = wxString(_T("#")) + hex;
+      clause->AddAttributeValueString(_T("brush_colour"), hex);

     }
     }

-    else if (brushColour != "WHITE")
-      clause->AddAttributeValueString("brush_colour", brushColour);
+    else if (brushColour != _T("WHITE"))
+      clause->AddAttributeValueString(_T("brush_colour"), brushColour);

 
     if (m_brush->GetStyle() != wxSOLID)
 
     if (m_brush->GetStyle() != wxSOLID)

-      clause->AddAttributeValue("brush_style", (long)m_brush->GetStyle());
+      clause->AddAttributeValue(_T("brush_style"), (long)m_brush->GetStyle());

   }
 
   // Output line ids
 
   }
 
   // Output line ids
 

-  int n_lines = m_lines.Number();
+  int n_lines = m_lines.GetCount();

   if (n_lines > 0)
   {
     wxExpr *list = new wxExpr(wxExprList);
   if (n_lines > 0)
   {
     wxExpr *list = new wxExpr(wxExprList);

-    wxNode *node = m_lines.First();
+    wxNode *node = m_lines.GetFirst();

     while (node)
     {
     while (node)
     {

-      wxShape *line = (wxShape *)node->Data();
+      wxShape *line = (wxShape *)node->GetData();

       wxExpr *id_expr = new wxExpr(line->GetId());
       list->Append(id_expr);
       wxExpr *id_expr = new wxExpr(line->GetId());
       list->Append(id_expr);

-      node = node->Next();
+      node = node->GetNext();

     }
     }

-    clause->AddAttributeValue("arcs", list);
+    clause->AddAttributeValue(_T("arcs"), list);

   }
 
   // Miscellaneous members
   if (m_attachmentMode != 0)
   }
 
   // Miscellaneous members
   if (m_attachmentMode != 0)

-    clause->AddAttributeValue("use_attachments", (long)m_attachmentMode);
+    clause->AddAttributeValue(_T("use_attachments"), (long)m_attachmentMode);

   if (m_sensitivity != OP_ALL)
   if (m_sensitivity != OP_ALL)

-    clause->AddAttributeValue("sensitivity", (long)m_sensitivity);
+    clause->AddAttributeValue(_T("sensitivity"), (long)m_sensitivity);

   if (!m_spaceAttachments)
   if (!m_spaceAttachments)

-    clause->AddAttributeValue("space_attachments", (long)m_spaceAttachments);
+    clause->AddAttributeValue(_T("space_attachments"), (long)m_spaceAttachments);

   if (m_fixedWidth)
   if (m_fixedWidth)

-    clause->AddAttributeValue("fixed_width", (long)m_fixedWidth);
+    clause->AddAttributeValue(_T("fixed_width"), (long)m_fixedWidth);

   if (m_fixedHeight)
   if (m_fixedHeight)

-    clause->AddAttributeValue("fixed_height", (long)m_fixedHeight);
+    clause->AddAttributeValue(_T("fixed_height"), (long)m_fixedHeight);

   if (m_shadowMode != SHADOW_NONE)
   if (m_shadowMode != SHADOW_NONE)

-    clause->AddAttributeValue("shadow_mode", (long)m_shadowMode);
-  if (m_centreResize != TRUE)
-    clause->AddAttributeValue("centre_resize", (long)0);
-  clause->AddAttributeValue("maintain_aspect_ratio", (long) m_maintainAspectRatio);
-  if (m_highlighted != FALSE)
-    clause->AddAttributeValue("hilite", (long)m_highlighted);
+    clause->AddAttributeValue(_T("shadow_mode"), (long)m_shadowMode);
+  if (m_centreResize != true)
+    clause->AddAttributeValue(_T("centre_resize"), (long)0);
+  clause->AddAttributeValue(_T("maintain_aspect_ratio"), (long) m_maintainAspectRatio);
+  if (m_highlighted != false)
+    clause->AddAttributeValue(_T("hilite"), (long)m_highlighted);

 
   if (m_parent) // For composite objects
 
   if (m_parent) // For composite objects

-    clause->AddAttributeValue("parent", (long)m_parent->GetId());
+    clause->AddAttributeValue(_T("parent"), (long)m_parent->GetId());

 
   if (m_rotation != 0.0)
 
   if (m_rotation != 0.0)

-    clause->AddAttributeValue("rotation", m_rotation);
+    clause->AddAttributeValue(_T("rotation"), m_rotation);

 
   if (!this->IsKindOf(CLASSINFO(wxLineShape)))
   {
 
   if (!this->IsKindOf(CLASSINFO(wxLineShape)))
   {

-    clause->AddAttributeValue("neck_length", (long) m_branchNeckLength);
-    clause->AddAttributeValue("stem_length", (long) m_branchStemLength);
-    clause->AddAttributeValue("branch_spacing", (long) m_branchSpacing);
-    clause->AddAttributeValue("branch_style", (long) m_branchStyle);
+    clause->AddAttributeValue(_T("neck_length"), (long) m_branchNeckLength);
+    clause->AddAttributeValue(_T("stem_length"), (long) m_branchStemLength);
+    clause->AddAttributeValue(_T("branch_spacing"), (long) m_branchSpacing);
+    clause->AddAttributeValue(_T("branch_style"), (long) m_branchStyle);

   }
 
   // Write user-defined attachment points, if any
   }
 
   // Write user-defined attachment points, if any

-  if (m_attachmentPoints.Number() > 0)
+  if (m_attachmentPoints.GetCount() > 0)

   {
     wxExpr *attachmentList = new wxExpr(wxExprList);
   {
     wxExpr *attachmentList = new wxExpr(wxExprList);

-    wxNode *node = m_attachmentPoints.First();
+    wxNode *node = m_attachmentPoints.GetFirst();

     while (node)
     {
     while (node)
     {

-      wxAttachmentPoint *point = (wxAttachmentPoint *)node->Data();
+      wxAttachmentPoint *point = (wxAttachmentPoint *)node->GetData();

       wxExpr *pointExpr = new wxExpr(wxExprList);
       pointExpr->Append(new wxExpr((long)point->m_id));
       pointExpr->Append(new wxExpr(point->m_x));
       pointExpr->Append(new wxExpr(point->m_y));
       attachmentList->Append(pointExpr);
       wxExpr *pointExpr = new wxExpr(wxExprList);
       pointExpr->Append(new wxExpr((long)point->m_id));
       pointExpr->Append(new wxExpr(point->m_x));
       pointExpr->Append(new wxExpr(point->m_y));
       attachmentList->Append(pointExpr);

-      node = node->Next();
+      node = node->GetNext();

     }
     }

-    clause->AddAttributeValue("user_attachments", attachmentList);
+    clause->AddAttributeValue(_T("user_attachments"), attachmentList);

   }
 
   // Write text regions
   }
 
   // Write text regions


@@ -1822,14 +1805,14 @@ void wxShape::WriteRegions(wxExpr *clause)
   // Output regions as region1 = (...), region2 = (...), etc
   // and formatted text as text1 = (...), text2 = (...) etc.
   int regionNo = 1;
   // Output regions as region1 = (...), region2 = (...), etc
   // and formatted text as text1 = (...), text2 = (...) etc.
   int regionNo = 1;

-  char regionNameBuf[20];
-  char textNameBuf[20];
-  wxNode *node = m_regions.First();
+  wxChar regionNameBuf[20];
+  wxChar textNameBuf[20];
+  wxNode *node = m_regions.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShapeRegion *region = (wxShapeRegion *)node->Data();
-    sprintf(regionNameBuf, "region%d", regionNo);
-    sprintf(textNameBuf, "text%d", regionNo);
+    wxShapeRegion *region = (wxShapeRegion *)node->GetData();
+    wxSprintf(regionNameBuf, _T("region%d"), regionNo);
+    wxSprintf(textNameBuf, _T("text%d"), regionNo);

 
     // Original text and region attributes:
     // region1 = (regionName regionText x y width height minWidth minHeight proportionX proportionY
 
     // Original text and region attributes:
     // region1 = (regionName regionText x y width height minWidth minHeight proportionX proportionY


@@ -1864,55 +1847,55 @@ void wxShape::WriteRegions(wxExpr *clause)
     // text1 = ((x y string) (x y string) ...)
     wxExpr *textExpr = new wxExpr(wxExprList);
 
     // text1 = ((x y string) (x y string) ...)
     wxExpr *textExpr = new wxExpr(wxExprList);
 

-    wxNode *textNode = region->m_formattedText.First();
+    wxNode *textNode = region->m_formattedText.GetFirst();

     while (textNode)
     {
     while (textNode)
     {

-      wxShapeTextLine *line = (wxShapeTextLine *)textNode->Data();
+      wxShapeTextLine *line = (wxShapeTextLine *)textNode->GetData();

       wxExpr *list2 = new wxExpr(wxExprList);
       list2->Append(new wxExpr(line->GetX()));
       list2->Append(new wxExpr(line->GetY()));
       list2->Append(new wxExpr(wxExprString, line->GetText()));
       textExpr->Append(list2);
       wxExpr *list2 = new wxExpr(wxExprList);
       list2->Append(new wxExpr(line->GetX()));
       list2->Append(new wxExpr(line->GetY()));
       list2->Append(new wxExpr(wxExprString, line->GetText()));
       textExpr->Append(list2);

-      textNode = textNode->Next();
+      textNode = textNode->GetNext();

     }
 
     // Now add both attributes to the clause
     clause->AddAttributeValue(regionNameBuf, regionExpr);
     clause->AddAttributeValue(textNameBuf, textExpr);
 
     }
 
     // Now add both attributes to the clause
     clause->AddAttributeValue(regionNameBuf, regionExpr);
     clause->AddAttributeValue(textNameBuf, textExpr);
 

-    node = node->Next();
+    node = node->GetNext();

     regionNo ++;
   }
 }
 
 void wxShape::ReadAttributes(wxExpr *clause)
 {
     regionNo ++;
   }
 }
 
 void wxShape::ReadAttributes(wxExpr *clause)
 {

-  clause->GetAttributeValue("id", m_id);
+  clause->GetAttributeValue(_T("id"), m_id);

   wxRegisterId(m_id);
 
   wxRegisterId(m_id);
 

-  clause->GetAttributeValue("x", m_xpos);
-  clause->GetAttributeValue("y", m_ypos);
+  clause->GetAttributeValue(_T("x"), m_xpos);
+  clause->GetAttributeValue(_T("y"), m_ypos);

 
   // Input text strings (FOR COMPATIBILITY WITH OLD FILES ONLY. SEE REGION CODE BELOW.)
   ClearText();
 
   // Input text strings (FOR COMPATIBILITY WITH OLD FILES ONLY. SEE REGION CODE BELOW.)
   ClearText();

-  wxExpr *strings = clause->AttributeValue("text");
+  wxExpr *strings = clause->AttributeValue(_T("text"));

   if (strings && strings->Type() == wxExprList)
   {
   if (strings && strings->Type() == wxExprList)
   {

-    m_formatted = TRUE;  // Assume text is formatted unless we prove otherwise
+    m_formatted = true;  // Assume text is formatted unless we prove otherwise

     wxExpr *node = strings->value.first;
     while (node)
     {
       wxExpr *string_expr = node;
       double the_x = 0.0;
       double the_y = 0.0;
     wxExpr *node = strings->value.first;
     while (node)
     {
       wxExpr *string_expr = node;
       double the_x = 0.0;
       double the_y = 0.0;

-      wxString the_string("");
+      wxString the_string = wxEmptyString;

 
       // string_expr can either be a string, or a list of
       // 3 elements: x, y, and string.
       if (string_expr->Type() == wxExprString)
       {
         the_string = string_expr->StringValue();
 
       // string_expr can either be a string, or a list of
       // 3 elements: x, y, and string.
       if (string_expr->Type() == wxExprString)
       {
         the_string = string_expr->StringValue();

-        m_formatted = FALSE;
+        m_formatted = false;

       }
       else if (string_expr->Type() == wxExprList)
       {
       }
       else if (string_expr->Type() == wxExprList)
       {


@@ -1937,85 +1920,85 @@ void wxShape::ReadAttributes(wxExpr *clause)
           }
       }
       wxShapeTextLine *line =
           }
       }
       wxShapeTextLine *line =

-            new wxShapeTextLine(the_x, the_y, (char*) (const char*) the_string);
+            new wxShapeTextLine(the_x, the_y, the_string);

       m_text.Append(line);
 
       node = node->next;
     }
   }
 
       m_text.Append(line);
 
       node = node->next;
     }
   }
 

-  wxString pen_string = "";
-  wxString brush_string = "";
+  wxString pen_string = wxEmptyString;
+  wxString brush_string = wxEmptyString;

   int pen_width = 1;
   int pen_style = wxSOLID;
   int brush_style = wxSOLID;
   m_attachmentMode = ATTACHMENT_MODE_NONE;
 
   int pen_width = 1;
   int pen_style = wxSOLID;
   int brush_style = wxSOLID;
   m_attachmentMode = ATTACHMENT_MODE_NONE;
 

-  clause->GetAttributeValue("pen_colour", pen_string);
-  clause->GetAttributeValue("text_colour", m_textColourName);
+  clause->GetAttributeValue(_T("pen_colour"), pen_string);
+  clause->GetAttributeValue(_T("text_colour"), m_textColourName);

 
   SetTextColour(m_textColourName);
 
 
   SetTextColour(m_textColourName);
 

-  clause->GetAttributeValue("region_name", m_regionName);
+  clause->GetAttributeValue(_T("region_name"), m_regionName);

 
 

-  clause->GetAttributeValue("brush_colour", brush_string);
-  clause->GetAttributeValue("pen_width", pen_width);
-  clause->GetAttributeValue("pen_style", pen_style);
-  clause->GetAttributeValue("brush_style", brush_style);
+  clause->GetAttributeValue(_T("brush_colour"), brush_string);
+  clause->GetAttributeValue(_T("pen_width"), pen_width);
+  clause->GetAttributeValue(_T("pen_style"), pen_style);
+  clause->GetAttributeValue(_T("brush_style"), brush_style);

 
   int iVal = (int) m_attachmentMode;
 
   int iVal = (int) m_attachmentMode;

-  clause->GetAttributeValue("use_attachments", iVal);
+  clause->GetAttributeValue(_T("use_attachments"), iVal);

   m_attachmentMode = iVal;
 
   m_attachmentMode = iVal;
 

-  clause->GetAttributeValue("sensitivity", m_sensitivity);
+  clause->GetAttributeValue(_T("sensitivity"), m_sensitivity);

 
   iVal = (int) m_spaceAttachments;
 
   iVal = (int) m_spaceAttachments;

-  clause->GetAttributeValue("space_attachments", iVal);
+  clause->GetAttributeValue(_T("space_attachments"), iVal);

   m_spaceAttachments = (iVal != 0);
 
   iVal = (int) m_fixedWidth;
   m_spaceAttachments = (iVal != 0);
 
   iVal = (int) m_fixedWidth;

-  clause->GetAttributeValue("fixed_width", iVal);
+  clause->GetAttributeValue(_T("fixed_width"), iVal);

   m_fixedWidth = (iVal != 0);
 
   iVal = (int) m_fixedHeight;
   m_fixedWidth = (iVal != 0);
 
   iVal = (int) m_fixedHeight;

-  clause->GetAttributeValue("fixed_height", iVal);
+  clause->GetAttributeValue(_T("fixed_height"), iVal);

   m_fixedHeight = (iVal != 0);
 
   m_fixedHeight = (iVal != 0);
 

-  clause->GetAttributeValue("format_mode", m_formatMode);
-  clause->GetAttributeValue("shadow_mode", m_shadowMode);
+  clause->GetAttributeValue(_T("format_mode"), m_formatMode);
+  clause->GetAttributeValue(_T("shadow_mode"), m_shadowMode);

 
   iVal = m_branchNeckLength;
 
   iVal = m_branchNeckLength;

-  clause->GetAttributeValue("neck_length", iVal);
+  clause->GetAttributeValue(_T("neck_length"), iVal);

   m_branchNeckLength = iVal;
 
   iVal = m_branchStemLength;
   m_branchNeckLength = iVal;
 
   iVal = m_branchStemLength;

-  clause->GetAttributeValue("stem_length", iVal);
+  clause->GetAttributeValue(_T("stem_length"), iVal);

   m_branchStemLength = iVal;
 
   iVal = m_branchSpacing;
   m_branchStemLength = iVal;
 
   iVal = m_branchSpacing;

-  clause->GetAttributeValue("branch_spacing", iVal);
+  clause->GetAttributeValue(_T("branch_spacing"), iVal);

   m_branchSpacing = iVal;
 
   m_branchSpacing = iVal;
 

-  clause->GetAttributeValue("branch_style", m_branchStyle);
+  clause->GetAttributeValue(_T("branch_style"), m_branchStyle);

 
   iVal = (int) m_centreResize;
 
   iVal = (int) m_centreResize;

-  clause->GetAttributeValue("centre_resize", iVal);
+  clause->GetAttributeValue(_T("centre_resize"), iVal);

   m_centreResize = (iVal != 0);
 
   iVal = (int) m_maintainAspectRatio;
   m_centreResize = (iVal != 0);
 
   iVal = (int) m_maintainAspectRatio;

-  clause->GetAttributeValue("maintain_aspect_ratio", iVal);
+  clause->GetAttributeValue(_T("maintain_aspect_ratio"), iVal);

   m_maintainAspectRatio = (iVal != 0);
 
   iVal = (int) m_highlighted;
   m_maintainAspectRatio = (iVal != 0);
 
   iVal = (int) m_highlighted;

-  clause->GetAttributeValue("hilite", iVal);
+  clause->GetAttributeValue(_T("hilite"), iVal);

   m_highlighted = (iVal != 0);
 
   m_highlighted = (iVal != 0);
 

-  clause->GetAttributeValue("rotation", m_rotation);
+  clause->GetAttributeValue(_T("rotation"), m_rotation);

 
 

-  if (pen_string == "")
-    pen_string = "BLACK";
-  if (brush_string == "")
-    brush_string = "WHITE";
+  if (pen_string == wxEmptyString)
+    pen_string = _T("BLACK");
+  if (brush_string == wxEmptyString)
+    brush_string = _T("WHITE");

 
   if (pen_string.GetChar(0) == '#')
   {
 
   if (pen_string.GetChar(0) == '#')
   {


@@ -2040,11 +2023,11 @@ void wxShape::ReadAttributes(wxExpr *clause)
     m_brush = wxWHITE_BRUSH;
 
   int point_size = 10;
     m_brush = wxWHITE_BRUSH;
 
   int point_size = 10;

-  clause->GetAttributeValue("point_size", point_size);
+  clause->GetAttributeValue(_T("point_size"), point_size);

   SetFont(oglMatchFont(point_size));
 
   // Read user-defined attachment points, if any
   SetFont(oglMatchFont(point_size));
 
   // Read user-defined attachment points, if any

-  wxExpr *attachmentList = clause->AttributeValue("user_attachments");
+  wxExpr *attachmentList = clause->AttributeValue(_T("user_attachments"));

   if (attachmentList)
   {
     wxExpr *pointExpr = attachmentList->GetFirst();
   if (attachmentList)
   {
     wxExpr *pointExpr = attachmentList->GetFirst();


@@ -2076,25 +2059,25 @@ void wxShape::ReadRegions(wxExpr *clause)
   // region1 = (regionName regionText x y width height minWidth minHeight proportionX proportionY
   //            formatMode fontSize fontFamily fontStyle fontWeight textColour)
   int regionNo = 1;
   // region1 = (regionName regionText x y width height minWidth minHeight proportionX proportionY
   //            formatMode fontSize fontFamily fontStyle fontWeight textColour)
   int regionNo = 1;

-  char regionNameBuf[20];
-  char textNameBuf[20];
+  wxChar regionNameBuf[20];
+  wxChar textNameBuf[20];

 
 

-  wxExpr *regionExpr = NULL;
+  wxExpr *regionExpr;

   wxExpr *textExpr = NULL;
   wxExpr *textExpr = NULL;

-  sprintf(regionNameBuf, "region%d", regionNo);
-  sprintf(textNameBuf, "text%d", regionNo);
+  wxSprintf(regionNameBuf, _T("region%d"), regionNo);
+  wxSprintf(textNameBuf, _T("text%d"), regionNo);

 
 

-  m_formatted = TRUE;  // Assume text is formatted unless we prove otherwise
+  m_formatted = true;  // Assume text is formatted unless we prove otherwise

 
 

-  while ((regionExpr = clause->AttributeValue(regionNameBuf)))
+  while ((regionExpr = clause->AttributeValue(regionNameBuf)) != NULL)

   {
     /*
      * Get the region information
      *
      */
 
   {
     /*
      * Get the region information
      *
      */
 

-    wxString regionName("");
-    wxString regionText("");
+    wxString regionName = wxEmptyString;
+    wxString regionText = wxEmptyString;

     double x = 0.0;
     double y = 0.0;
     double width = 0.0;
     double x = 0.0;
     double y = 0.0;
     double width = 0.0;


@@ -2108,8 +2091,8 @@ void wxShape::ReadRegions(wxExpr *clause)
     int fontFamily = wxSWISS;
     int fontStyle = wxNORMAL;
     int fontWeight = wxNORMAL;
     int fontFamily = wxSWISS;
     int fontStyle = wxNORMAL;
     int fontWeight = wxNORMAL;

-    wxString regionTextColour("");
-    wxString penColour("");
+    wxString regionTextColour = wxEmptyString;
+    wxString penColour = wxEmptyString;

     int penStyle = wxSOLID;
 
     if (regionExpr->Type() == wxExprList)
     int penStyle = wxSOLID;
 
     if (regionExpr->Type() == wxExprList)


@@ -2159,7 +2142,7 @@ void wxShape::ReadRegions(wxExpr *clause)
         regionTextColour = colourExpr->StringValue();
       }
       else
         regionTextColour = colourExpr->StringValue();
       }
       else

-        regionTextColour = "BLACK";
+        regionTextColour = _T("BLACK");

 
       if (penColourExpr)
         penColour = penColourExpr->StringValue();
 
       if (penColourExpr)
         penColour = penColourExpr->StringValue();


@@ -2176,7 +2159,7 @@ void wxShape::ReadRegions(wxExpr *clause)
     region->SetMinSize(minWidth, minHeight);
     region->SetFormatMode(formatMode);
     region->SetPenStyle(penStyle);
     region->SetMinSize(minWidth, minHeight);
     region->SetFormatMode(formatMode);
     region->SetPenStyle(penStyle);

-    if (penColour != "")
+    if (penColour != wxEmptyString)

       region->SetPenColour(penColour);
 
     region->m_textColour = regionTextColour;
       region->SetPenColour(penColour);
 
     region->m_textColour = regionTextColour;


@@ -2198,14 +2181,14 @@ void wxShape::ReadRegions(wxExpr *clause)
         wxExpr *string_expr = node;
         double the_x = 0.0;
         double the_y = 0.0;
         wxExpr *string_expr = node;
         double the_x = 0.0;
         double the_y = 0.0;

-        wxString the_string("");
+        wxString the_string = wxEmptyString;

 
         // string_expr can either be a string, or a list of
         // 3 elements: x, y, and string.
         if (string_expr->Type() == wxExprString)
         {
           the_string = string_expr->StringValue();
 
         // string_expr can either be a string, or a list of
         // 3 elements: x, y, and string.
         if (string_expr->Type() == wxExprString)
         {
           the_string = string_expr->StringValue();

-          m_formatted = FALSE;
+          m_formatted = false;

         }
         else if (string_expr->Type() == wxExprList)
         {
         }
         else if (string_expr->Type() == wxExprList)
         {


@@ -2232,7 +2215,7 @@ void wxShape::ReadRegions(wxExpr *clause)
         if (the_string)
         {
           wxShapeTextLine *line =
         if (the_string)
         {
           wxShapeTextLine *line =

-              new wxShapeTextLine(the_x, the_y, (char*) (const char*) the_string);
+              new wxShapeTextLine(the_x, the_y, the_string);

           region->m_formattedText.Append(line);
         }
         node = node->next;
           region->m_formattedText.Append(line);
         }
         node = node->next;


@@ -2240,28 +2223,28 @@ void wxShape::ReadRegions(wxExpr *clause)
     }
 
     regionNo ++;
     }
 
     regionNo ++;

-    sprintf(regionNameBuf, "region%d", regionNo);
-    sprintf(textNameBuf, "text%d", regionNo);
+    wxSprintf(regionNameBuf, _T("region%d"), regionNo);
+    wxSprintf(textNameBuf, _T("text%d"), regionNo);

   }
 
   // Compatibility: check for no regions (old file).
   // Lines and divided rectangles must deal with this compatibility
   // theirselves. Composites _may_ not have any regions anyway.
   }
 
   // Compatibility: check for no regions (old file).
   // Lines and divided rectangles must deal with this compatibility
   // theirselves. Composites _may_ not have any regions anyway.

-  if ((m_regions.Number() == 0) &&
+  if ((m_regions.GetCount() == 0) &&

       !this->IsKindOf(CLASSINFO(wxLineShape)) && !this->IsKindOf(CLASSINFO(wxDividedShape)) &&
       !this->IsKindOf(CLASSINFO(wxCompositeShape)))
   {
     wxShapeRegion *newRegion = new wxShapeRegion;
       !this->IsKindOf(CLASSINFO(wxLineShape)) && !this->IsKindOf(CLASSINFO(wxDividedShape)) &&
       !this->IsKindOf(CLASSINFO(wxCompositeShape)))
   {
     wxShapeRegion *newRegion = new wxShapeRegion;

-    newRegion->SetName("0");
+    newRegion->SetName(_T("0"));

     m_regions.Append((wxObject *)newRegion);
     m_regions.Append((wxObject *)newRegion);

-    if (m_text.Number() > 0)
+    if (m_text.GetCount() > 0)

     {
       newRegion->ClearText();
     {
       newRegion->ClearText();

-      wxNode *node = m_text.First();
+      wxNode *node = m_text.GetFirst();

       while (node)
       {
       while (node)
       {

-        wxShapeTextLine *textLine = (wxShapeTextLine *)node->Data();
-        wxNode *next = node->Next();
+        wxShapeTextLine *textLine = (wxShapeTextLine *)node->GetData();
+        wxNode *next = node->GetNext();

         newRegion->GetFormattedText().Append((wxObject *)textLine);
         delete node;
         node = next;
         newRegion->GetFormattedText().Append((wxObject *)textLine);
         delete node;
         node = next;


@@ -2308,37 +2291,37 @@ void wxShape::Copy(wxShape& copy)
 
   // Copy text regions
   copy.ClearRegions();
 
   // Copy text regions
   copy.ClearRegions();

-  wxNode *node = m_regions.First();
+  wxNode *node = m_regions.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShapeRegion *region = (wxShapeRegion *)node->Data();
+    wxShapeRegion *region = (wxShapeRegion *)node->GetData();

     wxShapeRegion *newRegion = new wxShapeRegion(*region);
     copy.m_regions.Append(newRegion);
     wxShapeRegion *newRegion = new wxShapeRegion(*region);
     copy.m_regions.Append(newRegion);

-    node = node->Next();
+    node = node->GetNext();

   }
 
   // Copy attachments
   copy.ClearAttachments();
   }
 
   // Copy attachments
   copy.ClearAttachments();

-  node = m_attachmentPoints.First();
+  node = m_attachmentPoints.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxAttachmentPoint *point = (wxAttachmentPoint *)node->Data();
+    wxAttachmentPoint *point = (wxAttachmentPoint *)node->GetData();

     wxAttachmentPoint *newPoint = new wxAttachmentPoint;
     newPoint->m_id = point->m_id;
     newPoint->m_x = point->m_x;
     newPoint->m_y = point->m_y;
     copy.m_attachmentPoints.Append((wxObject *)newPoint);
     wxAttachmentPoint *newPoint = new wxAttachmentPoint;
     newPoint->m_id = point->m_id;
     newPoint->m_x = point->m_x;
     newPoint->m_y = point->m_y;
     copy.m_attachmentPoints.Append((wxObject *)newPoint);

-    node = node->Next();
+    node = node->GetNext();

   }
 
   // Copy lines
   copy.m_lines.Clear();
   }
 
   // Copy lines
   copy.m_lines.Clear();

-  node = m_lines.First();
+  node = m_lines.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxLineShape* line = (wxLineShape*) node->Data();
+    wxLineShape* line = (wxLineShape*) node->GetData();

     copy.m_lines.Append(line);
     copy.m_lines.Append(line);

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
   }
 }
 


@@ -2446,23 +2429,23 @@ void wxShape::MakeControlPoints()
 
 void wxShape::MakeMandatoryControlPoints()
 {
 
 void wxShape::MakeMandatoryControlPoints()
 {

-  wxNode *node = m_children.First();
+  wxNode *node = m_children.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShape *child = (wxShape *)node->Data();
+    wxShape *child = (wxShape *)node->GetData();

     child->MakeMandatoryControlPoints();
     child->MakeMandatoryControlPoints();

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
 void wxShape::ResetMandatoryControlPoints()
 {
   }
 }
 
 void wxShape::ResetMandatoryControlPoints()
 {

-  wxNode *node = m_children.First();
+  wxNode *node = m_children.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShape *child = (wxShape *)node->Data();
+    wxShape *child = (wxShape *)node->GetData();

     child->ResetMandatoryControlPoints();
     child->ResetMandatoryControlPoints();

-    node = node->Next();
+    node = node->GetNext();

   }
 }
 
   }
 }
 


@@ -2470,7 +2453,7 @@ void wxShape::ResetControlPoints()
 {
   ResetMandatoryControlPoints();
 
 {
   ResetMandatoryControlPoints();
 

-  if (m_controlPoints.Number() < 1)
+  if (m_controlPoints.GetCount() < 1)

     return;
 
   double maxX, maxY, minX, minY;
     return;
 
   double maxX, maxY, minX, minY;


@@ -2487,55 +2470,55 @@ void wxShape::ResetControlPoints()
   double left = (double)(- (widthMin / 2.0));
   double right = (double)(widthMin / 2.0 + (maxX - minX));
 
   double left = (double)(- (widthMin / 2.0));
   double right = (double)(widthMin / 2.0 + (maxX - minX));
 

-  wxNode *node = m_controlPoints.First();
-  wxControlPoint *control = (wxControlPoint *)node->Data();
+  wxNode *node = m_controlPoints.GetFirst();
+  wxControlPoint *control = (wxControlPoint *)node->GetData();

   control->m_xoffset = left; control->m_yoffset = top;
 
   control->m_xoffset = left; control->m_yoffset = top;
 

-  node = node->Next(); control = (wxControlPoint *)node->Data();
+  node = node->GetNext(); control = (wxControlPoint *)node->GetData();

   control->m_xoffset = 0; control->m_yoffset = top;
 
   control->m_xoffset = 0; control->m_yoffset = top;
 

-  node = node->Next(); control = (wxControlPoint *)node->Data();
+  node = node->GetNext(); control = (wxControlPoint *)node->GetData();

   control->m_xoffset = right; control->m_yoffset = top;
 
   control->m_xoffset = right; control->m_yoffset = top;
 

-  node = node->Next(); control = (wxControlPoint *)node->Data();
+  node = node->GetNext(); control = (wxControlPoint *)node->GetData();

   control->m_xoffset = right; control->m_yoffset = 0;
 
   control->m_xoffset = right; control->m_yoffset = 0;
 

-  node = node->Next(); control = (wxControlPoint *)node->Data();
+  node = node->GetNext(); control = (wxControlPoint *)node->GetData();

   control->m_xoffset = right; control->m_yoffset = bottom;
 
   control->m_xoffset = right; control->m_yoffset = bottom;
 

-  node = node->Next(); control = (wxControlPoint *)node->Data();
+  node = node->GetNext(); control = (wxControlPoint *)node->GetData();

   control->m_xoffset = 0; control->m_yoffset = bottom;
 
   control->m_xoffset = 0; control->m_yoffset = bottom;
 

-  node = node->Next(); control = (wxControlPoint *)node->Data();
+  node = node->GetNext(); control = (wxControlPoint *)node->GetData();

   control->m_xoffset = left; control->m_yoffset = bottom;
 
   control->m_xoffset = left; control->m_yoffset = bottom;
 

-  node = node->Next(); control = (wxControlPoint *)node->Data();
+  node = node->GetNext(); control = (wxControlPoint *)node->GetData();

   control->m_xoffset = left; control->m_yoffset = 0;
 }
 
 void wxShape::DeleteControlPoints(wxDC *dc)
 {
   control->m_xoffset = left; control->m_yoffset = 0;
 }
 
 void wxShape::DeleteControlPoints(wxDC *dc)
 {

-  wxNode *node = m_controlPoints.First();
+  wxNode *node = m_controlPoints.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxControlPoint *control = (wxControlPoint *)node->Data();
+    wxControlPoint *control = (wxControlPoint *)node->GetData();

     if (dc)
         control->GetEventHandler()->OnErase(*dc);
     m_canvas->RemoveShape(control);
     delete control;
     delete node;
     if (dc)
         control->GetEventHandler()->OnErase(*dc);
     m_canvas->RemoveShape(control);
     delete control;
     delete node;

-    node = m_controlPoints.First();
+    node = m_controlPoints.GetFirst();

   }
   // Children of divisions are contained objects,
   // so stop here
   if (!IsKindOf(CLASSINFO(wxDivisionShape)))
   {
   }
   // Children of divisions are contained objects,
   // so stop here
   if (!IsKindOf(CLASSINFO(wxDivisionShape)))
   {

-    node = m_children.First();
+    node = m_children.GetFirst();

     while (node)
     {
     while (node)
     {

-      wxShape *child = (wxShape *)node->Data();
+      wxShape *child = (wxShape *)node->GetData();

       child->DeleteControlPoints(dc);
       child->DeleteControlPoints(dc);

-      node = node->Next();
+      node = node->GetNext();

     }
   }
 }
     }
   }
 }


@@ -2548,12 +2531,12 @@ void wxShape::OnDrawControlPoints(wxDC& dc)
   dc.SetBrush(* wxBLACK_BRUSH);
   dc.SetPen(* wxBLACK_PEN);
 
   dc.SetBrush(* wxBLACK_BRUSH);
   dc.SetPen(* wxBLACK_PEN);
 

-  wxNode *node = m_controlPoints.First();
+  wxNode *node = m_controlPoints.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxControlPoint *control = (wxControlPoint *)node->Data();
+    wxControlPoint *control = (wxControlPoint *)node->GetData();

     control->Draw(dc);
     control->Draw(dc);

-    node = node->Next();
+    node = node->GetNext();

   }
   // Children of divisions are contained objects,
   // so stop here.
   }
   // Children of divisions are contained objects,
   // so stop here.


@@ -2561,33 +2544,33 @@ void wxShape::OnDrawControlPoints(wxDC& dc)
   // (critical when drawing)
   if (!IsKindOf(CLASSINFO(wxDivisionShape)))
   {
   // (critical when drawing)
   if (!IsKindOf(CLASSINFO(wxDivisionShape)))
   {

-    node = m_children.First();
+    node = m_children.GetFirst();

     while (node)
     {
     while (node)
     {

-      wxShape *child = (wxShape *)node->Data();
+      wxShape *child = (wxShape *)node->GetData();

       child->GetEventHandler()->OnDrawControlPoints(dc);
       child->GetEventHandler()->OnDrawControlPoints(dc);

-      node = node->Next();
+      node = node->GetNext();

     }
   }
 }
 
 void wxShape::OnEraseControlPoints(wxDC& dc)
 {
     }
   }
 }
 
 void wxShape::OnEraseControlPoints(wxDC& dc)
 {

-  wxNode *node = m_controlPoints.First();
+  wxNode *node = m_controlPoints.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxControlPoint *control = (wxControlPoint *)node->Data();
+    wxControlPoint *control = (wxControlPoint *)node->GetData();

     control->Erase(dc);
     control->Erase(dc);

-    node = node->Next();
+    node = node->GetNext();

   }
   if (!IsKindOf(CLASSINFO(wxDivisionShape)))
   {
   }
   if (!IsKindOf(CLASSINFO(wxDivisionShape)))
   {

-    node = m_children.First();
+    node = m_children.GetFirst();

     while (node)
     {
     while (node)
     {

-      wxShape *child = (wxShape *)node->Data();
+      wxShape *child = (wxShape *)node->GetData();

       child->GetEventHandler()->OnEraseControlPoints(dc);
       child->GetEventHandler()->OnEraseControlPoints(dc);

-      node = node->Next();
+      node = node->GetNext();

     }
   }
 }
     }
   }
 }


@@ -2602,12 +2585,12 @@ void wxShape::Select(bool select, wxDC* dc)
     // so stop here
     if (!IsKindOf(CLASSINFO(wxDivisionShape)))
     {
     // so stop here
     if (!IsKindOf(CLASSINFO(wxDivisionShape)))
     {

-      wxNode *node = m_children.First();
+      wxNode *node = m_children.GetFirst();

       while (node)
       {
       while (node)
       {

-        wxShape *child = (wxShape *)node->Data();
+        wxShape *child = (wxShape *)node->GetData();

         child->MakeMandatoryControlPoints();
         child->MakeMandatoryControlPoints();

-        node = node->Next();
+        node = node->GetNext();

       }
     }
     if (dc)
       }
     }
     if (dc)


@@ -2618,12 +2601,12 @@ void wxShape::Select(bool select, wxDC* dc)
     DeleteControlPoints(dc);
     if (!IsKindOf(CLASSINFO(wxDivisionShape)))
     {
     DeleteControlPoints(dc);
     if (!IsKindOf(CLASSINFO(wxDivisionShape)))
     {

-      wxNode *node = m_children.First();
+      wxNode *node = m_children.GetFirst();

       while (node)
       {
       while (node)
       {

-        wxShape *child = (wxShape *)node->Data();
+        wxShape *child = (wxShape *)node->GetData();

         child->DeleteControlPoints(dc);
         child->DeleteControlPoints(dc);

-        node = node->Next();
+        node = node->GetNext();

       }
     }
   }
       }
     }
   }


@@ -2636,9 +2619,9 @@ bool wxShape::Selected() const
 
 bool wxShape::AncestorSelected() const
 {
 
 bool wxShape::AncestorSelected() const
 {

-  if (m_selected) return TRUE;
+  if (m_selected) return true;

   if (!GetParent())
   if (!GetParent())

-    return FALSE;
+    return false;

   else
     return GetParent()->AncestorSelected();
 }
   else
     return GetParent()->AncestorSelected();
 }


@@ -2648,18 +2631,18 @@ int wxShape::GetNumberOfAttachments() const
   // Should return the MAXIMUM attachment point id here,
   // so higher-level functions can iterate through all attachments,
   // even if they're not contiguous.
   // Should return the MAXIMUM attachment point id here,
   // so higher-level functions can iterate through all attachments,
   // even if they're not contiguous.

-  if (m_attachmentPoints.Number() == 0)
+  if (m_attachmentPoints.GetCount() == 0)

     return 4;
   else
   {
     int maxN = 3;
     return 4;
   else
   {
     int maxN = 3;

-    wxNode *node = m_attachmentPoints.First();
+    wxNode *node = m_attachmentPoints.GetFirst();

     while (node)
     {
     while (node)
     {

-      wxAttachmentPoint *point = (wxAttachmentPoint *)node->Data();
+      wxAttachmentPoint *point = (wxAttachmentPoint *)node->GetData();

       if (point->m_id > maxN)
         maxN = point->m_id;
       if (point->m_id > maxN)
         maxN = point->m_id;

-      node = node->Next();
+      node = node->GetNext();

     }
     return maxN+1;;
   }
     }
     return maxN+1;;
   }


@@ -2667,20 +2650,20 @@ int wxShape::GetNumberOfAttachments() const
 
 bool wxShape::AttachmentIsValid(int attachment) const
 {
 
 bool wxShape::AttachmentIsValid(int attachment) const
 {

-  if (m_attachmentPoints.Number() == 0)
+  if (m_attachmentPoints.GetCount() == 0)

   {
     return ((attachment >= 0) && (attachment < 4)) ;
   }
 
   {
     return ((attachment >= 0) && (attachment < 4)) ;
   }
 

-  wxNode *node = m_attachmentPoints.First();
+  wxNode *node = m_attachmentPoints.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxAttachmentPoint *point = (wxAttachmentPoint *)node->Data();
+    wxAttachmentPoint *point = (wxAttachmentPoint *)node->GetData();

     if (point->m_id == attachment)
     if (point->m_id == attachment)

-      return TRUE;
-    node = node->Next();
+      return true;
+    node = node->GetNext();

   }
   }

-  return FALSE;
+  return false;

 }
 
 bool wxShape::GetAttachmentPosition(int attachment, double *x, double *y,
 }
 
 bool wxShape::GetAttachmentPosition(int attachment, double *x, double *y,


@@ -2689,7 +2672,7 @@ bool wxShape::GetAttachmentPosition(int attachment, double *x, double *y,
     if (m_attachmentMode == ATTACHMENT_MODE_NONE)
     {
         *x = m_xpos; *y = m_ypos;
     if (m_attachmentMode == ATTACHMENT_MODE_NONE)
     {
         *x = m_xpos; *y = m_ypos;

-        return TRUE;
+        return true;

     }
     else if (m_attachmentMode == ATTACHMENT_MODE_BRANCHING)
     {
     }
     else if (m_attachmentMode == ATTACHMENT_MODE_BRANCHING)
     {


@@ -2697,26 +2680,26 @@ bool wxShape::GetAttachmentPosition(int attachment, double *x, double *y,
         GetBranchingAttachmentPoint(attachment, nth, pt, stemPt);
         *x = pt.x;
         *y = pt.y;
         GetBranchingAttachmentPoint(attachment, nth, pt, stemPt);
         *x = pt.x;
         *y = pt.y;

-        return TRUE;
+        return true;

     }
     else if (m_attachmentMode == ATTACHMENT_MODE_EDGE)
     {
     }
     else if (m_attachmentMode == ATTACHMENT_MODE_EDGE)
     {

-        if (m_attachmentPoints.Number() > 0)
+        if (m_attachmentPoints.GetCount() > 0)

         {
         {

-            wxNode *node = m_attachmentPoints.First();
+            wxNode *node = m_attachmentPoints.GetFirst();

             while (node)
             {
             while (node)
             {

-                wxAttachmentPoint *point = (wxAttachmentPoint *)node->Data();
+                wxAttachmentPoint *point = (wxAttachmentPoint *)node->GetData();

                 if (point->m_id == attachment)
                 {
                     *x = (double)(m_xpos + point->m_x);
                     *y = (double)(m_ypos + point->m_y);
                 if (point->m_id == attachment)
                 {
                     *x = (double)(m_xpos + point->m_x);
                     *y = (double)(m_ypos + point->m_y);

-                    return TRUE;
+                    return true;

                 }
                 }

-                node = node->Next();
+                node = node->GetNext();

             }
             *x = m_xpos; *y = m_ypos;
             }
             *x = m_xpos; *y = m_ypos;

-            return FALSE;
+            return false;

         }
         else
         {
         }
         else
         {


@@ -2728,7 +2711,9 @@ bool wxShape::GetAttachmentPosition(int attachment, double *x, double *y,
             double left = (double)(m_xpos - w/2.0);
             double right = (double)(m_xpos + w/2.0);
 
             double left = (double)(m_xpos - w/2.0);
             double right = (double)(m_xpos + w/2.0);
 

-            bool isEnd = (line && line->IsEnd(this));
+#if 0
+            /* bool isEnd = */ (line && line->IsEnd(this));
+#endif

 
             int physicalAttachment = LogicalToPhysicalAttachment(attachment);
 
 
             int physicalAttachment = LogicalToPhysicalAttachment(attachment);
 


@@ -2769,14 +2754,13 @@ bool wxShape::GetAttachmentPosition(int attachment, double *x, double *y,
                 }
                 default:
                 {
                 }
                 default:
                 {

-                    return FALSE;
-                    break;
+                    return false;

                 }
             }
                 }
             }

-            return TRUE;
+            return true;

         }
     }
         }
     }

-    return FALSE;
+    return false;

 }
 
 void wxShape::GetBoundingBoxMax(double *w, double *h)
 }
 
 void wxShape::GetBoundingBoxMax(double *w, double *h)


@@ -2792,33 +2776,33 @@ void wxShape::GetBoundingBoxMax(double *w, double *h)
   *h = hh;
 }
 
   *h = hh;
 }
 

-// Returns TRUE if image is a descendant of this composite
+// Returns true if image is a descendant of this composite

 bool wxShape::HasDescendant(wxShape *image)
 {
   if (image == this)
 bool wxShape::HasDescendant(wxShape *image)
 {
   if (image == this)

-    return TRUE;
-  wxNode *node = m_children.First();
+    return true;
+  wxNode *node = m_children.GetFirst();

   while (node)
   {
   while (node)
   {

-    wxShape *child = (wxShape *)node->Data();
+    wxShape *child = (wxShape *)node->GetData();

     bool ans = child->HasDescendant(image);
     if (ans)
     bool ans = child->HasDescendant(image);
     if (ans)

-      return TRUE;
-    node = node->Next();
+      return true;
+    node = node->GetNext();

   }
   }

-  return FALSE;
+  return false;

 }
 
 // Clears points from a list of wxRealPoints, and clears list
 void wxShape::ClearPointList(wxList& list)
 {
 }
 
 // Clears points from a list of wxRealPoints, and clears list
 void wxShape::ClearPointList(wxList& list)
 {

-    wxNode* node = list.First();
+    wxNode* node = list.GetFirst();

     while (node)
     {
     while (node)
     {

-        wxRealPoint* pt = (wxRealPoint*) node->Data();
+        wxRealPoint* pt = (wxRealPoint*) node->GetData();

         delete pt;
 
         delete pt;
 

-        node = node->Next();
+        node = node->GetNext();

     }
     list.Clear();
 }
     }
     list.Clear();
 }


@@ -2831,7 +2815,7 @@ wxRealPoint wxShape::CalcSimpleAttachment(const wxRealPoint& pt1, const wxRealPo
     bool isEnd = (line && line->IsEnd(this));
 
     // Are we horizontal or vertical?
     bool isEnd = (line && line->IsEnd(this));
 
     // Are we horizontal or vertical?

-    bool isHorizontal = (oglRoughlyEqual(pt1.y, pt2.y) == TRUE);
+    bool isHorizontal = (oglRoughlyEqual(pt1.y, pt2.y) == true);

 
     double x, y;
 
 
     double x, y;
 


@@ -2871,7 +2855,7 @@ wxRealPoint wxShape::CalcSimpleAttachment(const wxRealPoint& pt1, const wxRealPo
     }
     else
     {
     }
     else
     {

-        wxASSERT( oglRoughlyEqual(pt1.x, pt2.x) == TRUE );
+        wxASSERT( oglRoughlyEqual(pt1.x, pt2.x) == true );

 
         wxRealPoint firstPoint, secondPoint;
         if (pt1.y > pt2.y)
 
         wxRealPoint firstPoint, secondPoint;
         if (pt1.y > pt2.y)


@@ -2912,9 +2896,8 @@ wxRealPoint wxShape::CalcSimpleAttachment(const wxRealPoint& pt1, const wxRealPo
 // Return the zero-based position in m_lines of line.
 int wxShape::GetLinePosition(wxLineShape* line)
 {
 // Return the zero-based position in m_lines of line.
 int wxShape::GetLinePosition(wxLineShape* line)
 {

-    int i = 0;
-    for (i = 0; i < m_lines.Number(); i++)
-        if ((wxLineShape*) (m_lines.Nth(i)->Data()) == line)
+    for (size_t i = 0; i < m_lines.GetCount(); i++)
+        if ((wxLineShape*) (m_lines.Item(i)->GetData()) == line)

             return i;
 
     return 0;
             return i;
 
     return 0;


@@ -2929,7 +2912,7 @@ int wxShape::GetLinePosition(wxLineShape* line)
 //                      <- branching attachment point N-1
 
 // This function gets information about where branching connections go.
 //                      <- branching attachment point N-1
 
 // This function gets information about where branching connections go.

-// Returns FALSE if there are no lines at this attachment.
+// Returns false if there are no lines at this attachment.

 bool wxShape::GetBranchingAttachmentInfo(int attachment, wxRealPoint& root, wxRealPoint& neck,
     wxRealPoint& shoulder1, wxRealPoint& shoulder2)
 {
 bool wxShape::GetBranchingAttachmentInfo(int attachment, wxRealPoint& root, wxRealPoint& neck,
     wxRealPoint& shoulder1, wxRealPoint& shoulder2)
 {


@@ -2939,7 +2922,7 @@ bool wxShape::GetBranchingAttachmentInfo(int attachment, wxRealPoint& root, wxRe
     int lineCount = GetAttachmentLineCount(attachment);
 
     if (lineCount == 0)
     int lineCount = GetAttachmentLineCount(attachment);
 
     if (lineCount == 0)

-        return FALSE;
+        return false;

 
     int totalBranchLength = m_branchSpacing * (lineCount - 1);
 
 
     int totalBranchLength = m_branchSpacing * (lineCount - 1);
 


@@ -2998,11 +2981,11 @@ bool wxShape::GetBranchingAttachmentInfo(int attachment, wxRealPoint& root, wxRe
         }
         default:
         {
         }
         default:
         {

-            wxFAIL_MSG( "Unrecognised attachment point in GetBranchingAttachmentInfo." );
+            wxFAIL_MSG( wxT("Unrecognised attachment point in GetBranchingAttachmentInfo.") );

             break;
         }
     }
             break;
         }
     }

-    return TRUE;
+    return true;

 }
 
 // n is the number of the adjoining line, from 0 to N-1 where N is the number of lines
 }
 
 // n is the number of the adjoining line, from 0 to N-1 where N is the number of lines


@@ -3057,28 +3040,28 @@ bool wxShape::GetBranchingAttachmentPoint(int attachment, int n, wxRealPoint& pt
         }
         default:
         {
         }
         default:
         {

-            wxFAIL_MSG( "Unrecognised attachment point in GetBranchingAttachmentPoint." );
+            wxFAIL_MSG( wxT("Unrecognised attachment point in GetBranchingAttachmentPoint.") );

             break;
         }
     }
 
             break;
         }
     }
 

-    return TRUE;
+    return true;

 }
 
 // Get the number of lines at this attachment position.
 int wxShape::GetAttachmentLineCount(int attachment) const
 {
     int count = 0;
 }
 
 // Get the number of lines at this attachment position.
 int wxShape::GetAttachmentLineCount(int attachment) const
 {
     int count = 0;

-    wxNode* node = m_lines.First();
+    wxNode* node = m_lines.GetFirst();

     while (node)
     {
     while (node)
     {

-        wxLineShape* lineShape = (wxLineShape*) node->Data();
+        wxLineShape* lineShape = (wxLineShape*) node->GetData();

         if ((lineShape->GetFrom() == this) && (lineShape->GetAttachmentFrom() == attachment))
             count ++;
         else if ((lineShape->GetTo() == this) && (lineShape->GetAttachmentTo() == attachment))
             count ++;
 
         if ((lineShape->GetFrom() == this) && (lineShape->GetAttachmentFrom() == attachment))
             count ++;
         else if ((lineShape->GetTo() == this) && (lineShape->GetAttachmentTo() == attachment))
             count ++;
 

-        node = node->Next();
+        node = node->GetNext();

     }
     return count;
 }
     }
     return count;
 }


@@ -3122,7 +3105,7 @@ wxRealPoint wxShape::GetBranchingAttachmentRoot(int attachment)
         }
         default:
         {
         }
         default:
         {

-            wxFAIL_MSG( "Unrecognised attachment point in GetBranchingAttachmentRoot." );
+            wxFAIL_MSG( wxT("Unrecognised attachment point in GetBranchingAttachmentRoot.") );

             break;
         }
     }
             break;
         }
     }


@@ -3208,7 +3191,7 @@ bool wxShape::GetAttachmentPositionEdge(int attachment, double *x, double *y,
 // to logical (0 -> 1 if rotated by 90 degrees)
 int wxShape::PhysicalToLogicalAttachment(int physicalAttachment) const
 {
 // to logical (0 -> 1 if rotated by 90 degrees)
 int wxShape::PhysicalToLogicalAttachment(int physicalAttachment) const
 {

-    const double pi = 3.1415926535897932384626433832795 ;
+    const double pi = M_PI ;

     int i;
     if (oglRoughlyEqual(GetRotation(), 0.0))
     {
     int i;
     if (oglRoughlyEqual(GetRotation(), 0.0))
     {


@@ -3240,7 +3223,7 @@ int wxShape::PhysicalToLogicalAttachment(int physicalAttachment) const
 // to physical (0 is always North)
 int wxShape::LogicalToPhysicalAttachment(int logicalAttachment) const
 {
 // to physical (0 is always North)
 int wxShape::LogicalToPhysicalAttachment(int logicalAttachment) const
 {

-    const double pi = 3.1415926535897932384626433832795 ;
+    const double pi = M_PI ;

     int i;
     if (oglRoughlyEqual(GetRotation(), 0.0))
     {
     int i;
     if (oglRoughlyEqual(GetRotation(), 0.0))
     {


@@ -3270,7 +3253,7 @@ int wxShape::LogicalToPhysicalAttachment(int logicalAttachment) const
 
 void wxShape::Rotate(double WXUNUSED(x), double WXUNUSED(y), double theta)
 {
 
 void wxShape::Rotate(double WXUNUSED(x), double WXUNUSED(y), double theta)
 {

-    const double pi = 3.1415926535897932384626433832795 ;
+    const double pi = M_PI ;

     m_rotation = theta;
     if (m_rotation < 0.0)
     {
     m_rotation = theta;
     if (m_rotation < 0.0)
     {