Skip to content

Commit 72ae4b6

Browse files
committed
clang-tidy: modernize-use-override (NFC)
Add NOLINT to CommandTestBase due to gmock not adding "override" keyword, which causes warnings on clang.
1 parent 3851ba3 commit 72ae4b6

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

include/frc2/command/Command.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ class ProxyScheduleCommand;
4949
class Command : public frc::ErrorBase {
5050
public:
5151
Command() = default;
52-
virtual ~Command();
52+
~Command() override;
5353

5454
Command(const Command&);
5555
Command& operator=(const Command&);

include/frc2/command/CommandScheduler.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ class CommandScheduler final : public frc::Sendable,
3939
*/
4040
static CommandScheduler& GetInstance();
4141

42-
~CommandScheduler();
42+
~CommandScheduler() override;
4343
CommandScheduler(const CommandScheduler&) = delete;
4444
CommandScheduler& operator=(const CommandScheduler&) = delete;
4545

include/frc2/command/RunCommand.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ class RunCommand : public CommandHelper<CommandBase, RunCommand> {
4545

4646
RunCommand(const RunCommand& other) = default;
4747

48-
void Execute();
48+
void Execute() override;
4949

5050
protected:
5151
std::function<void()> m_toRun;

0 commit comments

Comments
 (0)