Skip to content

Commit

Permalink
Gui: declare methods as override
Browse files Browse the repository at this point in the history
  • Loading branch information
wwmayer committed Apr 19, 2023
1 parent 986a0d6 commit 1dcc58c
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 8 deletions.
6 changes: 3 additions & 3 deletions src/Gui/DlgParameterFind.h
Expand Up @@ -41,10 +41,10 @@ class GuiExport DlgParameterFind : public QDialog

public:
explicit DlgParameterFind(DlgParameterImp* parent);
~DlgParameterFind();
~DlgParameterFind() override;

void accept();
void reject();
void accept() override;
void reject() override;

private:
void setupConnections();
Expand Down
11 changes: 6 additions & 5 deletions src/Gui/TextureMapping.h
Expand Up @@ -25,6 +25,7 @@

#include <Gui/TaskView/TaskDialog.h>
#include <Gui/TaskView/TaskView.h>
#include <QDialog>

class SoGroup;
class SoTexture2;
Expand All @@ -40,17 +41,17 @@ class GuiExport TextureMapping : public QDialog

public:
explicit TextureMapping(QWidget* parent = nullptr, Qt::WindowFlags fl = Qt::WindowFlags());
~TextureMapping();
void accept();
void reject();
~TextureMapping() override;
void accept() override;
void reject() override;

private:
void onFileChooserFileNameSelected(const QString&);
void onCheckEnvToggled(bool);

protected:
void changeEvent(QEvent *e);
void keyPressEvent(QKeyEvent *e);
void changeEvent(QEvent *e) override;

Check warning on line 53 in src/Gui/TextureMapping.h

View workflow job for this annotation

GitHub Actions / Lint / Lint

parameter name 'e' is too short, expected at least 2 characters [readability-identifier-length]
void keyPressEvent(QKeyEvent *e) override;

Check warning on line 54 in src/Gui/TextureMapping.h

View workflow job for this annotation

GitHub Actions / Lint / Lint

parameter name 'e' is too short, expected at least 2 characters [readability-identifier-length]

private:
SoGroup* grp;
Expand Down

0 comments on commit 1dcc58c

Please sign in to comment.