From: Julian Smart <julian@anthemion.co.uk>
Date: Thu, 3 Dec 1998 09:51:18 +0000 (+0000)
Subject: Small Motif bug fixes
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/45d49251ba41915e3d8b89d13a770a74984b1055

Small Motif bug fixes


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@1101 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/docs/motif/changes.txt b/docs/motif/changes.txt
index b8494394f0..0508aaa92e 100644
--- a/docs/motif/changes.txt
+++ b/docs/motif/changes.txt
@@ -52,4 +52,12 @@ More recently:
 - Added time.cpp to makefile; set wxUSE_DATETIME to 1.
 - Added a parent-existance check to popup menu code to make it not crash.
 - Added some optimization in wxWindow::SetSize to produce less flicker.
-  It remains to be seen whether this produces any resize bugs.
\ No newline at end of file
+  It remains to be seen whether this produces any resize bugs.
+
+3/12/98
+-------
+
+- Debugged DrawEllipticArc (a ! in the wrong place).
+- Added SetClippingRegion( const wxRegion& region ).
+- Added wxPoint, wxSize, wxRect versions of SetSize etc.
+- Diagnosed but not yet cured a wxTreeCtrl bug (see todo.txt).
\ No newline at end of file
diff --git a/docs/motif/todo.txt b/docs/motif/todo.txt
index 1043abd94c..d73aa1bec9 100644
--- a/docs/motif/todo.txt
+++ b/docs/motif/todo.txt
@@ -42,6 +42,9 @@ High Priority
 - Tidy dialogs such as the colour and font selectors.
 
 - Use generic wxTreeCtrl, wxListCtrl: debug and enhance these.
+  wxTreeCtrl: crashes in wxImageList::Draw because it assumes
+  that wxBitmap == wxIcon, which is only true in wxGTK.
+  So add wxDC::DrawBitmap and use this instead.
 
 - Find out why modal dialogs give a grab warning.
 
diff --git a/include/wx/motif/region.h b/include/wx/motif/region.h
index ca68dc140a..969a44f654 100644
--- a/include/wx/motif/region.h
+++ b/include/wx/motif/region.h
@@ -103,7 +103,7 @@ public:
 	bool Combine(const wxRect& rect, wxRegionOp op);
 
     // Get the internal Region handle
-    WXRegion GetXRegion() ;
+    WXRegion GetXRegion() const;
 };
 
 class WXDLLEXPORT wxRegionIterator : public wxObject {
diff --git a/src/motif/dcclient.cpp b/src/motif/dcclient.cpp
index a91a4e7d57..808dbc3fd0 100644
--- a/src/motif/dcclient.cpp
+++ b/src/motif/dcclient.cpp
@@ -354,7 +354,7 @@ void wxWindowDC::DrawEllipticArc( long x, long y, long width, long height, doubl
           XLOG2DEV_2 (x), YLOG2DEV_2 (y),wd,hd,start,end);
     }
 
-  if (!m_pen.Ok() && m_pen.GetStyle () != wxTRANSPARENT)
+  if (m_pen.Ok() && m_pen.GetStyle () != wxTRANSPARENT)
     {
       if (m_autoSetting)
     SetPen (m_pen);
diff --git a/src/motif/region.cpp b/src/motif/region.cpp
index 7121fe63a3..d4ced6405d 100644
--- a/src/motif/region.cpp
+++ b/src/motif/region.cpp
@@ -105,7 +105,7 @@ wxRegion::~wxRegion()
 }
 
 // Get the internal region handle
-WXRegion wxRegion::GetXRegion()
+WXRegion wxRegion::GetXRegion() const
 {
     wxASSERT( m_refData !=NULL );