python: provide a better error if draw_event() is not overriden.

Otherwise it yells at the user that "pure virtual method called", and
that's no good.

src/python/magnum/platform/glfw.cpp

@@ -43,15 +43,10 @@ void glfw(py::module& m) {
     struct PublicizedApplication: Platform::Application {
         explicit PublicizedApplication(const Configuration& configuration, const GLConfiguration& glConfiguration): Platform::Application{Arguments{argc, nullptr}, configuration, glConfiguration} {}
 
-        /* MSVC dies with "error C3640: a referenced or virtual member function
+        void drawEvent() override {
-           of a local class must be defined" if this is just `= 0` here. Since
+            PyErr_SetString(PyExc_NotImplementedError, "the application has to provide a draw_event() method");
-           we're overriding this method below anyway, it doesn't have to be
+            throw py::error_already_set{};
-           pure virtual. */
+        }
-        #ifdef _MSC_VER
-        void drawEvent() override {}
-        #else
-        void drawEvent() override = 0;
-        #endif
 
         void keyPressEvent(KeyEvent&) override {}
         void keyReleaseEvent(KeyEvent&) override {}
@@ -71,12 +66,12 @@ void glfw(py::module& m) {
         void drawEvent() override {
             #ifdef __clang__
             /* ugh pybind don't tell me I AM THE FIRST ON EARTH to get a
-               warning here. Why there's no PYBIND11_OVERLOAD_PURE_NAME_ARG()
+               warning here. Why there's no PYBIND11_OVERLOAD_NAME_ARG()
                variant *with* arguments and one without? */
             #pragma GCC diagnostic push
             #pragma GCC diagnostic ignored "-Wgnu-zero-variadic-macro-arguments"
             #endif
-            PYBIND11_OVERLOAD_PURE_NAME(
+            PYBIND11_OVERLOAD_NAME(
                 void,
                 PublicizedApplication,
                 "draw_event",

src/python/magnum/platform/sdl2.cpp

@@ -43,15 +43,10 @@ void sdl2(py::module& m) {
     struct PublicizedApplication: Platform::Application {
         explicit PublicizedApplication(const Configuration& configuration, const GLConfiguration& glConfiguration): Platform::Application{Arguments{argc, nullptr}, configuration, glConfiguration} {}
 
-        /* MSVC dies with "error C3640: a referenced or virtual member function
+        void drawEvent() override {
-           of a local class must be defined" if this is just `= 0` here. Since
+            PyErr_SetString(PyExc_NotImplementedError, "the application has to provide a draw_event() method");
-           we're overriding this method below anyway, it doesn't have to be
+            throw py::error_already_set{};
-           pure virtual. */
+        }
-        #ifdef _MSC_VER
-        void drawEvent() override {}
-        #else
-        void drawEvent() override = 0;
-        #endif
 
         void keyPressEvent(KeyEvent&) override {}
         void keyReleaseEvent(KeyEvent&) override {}
@@ -71,12 +66,12 @@ void sdl2(py::module& m) {
         void drawEvent() override {
             #ifdef __clang__
             /* ugh pybind don't tell me I AM THE FIRST ON EARTH to get a
-               warning here. Why there's no PYBIND11_OVERLOAD_PURE_NAME_ARG()
+               warning here. Why there's no PYBIND11_OVERLOAD_NAME_ARG()
                variant *with* arguments and one without? */
             #pragma GCC diagnostic push
             #pragma GCC diagnostic ignored "-Wgnu-zero-variadic-macro-arguments"
             #endif
-            PYBIND11_OVERLOAD_PURE_NAME(
+            PYBIND11_OVERLOAD_NAME(
                 void,
                 PublicizedApplication,
                 "draw_event",