fixed crash in <sub/sup> handling (bug #1545152)

[wxWidgets.git] / wxPython / src / _display.i

diff --git a/wxPython/src/_display.i b/wxPython/src/_display.i
index db32d011a8456433925a0f31839f5f476bc7f4ca..6277cad3f58db4278bc2817a16a27b7acfa9bb12 100644 (file)
--- a/wxPython/src/_display.i
+++ b/wxPython/src/_display.i
@@ -16,7 +16,7 @@
 //---------------------------------------------------------------------------
 
 %{
 //---------------------------------------------------------------------------
 
 %{

-#include "wx/display.h"
+#include <wx/display.h>

 %}
 
 
 %}
 
 


@@ -31,41 +31,39 @@ struct wxVideoMode
 {
     wxVideoMode(int width = 0, int height = 0, int depth = 0, int freq = 0);
     ~wxVideoMode();
 {
     wxVideoMode(int width = 0, int height = 0, int depth = 0, int freq = 0);
     ~wxVideoMode();

-    
+

     DocDeclStr(
         bool , Matches(const wxVideoMode& other) const,
     DocDeclStr(
         bool , Matches(const wxVideoMode& other) const,

-        "Returns true if this mode matches the other one in the sense that all
-non zero fields of the other mode have the same value in this
+        "Returns True if this mode matches the other one in the sense that all
+non-zero fields of the other mode have the same value in this

 one (except for refresh which is allowed to have a greater value)", "");
 
     DocDeclStr(
         int , GetWidth() const,
         "Returns the screen width in pixels (e.g. 640*480), 0 means unspecified", "");
 one (except for refresh which is allowed to have a greater value)", "");
 
     DocDeclStr(
         int , GetWidth() const,
         "Returns the screen width in pixels (e.g. 640*480), 0 means unspecified", "");

-    
+

     DocDeclStr(
         int , GetHeight() const,
     DocDeclStr(
         int , GetHeight() const,

-        "Returns the screen width in pixels (e.g. 640*480), 0 means
-unspecified", "");
-    
+        "Returns the screen height in pixels (e.g. 640*480), 0 means unspecified", "");
+

     DocDeclStr(
         int , GetDepth() const,
         "Returns the screen's bits per pixel (e.g. 32), 1 is monochrome and 0
 means unspecified/known", "");
     DocDeclStr(
         int , GetDepth() const,
         "Returns the screen's bits per pixel (e.g. 32), 1 is monochrome and 0
 means unspecified/known", "");

-    
+

 
     DocDeclStr(
         bool , IsOk() const,
         "returns true if the object has been initialized", "");
 
     DocDeclStr(
         bool , IsOk() const,
         "returns true if the object has been initialized", "");

-    

 
 
     %pythoncode { def __nonzero__(self): return self.IsOk() }
     %extend {
 
 
     %pythoncode { def __nonzero__(self): return self.IsOk() }
     %extend {

-        bool __eq__(const wxVideoMode* other) { return other ? (*self == *other) : False; }
-        bool __ne__(const wxVideoMode* other) { return other ? (*self != *other) : True;  }
+        bool __eq__(const wxVideoMode* other) { return other ? (*self == *other) : false; }
+        bool __ne__(const wxVideoMode* other) { return other ? (*self != *other) : true;  }

     }
 
     }
 

-    
+

     // the screen size in pixels (e.g. 640*480), 0 means unspecified
     int w, h;
 
     // the screen size in pixels (e.g. 640*480), 0 means unspecified
     int w, h;
 


@@ -77,6 +75,12 @@ means unspecified/known", "");
 };
 
 
 };
 
 

+%{
+#if !wxUSE_DISPLAY
+const wxVideoMode     wxDefaultVideoMode;
+#endif
+%}
+

 %immutable;
 const wxVideoMode     wxDefaultVideoMode;
 %mutable;
 %immutable;
 const wxVideoMode     wxDefaultVideoMode;
 %mutable;


@@ -84,70 +88,64 @@ const wxVideoMode     wxDefaultVideoMode;
 
 //---------------------------------------------------------------------------
 
 
 //---------------------------------------------------------------------------
 

-DocStr(wxDisplay,
-       "Represents a display/monitor attached to the system", "");

 
 
 
 

+DocStr(wxDisplay,
+       "Represents a display/monitor attached to the system", "");

 class wxDisplay
 {
 public:
 class wxDisplay
 {
 public:

-    // 

     DocCtorStr(
         wxDisplay(size_t index = 0),
         "Set up a Display instance with the specified display.  The displays
 are numbered from 0 to GetCount() - 1, 0 is always the primary display
 and the only one which is always supported", "");
 
     DocCtorStr(
         wxDisplay(size_t index = 0),
         "Set up a Display instance with the specified display.  The displays
 are numbered from 0 to GetCount() - 1, 0 is always the primary display
 and the only one which is always supported", "");
 

-    virtual ~wxDisplay();
-    
+    ~wxDisplay();
+

     DocDeclStr(
         static size_t , GetCount(),
         "Return the number of available displays.", "");
     DocDeclStr(
         static size_t , GetCount(),
         "Return the number of available displays.", "");

-    

 
     DocDeclStr(
         static int , GetFromPoint(const wxPoint& pt),
         "Find the display where the given point lies, return wx.NOT_FOUND if it
 doesn't belong to any display", "");
 
     DocDeclStr(
         static int , GetFromPoint(const wxPoint& pt),
         "Find the display where the given point lies, return wx.NOT_FOUND if it
 doesn't belong to any display", "");

-    

 
     DocStr(GetFromWindow,
            "Find the display where the given window lies, return wx.NOT_FOUND if
 it is not shown at all.", "");
 
     DocStr(GetFromWindow,
            "Find the display where the given window lies, return wx.NOT_FOUND if
 it is not shown at all.", "");

-#ifdef __WXMSW__
+

     static int GetFromWindow(wxWindow *window);
     static int GetFromWindow(wxWindow *window);

-#else
-    %extend {
-        static int GetFromWindow(wxWindow *window) 
-            { wxPyRaiseNotImplemented(); return wxNOT_FOUND; }
-    }
-#endif
-    
+
+

     DocDeclStr(
     DocDeclStr(

-        virtual bool , IsOk() const,
+        bool , IsOk() const,

         "Return true if the object was initialized successfully", "");
     %pythoncode { def __nonzero__(self): return self.IsOk() }
         "Return true if the object was initialized successfully", "");
     %pythoncode { def __nonzero__(self): return self.IsOk() }

- 

 
     DocDeclStr(
 
     DocDeclStr(

-        virtual wxRect , GetGeometry() const,
+        wxRect , GetGeometry() const,

         "Returns the bounding rectangle of the display whose index was passed
 to the constructor.", "");
         "Returns the bounding rectangle of the display whose index was passed
 to the constructor.", "");

-    

 
     DocDeclStr(
 
     DocDeclStr(

-        virtual wxString , GetName() const,
+        wxRect , GetClientArea() const,
+        "Returns the bounding rectangle the client area of the display,
+i.e., without taskbars and such.", "");
+
+    DocDeclStr(
+        wxString , GetName() const,

         "Returns the display's name. A name is not available on all platforms.", "");
         "Returns the display's name. A name is not available on all platforms.", "");

-    

 
     DocDeclStr(
         bool , IsPrimary() const,
 
     DocDeclStr(
         bool , IsPrimary() const,

-        "Returns true if the display is the primary display. The primary
+        "Returns True if the display is the primary display. The primary

 display is the one whose index is 0.", "");
 display is the one whose index is 0.", "");

-    

 
 
 
 

-    %extend {
+    %extend
+    {

         DocAStr(GetModes,
                 "GetModes(VideoMode mode=DefaultVideoMode) -> [videoMode...]",
                 "Enumerate all video modes supported by this display matching the given
         DocAStr(GetModes,
                 "GetModes(VideoMode mode=DefaultVideoMode) -> [videoMode...]",
                 "Enumerate all video modes supported by this display matching the given


@@ -157,13 +155,16 @@ As any mode matches the default value of the argument and there is
 always at least one video mode supported by display, the returned
 array is only empty for the default value of the argument if this
 function is not supported at all on this platform.", "");
 always at least one video mode supported by display, the returned
 array is only empty for the default value of the argument if this
 function is not supported at all on this platform.", "");

-        
-        PyObject* GetModes(const wxVideoMode& mode = wxDefaultVideoMode) {
+
+        PyObject* GetModes(const wxVideoMode& mode = wxDefaultVideoMode)
+        {
+%#if wxUSE_DISPLAY

             PyObject* pyList = NULL;
             wxArrayVideoModes arr = self->GetModes(mode);
             PyObject* pyList = NULL;
             wxArrayVideoModes arr = self->GetModes(mode);

-            bool blocked = wxPyBeginBlockThreads();
+            wxPyBlock_t blocked = wxPyBeginBlockThreads();

             pyList = PyList_New(0);
             pyList = PyList_New(0);

-            for (int i=0; i < arr.GetCount(); i++) {
+            for (size_t i=0; i < arr.GetCount(); i++)
+            {

                 wxVideoMode* m = new wxVideoMode(arr.Item(i));
                 PyObject* pyObj = wxPyConstructObject(m, wxT("wxVideoMode"), true);
                 PyList_Append(pyList, pyObj);
                 wxVideoMode* m = new wxVideoMode(arr.Item(i));
                 PyObject* pyObj = wxPyConstructObject(m, wxT("wxVideoMode"), true);
                 PyList_Append(pyList, pyObj);


@@ -171,25 +172,68 @@ function is not supported at all on this platform.", "");
             }
             wxPyEndBlockThreads(blocked);
             return pyList;
             }
             wxPyEndBlockThreads(blocked);
             return pyList;

+%#else
+        wxPyRaiseNotImplemented();
+        return NULL;
+%#endif

         }
         }

-    }
-    

 
 

-    DocDeclStr(
-        virtual wxVideoMode , GetCurrentMode() const,
-        "Get the current video mode.", "");
-    
+        DocStr(GetCurrentMode,
+               "Get the current video mode.", "");
+        wxVideoMode GetCurrentMode() const
+        {
+%#if wxUSE_DISPLAY
+            return self->GetCurrentMode();
+%#else
+            wxPyRaiseNotImplemented();
+            return wxDefaultVideoMode;
+%#endif
+        }

 
 

-    DocDeclStr(
-        virtual bool , ChangeMode(const wxVideoMode& mode = wxDefaultVideoMode),
-        "Change current mode, return true if succeeded, false otherwise", "");
-    

 
 

-    DocDeclStr(
-        void , ResetMode(),
-        "Restore the default video mode (just a more readable synonym)", "");
-    
+        DocStr(
+            ChangeMode,
+            "Changes the video mode of this display to the mode specified in the
+mode parameter.
+
+If wx.DefaultVideoMode is passed in as the mode parameter, the defined
+behaviour is that wx.Display will reset the video mode to the default
+mode used by the display.  On Windows, the behavior is normal.
+However, there are differences on other platforms. On Unix variations
+using X11 extensions it should behave as defined, but some
+irregularities may occur.
+
+On wxMac passing in wx.DefaultVideoMode as the mode parameter does
+nothing.  This happens because Carbon no longer has access to
+DMUseScreenPrefs, an undocumented function that changed the video mode
+to the system default by using the system's 'scrn' resource.
+
+Returns True if succeeded, False otherwise", "");
+
+        bool ChangeMode(const wxVideoMode& mode = wxDefaultVideoMode)
+        {
+%#if wxUSE_DISPLAY
+            return self->ChangeMode(mode);
+%#else
+            wxPyRaiseNotImplemented();
+            return false;
+%#endif
+        }
+
+
+        DocStr(
+            ResetMode,
+            "Restore the default video mode (just a more readable synonym)", "");
+        void ResetMode()
+        {
+%#if wxUSE_DISPLAY
+            self->ResetMode();
+%#else
+            wxPyRaiseNotImplemented();
+%#endif
+        }

 
 

+    } // end of %extend

 };
 
 //---------------------------------------------------------------------------
 };
 
 //---------------------------------------------------------------------------