]> git.tdb.fi Git - libs/gltk.git/commitdiff
Use the override specifier when overriding a virtual function
authorMikko Rasa <tdb@tdb.fi>
Sun, 20 Aug 2023 10:00:21 +0000 (13:00 +0300)
committerMikko Rasa <tdb@tdb.fi>
Sun, 20 Aug 2023 21:52:37 +0000 (00:52 +0300)
22 files changed:
source/button.h
source/column.h
source/container.h
source/dialog.h
source/draghandle.h
source/dropdown.h
source/entry.h
source/floatingarrangement.h
source/grid.h
source/image.h
source/indicator.h
source/label.h
source/lineararrangement.h
source/list.h
source/listdata.h
source/panel.h
source/progressbar.h
source/root.h
source/row.h
source/slider.h
source/stack.h
source/toggle.h

index 962b2e11908e0608b2ced7133274a0d0e648ec59..165ca4f8447d779a3b82ac05bd1180aa16512212 100644 (file)
@@ -35,25 +35,25 @@ private:
 public:
        Button(const std::string & = std::string());
 
-       virtual const char *get_class() const { return "button"; }
+       const char *get_class() const override { return "button"; }
 
 private:
-       virtual void autosize_special(const Part &, Geometry &) const;
+       void autosize_special(const Part &, Geometry &) const override;
 
 public:
        void set_text(const std::string &);
        void set_icon(const GL::Texture2D *);
 
 private:
-       virtual void rebuild_special(const Part &);
+       void rebuild_special(const Part &) override;
 
 public:
-       virtual void button_press(int, int, unsigned);
-       virtual void button_release(int, int, unsigned);
-       virtual void pointer_motion(int, int);
-       virtual bool navigate(Navigation);
+       void button_press(int, int, unsigned) override;
+       void button_release(int, int, unsigned) override;
+       void pointer_motion(int, int) override;
+       bool navigate(Navigation) override;
 private:
-       virtual void on_style_change();
+       void on_style_change() override;
 };
 
 } // namespace GLtk
index 6d6acd6148aae1960018955977bcec81f18d9ac2..d82c8cad65a8f0d351a3cd0211ccc0fd548570f7 100644 (file)
@@ -13,7 +13,7 @@ public:
        Column(Layout &);
 
 private:
-       virtual void finish_widget(Widget &);
+       void finish_widget(Widget &) override;
 };
 
 } // namespace GLtk
index a0cba3b776f8ffe3fce88443775bff461304d85b..26100b72d315234329b9814ffdd80d14fc50c8f8 100644 (file)
@@ -75,28 +75,28 @@ private:
        void check_animation_interval();
 
 protected:
-       virtual void rebuild_hierarchy();
+       void rebuild_hierarchy() override;
 
 public:
-       virtual void button_press(int, int, unsigned);
-       virtual void button_release(int, int, unsigned);
-       virtual void pointer_motion(int, int);
+       void button_press(int, int, unsigned) override;
+       void button_release(int, int, unsigned) override;
+       void pointer_motion(int, int) override;
 private:
        Widget *get_pointer_target(int, int, bool) const;
 public:
-       virtual void pointer_leave();
-       virtual void touch_press(int, int, unsigned);
-       virtual void touch_release(int, int, unsigned);
-       virtual void touch_motion(int, int, unsigned);
-       virtual bool key_press(unsigned, unsigned);
-       virtual bool key_release(unsigned, unsigned);
-       virtual bool character(wchar_t);
-       virtual void focus_in();
-       virtual void focus_out();
-       virtual bool navigate(Navigation);
-       virtual void animate(const Time::TimeDelta &);
+       void pointer_leave() override;
+       void touch_press(int, int, unsigned) override;
+       void touch_release(int, int, unsigned) override;
+       void touch_motion(int, int, unsigned) override;
+       bool key_press(unsigned, unsigned) override;
+       bool key_release(unsigned, unsigned) override;
+       bool character(wchar_t) override;
+       void focus_in() override;
+       void focus_out() override;
+       bool navigate(Navigation) override;
+       void animate(const Time::TimeDelta &) override;
 protected:
-       virtual void on_reparent();
+       void on_reparent() override;
        virtual void on_child_added(Widget &) { }
        virtual void on_child_removed(Widget &) { }
        virtual void on_input_focus_changed(Widget *);
index e71222739cc415acf1876f3e1f3e60a560f11752..285478ebb570ddead2d12deae299805dd595ecee 100644 (file)
@@ -30,7 +30,7 @@ private:
        bool stale = false;
 
 public:
-       virtual const char *get_class() const { return "dialog"; }
+       const char *get_class() const override { return "dialog"; }
 
        /** Adds an action button to the dialog.  Pressing the button will invoke
        response handlers and delete the dialog. */
@@ -40,9 +40,9 @@ public:
        away from the dialog. */
        void set_modal(bool);
 
-       virtual void button_release(int, int, unsigned);
-       virtual bool key_release(unsigned, unsigned);
-       virtual bool navigate(Navigation);
+       void button_release(int, int, unsigned) override;
+       bool key_release(unsigned, unsigned) override;
+       bool navigate(Navigation) override;
 protected:
        void response(int);
        void check_stale();
index d3cd6f2b94cd88c6c17f9d22381c289308e26b47..68381feba520b4b0be7e0d3bc4e2dda96cb51cc5 100644 (file)
@@ -19,13 +19,13 @@ private:
        int drag_y = 0;
 
 public:
-       virtual const char *get_class() const { return "draghandle"; }
+       const char *get_class() const override { return "draghandle"; }
 
-       virtual void button_press(int, int, unsigned);
-       virtual void button_release(int, int, unsigned);
-       virtual void pointer_motion(int, int);
+       void button_press(int, int, unsigned) override;
+       void button_release(int, int, unsigned) override;
+       void pointer_motion(int, int) override;
 private:
-       virtual void on_reparent();
+       void on_reparent() override;
 };
 
 } // namespace GLtk
index 70a9faaf646c4b6381bd023eda3262d0a48c41cd..c5b535d599bcaae434bb29f0fd53f06f0334890f 100644 (file)
@@ -37,10 +37,10 @@ private:
        void init();
 
 public:
-       virtual const char *get_class() const { return "dropdown"; }
+       const char *get_class() const override { return "dropdown"; }
 
 private:
-       virtual void autosize_special(const Part &, Geometry &) const;
+       void autosize_special(const Part &, Geometry &) const override;
 
 public:
        void set_data(ListData &d) { list.set_data(d); }
@@ -54,15 +54,15 @@ public:
        int get_selected_index() const { return list.get_selected_index(); }
 
 private:
-       virtual void rebuild_special(const Part &);
-       virtual void render_special(const Part &, GL::Renderer &) const;
+       void rebuild_special(const Part &) override;
+       void render_special(const Part &, GL::Renderer &) const override;
 
 public:
-       virtual void button_press(int, int, unsigned);
-       virtual bool navigate(Navigation);
+       void button_press(int, int, unsigned) override;
+       bool navigate(Navigation) override;
 private:
-       virtual void on_size_change();
-       virtual void on_style_change();
+       void on_size_change() override;
+       void on_style_change() override;
 
        void open_list();
        void close_list();
index 66b3327d70c29b3d7200d91ca0d9c898b3ebec6e..0573671abfded057908d9fc320b0671e59a883e4 100644 (file)
@@ -55,10 +55,10 @@ private:
 public:
        Entry(const std::string & = std::string());
 
-       virtual const char *get_class() const { return "entry"; }
+       const char *get_class() const override { return "entry"; }
 
 private:
-       virtual void autosize_special(const Part &, Geometry &) const;
+       void autosize_special(const Part &, Geometry &) const override;
 
 public:
        void set_text(const std::string &);
@@ -82,20 +82,20 @@ public:
        bool is_multiline() const { return multiline; }
 
 private:
-       virtual void rebuild_special(const Part &);
-       virtual void render_special(const Part &, GL::Renderer &) const;
+       void rebuild_special(const Part &) override;
+       void render_special(const Part &, GL::Renderer &) const override;
 
 public:
-       virtual void touch_press(int, int, unsigned);
-       virtual bool key_press(unsigned, unsigned);
-       virtual bool character(wchar_t);
-       virtual void focus_in();
-       virtual void focus_out();
-       virtual bool navigate(Navigation);
-       virtual void animate(const Time::TimeDelta &);
+       void touch_press(int, int, unsigned) override;
+       bool key_press(unsigned, unsigned) override;
+       bool character(wchar_t) override;
+       void focus_in() override;
+       void focus_out() override;
+       bool navigate(Navigation) override;
+       void animate(const Time::TimeDelta &) override;
 private:
-       virtual void on_size_change();
-       virtual void on_style_change();
+       void on_size_change() override;
+       void on_style_change() override;
 
        void move_edit_position(Navigation, bool);
        void adjust_edit_position_for_change(unsigned, int);
index 8539b5964037e382a8f02019c998f3bb76af1868..1ac4fa95de000e5320930ff21e2a8999dce3012e 100644 (file)
@@ -13,9 +13,9 @@ public:
        FloatingArrangement(Layout &);
 
 private:
-       virtual void process_widget(Widget &, Side, bool) { }
-       virtual void finish_widget(Widget &);
-       virtual void finish_slot() { }
+       void process_widget(Widget &, Side, bool) override { }
+       void finish_widget(Widget &) override;
+       void finish_slot() override { }
 };
 
 } // namespace GLtk
index 9324f95fac937b1bb53c1d1167a861da338109d8..4ed43d56442f59d0b475ba7b3cf6fd097b676225 100644 (file)
@@ -40,9 +40,9 @@ public:
        void next_row();
 
 private:
-       virtual void process_widget(Widget &, Side, bool);
-       virtual void finish_widget(Widget &);
-       virtual void finish_slot();
+       void process_widget(Widget &, Side, bool) override;
+       void finish_widget(Widget &) override;
+       void finish_slot() override;
        void finish_row();
 };
 
index 12e77f2a7f6df28f935bcf2c66dac9290913f33f..99e0874f124e36b72ca3a08660896a9b40cc6f5c 100644 (file)
@@ -28,10 +28,10 @@ private:
 public:
        Image(const GL::Texture2D * = nullptr);
 
-       virtual const char *get_class() const { return "image"; }
+       const char *get_class() const override { return "image"; }
 
 private:
-       virtual void autosize_special(const Part &, Geometry &) const;
+       void autosize_special(const Part &, Geometry &) const override;
 
 public:
        void set_image(const GL::Texture2D *);
@@ -40,9 +40,9 @@ public:
 
 private:
        void update_icon();
-       virtual void rebuild_special(const Part &);
-       virtual void on_style_change();
-       virtual void on_reparent();
+       void rebuild_special(const Part &) override;
+       void on_style_change() override;
+       void on_reparent() override;
 };
 
 } // namespace GLtk
index 2b5b2a397f22cf79c11ec1e567fbaad6e9cf0d3a..9ec9f33c8c2a1fa41f3048ae669e7b06ce2383b4 100644 (file)
@@ -13,7 +13,7 @@ An Indicator visualizes a boolean state.  It can be either active or inactive.
 class MSPGLTK_API Indicator: public Widget
 {
 public:
-       virtual const char *get_class() const { return "indicator"; }
+       const char *get_class() const override { return "indicator"; }
 
        void set_active(bool);
 };
index 3275bca091962e4d77555c45590a733fab80f578..6b350ed7ecd690e25da9a288ea877554181eabb7 100644 (file)
@@ -28,15 +28,15 @@ private:
 public:
        Label(const std::string & = std::string());
 
-       virtual const char *get_class() const { return "label"; }
+       const char *get_class() const override { return "label"; }
 
        void set_text(const std::string &);
 
 private:
-       virtual void autosize_special(const Part &, Geometry &) const;
-       virtual void rebuild_special(const Part &);
+       void autosize_special(const Part &, Geometry &) const override;
+       void rebuild_special(const Part &) override;
 
-       virtual void on_style_change();
+       void on_style_change() override;
 };
 
 } // namespace GLtk
index f0ec2453d02121d445b3caf9b3307948709c0110..7535bc7879247c57c151a0f839e39245f5146aa2 100644 (file)
@@ -43,9 +43,9 @@ public:
        void spacing(unsigned);
 
 protected:
-       virtual void process_widget(Widget &, Side, bool);
-       virtual void finish_widget(Widget &);
-       virtual void finish_slot();
+       void process_widget(Widget &, Side, bool) override;
+       void finish_widget(Widget &) override;
+       void finish_slot() override;
 };
 
 } // namespace GLtk
index 37d4d7a33201bdf80894508bd8ab8dd58a962fa2..060fb813f405cc9b29fb762c8a5ac93e7feedc61 100644 (file)
@@ -67,14 +67,14 @@ public:
                Item();
 
        public:
-               virtual const char *get_class() const { return "listitem"; }
+               const char *get_class() const override { return "listitem"; }
 
        protected:
-               virtual void autosize_special(const Part &, Geometry &) const;
+               void autosize_special(const Part &, Geometry &) const override;
        public:
                void set_active(bool);
 
-               virtual void render_special(const Part &, GL::Renderer &) const;
+               void render_special(const Part &, GL::Renderer &) const override;
        };
 
        class SimpleItem: public Item
@@ -82,7 +82,7 @@ public:
        protected:
                SimpleItem() = default;
 
-               virtual void on_style_change();
+               void on_style_change() override;
        };
 
        class MultiColumnItem: public Item
@@ -93,7 +93,7 @@ public:
                virtual void check_widths(std::vector<unsigned> &) const;
                virtual void set_widths(const std::vector<unsigned> &);
 
-               virtual void on_style_change();
+               void on_style_change() override;
        };
 
 private:
@@ -129,7 +129,7 @@ private:
                TypedItemFactory(const ListData &d)
                { set_data(d); }
 
-               virtual void set_data(const ListData &d)
+               void set_data(const ListData &d) override
                {
                        if(const ListDataStore<ValueType> *ds = dynamic_cast<const ListDataStore<ValueType> *>(&d))
                                data = ds;
@@ -137,7 +137,7 @@ private:
                                throw incompatible_data(typeid(ValueType));
                }
 
-               virtual Item *create_item(unsigned i) const
+               Item *create_item(unsigned i) const override
                {
                        return new I(data->get(i));
                }
@@ -182,10 +182,10 @@ public:
        List(ListData &);
        virtual ~List();
 
-       virtual const char *get_class() const { return "list"; }
+       const char *get_class() const override { return "list"; }
 
 private:
-       virtual void autosize_special(const Part &, Geometry &) const;
+       void autosize_special(const Part &, Geometry &) const override;
 
 public:
        void set_data(ListData &);
@@ -216,19 +216,19 @@ public:
 private:
        void set_selected_item(Widget *);
 
-       virtual void rebuild_special(const Part &);
-       virtual void render_special(const Part &, GL::Renderer &) const;
+       void rebuild_special(const Part &) override;
+       void render_special(const Part &, GL::Renderer &) const override;
 
 public:
-       virtual bool key_press(unsigned, unsigned);
-       virtual void button_press(int, int, unsigned);
-       virtual void touch_press(int, int, unsigned);
-       virtual void touch_release(int, int, unsigned);
-       virtual void touch_motion(int, int, unsigned);
-       virtual void focus_in();
-       virtual bool navigate(Navigation);
+       bool key_press(unsigned, unsigned) override;
+       void button_press(int, int, unsigned) override;
+       void touch_press(int, int, unsigned) override;
+       void touch_release(int, int, unsigned) override;
+       void touch_motion(int, int, unsigned) override;
+       void focus_in() override;
+       bool navigate(Navigation) override;
 private:
-       virtual void on_style_change();
+       void on_style_change() override;
 
        void move_focus(Navigation, bool);
        void set_focus_index(int);
index 00d2b9b65a154e587789b7ffaf18af1ad2334007..77be5ee7b8e1cb52200936eb05d85ddc0b8cc3c4 100644 (file)
@@ -93,7 +93,7 @@ public:
                signal_cleared.emit();
        }
 
-       virtual unsigned size() const { return items.size(); }
+       unsigned size() const override { return items.size(); }
 };
 
 template<typename T>
@@ -116,7 +116,7 @@ private:
 public:
        FunctionListData(Func f): func(f) { }
 
-       virtual std::string get_string(unsigned i) const
+       std::string get_string(unsigned i) const override
        { return func(this->get(i)); }
 };
 
index b5755d0cbb1d4f39d7c0975d45cb19656eac8cf0..ce4b2a740073dc2b2eb743578d1631e38e477f67 100644 (file)
@@ -81,24 +81,24 @@ public:
        template<typename T>
        static void register_child_type(const std::string &);
 
-       virtual const char *get_class() const { return "panel"; }
+       const char *get_class() const override { return "panel"; }
 
        void set_layout(Layout *);
        Layout *get_layout() { return layout; }
 
 protected:
-       virtual void autosize_special(const Part &, Geometry &) const;
-       virtual void render_special(const Part &, GL::Renderer &) const;
+       void autosize_special(const Part &, Geometry &) const override;
+       void render_special(const Part &, GL::Renderer &) const override;
 
 public:
-       virtual bool navigate(Navigation);
+       bool navigate(Navigation) override;
 protected:
        Widget *find_next_child(int, int, int, int, int) const;
        static int compute_delta(int, int, int, int, int);
 
-       virtual void on_size_change();
-       virtual void on_child_added(Widget &);
-       virtual void on_child_removed(Widget &);
+       void on_size_change() override;
+       void on_child_added(Widget &) override;
+       void on_child_removed(Widget &) override;
 };
 
 
index 6e44e731dff4a3355371aa04eb69376d66eaeffd..756219244453279b9e885996b230e12caae3ddd5 100644 (file)
@@ -14,14 +14,14 @@ private:
        float fraction = 0.0f;
 
 public:
-       virtual const char *get_class() const { return "progressbar"; }
+       const char *get_class() const override { return "progressbar"; }
 
        void set_range(float);
        void set_value(float);
 
 private:
-       virtual void autosize_special(const Part &, Geometry &) const;
-       virtual void rebuild_special(const Part &);
+       void autosize_special(const Part &, Geometry &) const override;
+       void rebuild_special(const Part &) override;
 };
 
 } // namespace GLtk
index 0fe624f4c4e3f5d03e7fab70ee4bc00d594cd20e..c71d50d6f7577d4ec3e710149c70115d50850ae3 100644 (file)
@@ -57,15 +57,15 @@ public:
        Root(Resources &, Graphics::Window *, Input::Keyboard *, Input::Mouse *, Input::Touchscreen * = nullptr);
        virtual ~Root();
 
-       virtual const char *get_class() const { return "root"; }
+       const char *get_class() const override { return "root"; }
 
        Resources &get_resources() const { return resources; }
        virtual unsigned get_width() const { return geom.w; }
        virtual unsigned get_height() const { return geom.h; }
 
        void tick();
-       virtual void setup_frame(GL::Renderer &);
-       virtual void render(GL::Renderer &, GL::Tag = GL::Tag()) const;
+       void setup_frame(GL::Renderer &) override;
+       void render(GL::Renderer &, GL::Tag = GL::Tag()) const override;
 
 private:
        bool button_press_event(unsigned);
@@ -79,8 +79,8 @@ private:
        void get_touch(unsigned, int &, int &);
        void update_camera();
 
-       virtual void on_size_change();
-       virtual void on_child_added(Widget &);
+       void on_size_change() override;
+       void on_child_added(Widget &) override;
 };
 
 } // namespace GLtk
index 849ba62bf6afd51de7a13171dcbbaae23cc4a105..678a3853cc2023b399286b628d3330ea6f0fce21 100644 (file)
@@ -13,7 +13,7 @@ public:
        Row(Layout &);
 
 private:
-       virtual void finish_widget(Widget &);
+       void finish_widget(Widget &) override;
 };
 
 } // namespace GLtk
index 70c5fc59b2882d948f7ceb543465bde4127e0895..057b194a2c532d7d96f9a96c421a63b2f21bb84f 100644 (file)
@@ -59,17 +59,17 @@ public:
        double get_value() const { return value; }
 
 protected:
-       virtual void autosize_special(const Part &, Geometry &) const;
-       virtual void rebuild_special(const Part &);
+       void autosize_special(const Part &, Geometry &) const override;
+       void rebuild_special(const Part &) override;
 
 public:
-       virtual void button_press(int, int, unsigned);
-       virtual void button_release(int, int, unsigned);
-       virtual void pointer_motion(int, int);
+       void button_press(int, int, unsigned) override;
+       void button_release(int, int, unsigned) override;
+       void pointer_motion(int, int) override;
 
 protected:
-       virtual void on_size_change();
-       virtual void on_style_change();
+       void on_size_change() override;
+       void on_style_change() override;
 };
 
 
@@ -78,7 +78,7 @@ class MSPGLTK_API VSlider: public Slider
 public:
        VSlider(): Slider(VERTICAL) { }
 
-       virtual const char *get_class() const { return "vslider"; }
+       const char *get_class() const override { return "vslider"; }
 };
 
 class MSPGLTK_API HSlider: public Slider
@@ -86,7 +86,7 @@ class MSPGLTK_API HSlider: public Slider
 public:
        HSlider(): Slider(HORIZONTAL) { }
 
-       virtual const char *get_class() const { return "hslider"; }
+       const char *get_class() const override { return "hslider"; }
 };
 
 } // namespace GLtk
index 53943fda13f13b1d767f2b392947cf6f61e2fcec..193a80667a2166bcbec33e0d10643d6a32d1b85e 100644 (file)
@@ -24,9 +24,9 @@ public:
        Stack(Layout &);
 
 private:
-       virtual void process_widget(Widget &, Side, bool);
-       virtual void finish_widget(Widget &);
-       virtual void finish_slot() { }
+       void process_widget(Widget &, Side, bool) override;
+       void finish_widget(Widget &) override;
+       void finish_slot() override { }
 };
 
 } // namespace GLtk
index 7c572c51e98c74d18cbf5860e56d6460224afb23..2b38ffc2084f023ac0ea011f9720b58e85e2195f 100644 (file)
@@ -20,7 +20,7 @@ public:
        public:
                Loader(Toggle &);
        private:
-               virtual void finish();
+               void finish() override;
                void text(const std::string &);
        };
 
@@ -35,10 +35,10 @@ private:
 public:
        Toggle(const std::string & = std::string());
 
-       virtual const char *get_class() const { return "toggle"; }
+       const char *get_class() const override { return "toggle"; }
 
 private:
-       virtual void autosize_special(const Part &, Geometry &) const;
+       void autosize_special(const Part &, Geometry &) const override;
 
 public:
        void set_text(const std::string &);
@@ -51,14 +51,14 @@ public:
        bool get_value() const { return value; }
 
 private:
-       virtual void rebuild_special(const Part &);
+       void rebuild_special(const Part &) override;
 
 public:
-       virtual void button_press(int, int, unsigned);
-       virtual void button_release(int, int, unsigned);
-       virtual bool navigate(Navigation);
+       void button_press(int, int, unsigned) override;
+       void button_release(int, int, unsigned) override;
+       bool navigate(Navigation) override;
 private:
-       virtual void on_style_change();
+       void on_style_change() override;
 };
 
 } // namespace GLtk