Stop using object array for wxArrayVideoModes

This can be just a simple wxBaseArray.

Also document this container, as it is returned from the public
wxDisplay::GetModes().

include/wx/display.h

@@ -21,7 +21,7 @@
     #include "wx/dynarray.h"
     #include "wx/vidmode.h"
 
-    WX_DECLARE_EXPORTED_OBJARRAY(wxVideoMode, wxArrayVideoModes);
+    using wxArrayVideoModes = wxBaseArray<wxVideoMode>;
 
     // default, uninitialized, video mode object
     extern WXDLLIMPEXP_DATA_CORE(const wxVideoMode) wxDefaultVideoMode;

interface/wx/vidmode.h

@@ -85,3 +85,11 @@ public:
 */
 const wxVideoMode wxDefaultVideoMode;
 
+/**
+    A vector of video modes.
+
+    This class is actually a legacy container (see @ref overview_container for
+    more details), but it can, and should be, handled as just a vector of
+    wxVideoMode objects in the application code.
+*/
+using wxArrayVideoModes = std::vector<wxVideoMode>;

src/common/dpycmn.cpp

@@ -32,9 +32,6 @@
 
 #if wxUSE_DISPLAY
 
-#include "wx/arrimpl.cpp"
-WX_DEFINE_OBJARRAY(wxArrayVideoModes)
-
 const wxVideoMode wxDefaultVideoMode;
 
 #endif // wxUSE_DISPLAY