Removed ambiguity between deprecated and new constructor.

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

diff --git a/contrib/src/ogl/divided.cpp b/contrib/src/ogl/divided.cpp
index 50737984c032dc384c993b36a7ce56ef252bddbb..d4190100c84545e34987baa3f18ef531bd37cb21 100644 (file)
--- a/contrib/src/ogl/divided.cpp
+++ b/contrib/src/ogl/divided.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__


@@ -93,7 +93,7 @@ void wxDividedShape::OnDrawContents(wxDC& dc)
   if (!formatted)
   {
     FormatRegionText();
   if (!formatted)
   {
     FormatRegionText();

-    formatted = TRUE;
+    formatted = true;

   }
 */
   if (GetDisableLabel()) return;
   }
 */
   if (GetDisableLabel()) return;


@@ -102,7 +102,7 @@ void wxDividedShape::OnDrawContents(wxDC& dc)
   double yMargin = 2;
   dc.SetBackgroundMode(wxTRANSPARENT);
 
   double yMargin = 2;
   dc.SetBackgroundMode(wxTRANSPARENT);
 

-  wxNode *node = GetRegions().GetFirst();
+  wxObjectList::compatibility_iterator node = GetRegions().GetFirst();

   while (node)
   {
     wxShapeRegion *region = (wxShapeRegion *)node->GetData();
   while (node)
   {
     wxShapeRegion *region = (wxShapeRegion *)node->GetData();


@@ -157,7 +157,7 @@ void wxDividedShape::SetRegionSizes()
 //  double leftX = (double)(m_xpos - (m_width / 2.0));
 //  double rightX = (double)(m_xpos + (m_width / 2.0));
 
 //  double leftX = (double)(m_xpos - (m_width / 2.0));
 //  double rightX = (double)(m_xpos + (m_width / 2.0));
 

-  wxNode *node = GetRegions().GetFirst();
+  wxObjectList::compatibility_iterator node = GetRegions().GetFirst();

   while (node)
   {
     wxShapeRegion *region = (wxShapeRegion *)node->GetData();
   while (node)
   {
     wxShapeRegion *region = (wxShapeRegion *)node->GetData();


@@ -245,7 +245,7 @@ bool wxDividedShape::GetAttachmentPosition(int attachment, double *x, double *y,
   {
     bool isLeft = !(attachment < (n+1));
     int i = (isLeft) ? (totalNumberAttachments - attachment - 1) : (attachment-1);
   {
     bool isLeft = !(attachment < (n+1));
     int i = (isLeft) ? (totalNumberAttachments - attachment - 1) : (attachment-1);

-    wxNode *node = GetRegions().Item(i);
+    wxObjectList::compatibility_iterator node = GetRegions().Item(i);

     if (node)
     {
       wxShapeRegion *region = (wxShapeRegion *)node->GetData();
     if (node)
     {
       wxShapeRegion *region = (wxShapeRegion *)node->GetData();


@@ -285,10 +285,10 @@ bool wxDividedShape::GetAttachmentPosition(int attachment, double *x, double *y,
     {
       *x = m_xpos;
       *y = m_ypos;
     {
       *x = m_xpos;
       *y = m_ypos;

-      return FALSE;
+      return false;

     }
   }
     }
   }

-  return TRUE;
+  return true;

 }
 
 int wxDividedShape::GetNumberOfAttachments() const
 }
 
 int wxDividedShape::GetNumberOfAttachments() const


@@ -298,7 +298,7 @@ int wxDividedShape::GetNumberOfAttachments() const
   int n = (GetRegions().GetCount() * 2) + 2;
 
   int maxN = n - 1;
   int n = (GetRegions().GetCount() * 2) + 2;
 
   int maxN = n - 1;

-  wxNode *node = m_attachmentPoints.GetFirst();
+  wxObjectList::compatibility_iterator node = m_attachmentPoints.GetFirst();

   while (node)
   {
     wxAttachmentPoint *point = (wxAttachmentPoint *)node->GetData();
   while (node)
   {
     wxAttachmentPoint *point = (wxAttachmentPoint *)node->GetData();


@@ -317,9 +317,9 @@ bool wxDividedShape::AttachmentIsValid(int attachment) const
     return wxShape::AttachmentIsValid(attachment);
   }
   else if (attachment >= 0)
     return wxShape::AttachmentIsValid(attachment);
   }
   else if (attachment >= 0)

-    return TRUE;
+    return true;

   else
   else

-    return FALSE;
+    return false;

 }
 
 void wxDividedShape::Copy(wxShape& copy)
 }
 
 void wxDividedShape::Copy(wxShape& copy)


@@ -341,7 +341,7 @@ void wxDividedShape::MakeMandatoryControlPoints()
   double currentY = (double)(GetY() - (m_height / 2.0));
   double maxY = (double)(GetY() + (m_height / 2.0));
 
   double currentY = (double)(GetY() - (m_height / 2.0));
   double maxY = (double)(GetY() + (m_height / 2.0));
 

-  wxNode *node = GetRegions().GetFirst();
+  wxObjectList::compatibility_iterator node = GetRegions().GetFirst();

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


@@ -379,14 +379,14 @@ void wxDividedShape::ResetMandatoryControlPoints()
   double currentY = (double)(GetY() - (m_height / 2.0));
   double maxY = (double)(GetY() + (m_height / 2.0));
 
   double currentY = (double)(GetY() - (m_height / 2.0));
   double maxY = (double)(GetY() + (m_height / 2.0));
 

-  wxNode *node = m_controlPoints.GetFirst();
+  wxObjectList::compatibility_iterator node = m_controlPoints.GetFirst();

   int i = 0;
   while (node)
   {
     wxControlPoint *controlPoint = (wxControlPoint *)node->GetData();
     if (controlPoint->IsKindOf(CLASSINFO(wxDividedShapeControlPoint)))
     {
   int i = 0;
   while (node)
   {
     wxControlPoint *controlPoint = (wxControlPoint *)node->GetData();
     if (controlPoint->IsKindOf(CLASSINFO(wxDividedShapeControlPoint)))
     {

-      wxNode *node1 = GetRegions().Item(i);
+      wxObjectList::compatibility_iterator node1 = GetRegions().Item(i);

       wxShapeRegion *region = (wxShapeRegion *)node1->GetData();
 
       double proportion = region->m_regionProportionY;
       wxShapeRegion *region = (wxShapeRegion *)node1->GetData();
 
       double proportion = region->m_regionProportionY;


@@ -524,7 +524,7 @@ void wxDividedShape::EditRegions()
 
   wxEndBusyCursor();
 
 
   wxEndBusyCursor();
 

-  dialog->Show(TRUE);
+  dialog->Show(true);

 
   node = GetRegions().GetFirst();
   i = 0;
 
   node = GetRegions().GetFirst();
   i = 0;


@@ -585,7 +585,7 @@ void wxDividedShapeControlPoint::OnDragLeft(bool WXUNUSED(draw), double WXUNUSED
     GetCanvas()->PrepareDC(dc);
 
     dc.SetLogicalFunction(OGLRBLF);
     GetCanvas()->PrepareDC(dc);
 
     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));
 


@@ -604,7 +604,7 @@ void wxDividedShapeControlPoint::OnBeginDragLeft(double WXUNUSED(x), double y, i
 
     wxDividedShape *dividedObject = (wxDividedShape *)m_shape;
     dc.SetLogicalFunction(OGLRBLF);
 
     wxDividedShape *dividedObject = (wxDividedShape *)m_shape;
     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));
 


@@ -622,7 +622,7 @@ void wxDividedShapeControlPoint::OnEndDragLeft(double WXUNUSED(x), double y, int
     GetCanvas()->PrepareDC(dc);
 
     wxDividedShape *dividedObject = (wxDividedShape *)m_shape;
     GetCanvas()->PrepareDC(dc);
 
     wxDividedShape *dividedObject = (wxDividedShape *)m_shape;

-    wxNode *node = dividedObject->GetRegions().Item(regionId);
+    wxObjectList::compatibility_iterator node = dividedObject->GetRegions().Item(regionId);

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


@@ -642,10 +642,6 @@ void wxDividedShapeControlPoint::OnEndDragLeft(double WXUNUSED(x), double y, int
 
     // Save values
     double thisRegionTop = 0.0;
 
     // Save values
     double thisRegionTop = 0.0;

-    #if 0
-    // this variable is not readed later
-    double thisRegionBottom = 0.0;
-    #endif

     double nextRegionBottom = 0.0;
 
     node = dividedObject->GetRegions().GetFirst();
     double nextRegionBottom = 0.0;
 
     node = dividedObject->GetRegions().GetFirst();


@@ -660,10 +656,6 @@ void wxDividedShapeControlPoint::OnEndDragLeft(double WXUNUSED(x), double y, int
       if (region == thisRegion)
       {
         thisRegionTop = currentY;
       if (region == thisRegion)
       {
         thisRegionTop = currentY;

-        #if 0
-        // no need for assignment if value is not used later
-        thisRegionBottom = actualY;
-        #endif

         if (node->GetNext())
           nextRegion = (wxShapeRegion *)node->GetNext()->GetData();
       }
         if (node->GetNext())
           nextRegion = (wxShapeRegion *)node->GetNext()->GetData();
       }


@@ -700,9 +692,8 @@ void wxDividedShapeControlPoint::OnEndDragLeft(double WXUNUSED(x), double y, int
         wxShapeRegion *region = (wxShapeRegion *)node->GetData();
         if (region->GetText())
         {
         wxShapeRegion *region = (wxShapeRegion *)node->GetData();
         if (region->GetText())
         {

-        wxChar *s = copystring(region->GetText());
-        dividedObject->FormatText(dc, s, i);
-        delete[] s;
+        wxString s(region->GetText());
+        dividedObject->FormatText(dc, s.c_str(), i);

         }
         node = node->GetNext();
         i++;
         }
         node = node->GetNext();
         i++;