Skip to content

Commit

Permalink
Fix some gcc 7.3 reported warnings
Browse files Browse the repository at this point in the history
```
src/client/joystick_controller.h:85:2: warning: duplicate 'virtual' declaration specifier [-Wduplicate-decl-specifier]
        virtual bool isTriggered(const irr::SEvent::SJoystickEvent &ev) const;
/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/7.3.1/../../../../include/c++/7.3.1/ext/new_allocator.h:140:22: warning: destructor called on non-final 'JoystickAxisCmb' that has virtual functions but non-virtual destructor [-Wdelete-non-virtual-dtor]
                destroy(_Up* __p) { __p->~_Up(); }
/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/7.3.1/../../../../include/c++/7.3.1/ext/new_allocator.h:140:22: warning: destructor called on non-final 'JoystickButtonCmb' that has virtual functions but non-virtual destructor [-Wdelete-non-virtual-dtor]
        destroy(_Up* __p) { __p->~_Up(); }
```
  • Loading branch information
nerzhul committed Mar 23, 2018
1 parent 9293d8e commit eab652f
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/client/joystick_controller.h
Expand Up @@ -60,6 +60,8 @@ struct JoystickButtonCmb : public JoystickCombination {
this->key = key;
}

virtual ~JoystickButtonCmb() = default;

virtual bool isTriggered(const irr::SEvent::SJoystickEvent &ev) const;

u32 filter_mask;
Expand All @@ -78,7 +80,9 @@ struct JoystickAxisCmb : public JoystickCombination {
this->key = key;
}

virtual bool isTriggered(const irr::SEvent::SJoystickEvent &ev) const;
virtual ~JoystickAxisCmb() = default;

bool isTriggered(const irr::SEvent::SJoystickEvent &ev) const override;

u16 axis_to_compare;

Expand Down

0 comments on commit eab652f

Please sign in to comment.