[LyX/master] Add more `override` specifiers

Yuriy Skalko yuriy.skalko at gmail.com
Fri Nov 20 22:03:19 UTC 2020


commit 2941bc01981f151df9360312594bd0517ecc7d3a
Author: Yuriy Skalko <yuriy.skalko at gmail.com>
Date:   Thu Nov 19 15:40:14 2020 +0200

    Add more `override` specifiers
---
 src/frontends/qt/GuiClickableLabel.h |    2 +-
 src/frontends/qt/GuiFontExample.h    |    4 +-
 src/frontends/qt/GuiIdListModel.h    |   15 ++++----
 src/frontends/qt/GuiImage.h          |   12 +++---
 src/frontends/qt/GuiPainter.h        |   64 +++++++++++++++++-----------------
 src/frontends/qt/LaTeXHighlighter.h  |    2 +-
 src/frontends/qt/TocModel.h          |    2 +-
 7 files changed, 50 insertions(+), 51 deletions(-)

diff --git a/src/frontends/qt/GuiClickableLabel.h b/src/frontends/qt/GuiClickableLabel.h
index 4ae4905..b14ba15 100644
--- a/src/frontends/qt/GuiClickableLabel.h
+++ b/src/frontends/qt/GuiClickableLabel.h
@@ -27,7 +27,7 @@ Q_SIGNALS:
 	void clicked();
 
 protected:
-	void mouseReleaseEvent(QMouseEvent *);
+	void mouseReleaseEvent(QMouseEvent *) override;
 };
 
 }
diff --git a/src/frontends/qt/GuiFontExample.h b/src/frontends/qt/GuiFontExample.h
index f0592fd..57862e9 100644
--- a/src/frontends/qt/GuiFontExample.h
+++ b/src/frontends/qt/GuiFontExample.h
@@ -27,10 +27,10 @@ public:
 
 	void set(QFont const & font, QString const & text);
 
-	virtual QSize sizeHint() const;
+	QSize sizeHint() const override;
 
 protected:
-	virtual void paintEvent(QPaintEvent * p);
+	void paintEvent(QPaintEvent * p) override;
 
 private:
 	QFont font_;
diff --git a/src/frontends/qt/GuiIdListModel.h b/src/frontends/qt/GuiIdListModel.h
index a63b40d..53da3ec 100644
--- a/src/frontends/qt/GuiIdListModel.h
+++ b/src/frontends/qt/GuiIdListModel.h
@@ -44,23 +44,22 @@ public:
 	// Methods overridden from QAbstractListModel
 	//////////////////////////////////////////////////////////////////////
 	///
-	int rowCount(QModelIndex const & = QModelIndex()) const
+	int rowCount(QModelIndex const & = QModelIndex()) const override
 		{ return int(userData_.size()); }
 
 	///
-	virtual QVariant data(QModelIndex const & index,
-	                      int role = Qt::DisplayRole) const;
+	QVariant data(QModelIndex const & index, int role = Qt::DisplayRole) const override;
 	///
-	bool insertRows(int row, int count, QModelIndex const & parent = QModelIndex());
+	bool insertRows(int row, int count, QModelIndex const & parent = QModelIndex()) override;
 	///
-	bool removeRows(int row, int count, QModelIndex const & parent = QModelIndex());
+	bool removeRows(int row, int count, QModelIndex const & parent = QModelIndex()) override;
 	///
 	void clear() { removeRows(0, rowCount()); }
 	///
-	virtual bool setData (QModelIndex const & index,
-			const QVariant & value, int role = Qt::EditRole );
+	bool setData (QModelIndex const & index,
+			const QVariant & value, int role = Qt::EditRole) override;
 	///
-	virtual QMap<int, QVariant> itemData(QModelIndex const & index ) const;
+	QMap<int, QVariant> itemData(QModelIndex const & index ) const override;
 	//////////////////////////////////////////////////////////////////////
 	// New methods
 	//////////////////////////////////////////////////////////////////////
diff --git a/src/frontends/qt/GuiImage.h b/src/frontends/qt/GuiImage.h
index 6f60644..075c4e9 100644
--- a/src/frontends/qt/GuiImage.h
+++ b/src/frontends/qt/GuiImage.h
@@ -34,24 +34,24 @@ public:
 
 private:
 	/// Create a copy
-	Image * clone() const;
+	Image * clone() const override;
 	/// Get the image width
-	unsigned int width() const;
+	unsigned int width() const override;
 	/// Get the image height
-	unsigned int height() const;
+	unsigned int height() const override;
 	// FIXME Is the image drawable ?
-	bool isDrawable() const { return true; }
+	bool isDrawable() const override { return true; }
 	/**
 	 * Load the image file into memory.
 	 */
-	bool load(support::FileName const & filename);
+	bool load(support::FileName const & filename) override;
 	bool load();
 	/**
 	 * Finishes the process of modifying transformed_, using
 	 * \c params to decide on color, grayscale etc.
 	 * \returns true if successful.
 	 */
-	bool setPixmap(Params const & params);
+	bool setPixmap(Params const & params) override;
 
 	/// Clip the image using params.
 	bool clip(Params const & params);
diff --git a/src/frontends/qt/GuiPainter.h b/src/frontends/qt/GuiPainter.h
index 2e2a7f2..bffcd9a 100644
--- a/src/frontends/qt/GuiPainter.h
+++ b/src/frontends/qt/GuiPainter.h
@@ -36,15 +36,15 @@ public:
 	virtual ~GuiPainter();
 
 	/// This painter paints
-	virtual bool isNull() const { return false; }
+	bool isNull() const override { return false; }
 
 	/// draw a line from point to point
-	virtual void line(
+	void line(
 		int x1, int y1,
 		int x2, int y2,
 		Color,
 		line_style ls = line_solid,
-		int lw = thin_line);
+		int lw = thin_line) override;
 
 	/**
 	 * lines -  draw a set of lines
@@ -52,14 +52,14 @@ public:
 	 * @param yp array of points' y co-ords
 	 * @param np size of the points array
 	 */
-	virtual void lines(
+	void lines(
 		int const * xp,
 		int const * yp,
 		int np,
 		Color,
 		fill_style fs = fill_none,
 		line_style ls = line_solid,
-		int lw = thin_line);
+		int lw = thin_line) override;
 
 	/**
 	 * path -  draw a path with bezier curves
@@ -71,72 +71,72 @@ public:
 	 * @param c2y array of second control points' y co-ords
 	 * @param np size of the points array
 	 */
-	virtual void path(int const * xp, int const * yp,
+	void path(int const * xp, int const * yp,
 		int const * c1x, int const * c1y,
 		int const * c2x, int const * c2y,
 		int np, Color,
 		fill_style = fill_none, line_style = line_solid,
-		int line_width = thin_line);
+		int line_width = thin_line) override;
 
 	/// draw a rectangle
-	virtual void rectangle(
+	void rectangle(
 		int x, int y,
 		int w, int h,
 		Color,
 		line_style = line_solid,
-		int lw = thin_line);
+		int lw = thin_line) override;
 
 	/// draw a filled rectangle
-	virtual void fillRectangle(
+	void fillRectangle(
 		int x, int y,
 		int w, int h,
-		Color);
+		Color) override;
 
 	/// draw an arc
-	virtual void arc(
+	void arc(
 		int x, int y,
 		unsigned int w, unsigned int h,
 		int a1, int a2,
-		Color);
+		Color) override;
 
 	/// draw a pixel
-	virtual void point(int x, int y, Color);
+	void point(int x, int y, Color) override;
 
 	/// draw an image from the image cache
-	virtual void image(int x, int y, int w, int h,
-		lyx::graphics::Image const & image);
+	void image(int x, int y, int w, int h,
+		lyx::graphics::Image const & image) override;
 
 	/// draw a string at position x, y (y is the baseline).
-	virtual void text(int x, int y, docstring const & str, FontInfo const & f);
+	void text(int x, int y, docstring const & str, FontInfo const & f) override;
 
 	/// draw a char at position x, y (y is the baseline)
-	virtual void text(int x, int y, char_type c, FontInfo const & f);
+	void text(int x, int y, char_type c, FontInfo const & f) override;
 
 	/** draw a string at position x, y (y is the baseline). The
 	 * text direction is enforced by the \c Font.
 	 */
-	virtual void text(int x, int y, docstring const & str, Font const & f,
-                      double wordspacing, double textwidth);
+	void text(int x, int y, docstring const & str, Font const & f,
+                      double wordspacing, double textwidth) override;
 
 	/** draw a string at position x, y (y is the baseline), but
 	 * make sure that the part between \c from and \c to is in
 	 * \c other color. The text direction is enforced by the \c Font.
 	 */
-	virtual void text(int x, int y, docstring const & str, Font const & f,
+	void text(int x, int y, docstring const & str, Font const & f,
 	                  Color other, size_type from, size_type to,
-                      double wordspacing, double textwidth);
+                      double wordspacing, double textwidth) override;
 
 	///
-	virtual void textDecoration(FontInfo const & f, int x, int y, int width);
+	void textDecoration(FontInfo const & f, int x, int y, int width) override;
 
 	/// draw a string and enclose it inside a button frame
-	virtual void buttonText(int x, int baseline, docstring const & s,
-		FontInfo const & font, Color back, Color frame, int offset);
+	void buttonText(int x, int baseline, docstring const & s,
+		FontInfo const & font, Color back, Color frame, int offset) override;
 
 	/// start monochrome painting mode, i.e. map every color a shade of \c blend.
-	virtual void enterMonochromeMode(Color const & blend);
+	void enterMonochromeMode(Color const & blend) override;
 	/// leave monochrome painting mode
-	virtual void leaveMonochromeMode();
+	void leaveMonochromeMode() override;
 
 	/**
 	 * Draw a string and enclose it inside a rectangle. If
@@ -144,14 +144,14 @@ public:
 	 * the given color. If frame is specified, a thin frame is drawn
 	 * around the text with the given color.
 	 */
-	virtual void rectText(int x, int baseline, docstring const & str,
-		FontInfo const & font, Color back, Color frame);
+	void rectText(int x, int baseline, docstring const & str,
+		FontInfo const & font, Color back, Color frame) override;
 
 	/// draw a character of a preedit string for cjk support.
-	virtual int preeditText(int x, int y,
-		char_type c, FontInfo const & f, preedit_style style);
+	int preeditText(int x, int y,
+		char_type c, FontInfo const & f, preedit_style style) override;
 
-	void wavyHorizontalLine(int x, int y, int width, ColorCode col);
+	void wavyHorizontalLine(int x, int y, int width, ColorCode col) override;
 
 private:
 	/// check the font, and if set, draw an underline
diff --git a/src/frontends/qt/LaTeXHighlighter.h b/src/frontends/qt/LaTeXHighlighter.h
index c780b9f..47df8b9 100644
--- a/src/frontends/qt/LaTeXHighlighter.h
+++ b/src/frontends/qt/LaTeXHighlighter.h
@@ -28,7 +28,7 @@ public:
 	explicit LaTeXHighlighter(QTextDocument * parent, bool at_letter = false);
 
 protected:
-	void highlightBlock(QString const & text);
+	void highlightBlock(QString const & text) override;
 
 private:
 	QTextCharFormat commentFormat;
diff --git a/src/frontends/qt/TocModel.h b/src/frontends/qt/TocModel.h
index b2d74a2..55507a0 100644
--- a/src/frontends/qt/TocModel.h
+++ b/src/frontends/qt/TocModel.h
@@ -90,7 +90,7 @@ public:
 		: QSortFilterProxyModel(w)
 	{}
 
-	bool lessThan (const QModelIndex & left, const QModelIndex & right) const
+	bool lessThan(const QModelIndex & left, const QModelIndex & right) const override
 	{
 		if (left.model()->data(left, Qt::UserRole).toString()
 			  == QString("tableofcontents"))


More information about the lyx-cvs mailing list