diff options
| author | Daniel Bertalan <dani@danielbertalan.dev> | 2021-12-04 10:09:09 +0100 |
|---|---|---|
| committer | Brian Gianforcaro <b.gianfo@gmail.com> | 2021-12-11 13:14:15 -0800 |
| commit | 4a81b33c07cbb440c7fefcc073e94e113feeac52 (patch) | |
| tree | 8cad5817266b1a99a484b95338d7992f2c147e6a /Userland/Shell | |
| parent | 813593a485671e78f17c4e79f1ce12c31dd35aa6 (diff) | |
| download | serenity-4a81b33c07cbb440c7fefcc073e94e113feeac52.zip | |
Everywhere: Fix -Winconsistent-missing-override warnings from Clang
This option is already enabled when building Lagom, so let's enable it
for the main build too. We will no longer be surprised by Lagom Clang
CI builds failing while everything compiles locally.
Furthermore, the stronger `-Wsuggest-override` warning is enabled in
this commit, which enforces the use of the `override` keyword in all
classes, not just those which already have some methods marked as
`override`. This works with both GCC and Clang.
Diffstat (limited to 'Userland/Shell')
| -rw-r--r-- | Userland/Shell/Shell.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Userland/Shell/Shell.cpp b/Userland/Shell/Shell.cpp index 2d24d5c811..ef4979a888 100644 --- a/Userland/Shell/Shell.cpp +++ b/Userland/Shell/Shell.cpp @@ -1671,7 +1671,7 @@ void Shell::bring_cursor_to_beginning_of_a_line() const bool Shell::has_history_event(StringView source) { struct : public AST::NodeVisitor { - virtual void visit(const AST::HistoryEvent* node) + virtual void visit(const AST::HistoryEvent* node) override { has_history_event = true; AST::NodeVisitor::visit(node); |