summaryrefslogtreecommitdiff
path: root/Userland/Libraries/LibSQL
diff options
context:
space:
mode:
authorDaniel Bertalan <dani@danielbertalan.dev>2021-12-04 10:09:09 +0100
committerBrian Gianforcaro <b.gianfo@gmail.com>2021-12-11 13:14:15 -0800
commit4a81b33c07cbb440c7fefcc073e94e113feeac52 (patch)
tree8cad5817266b1a99a484b95338d7992f2c147e6a /Userland/Libraries/LibSQL
parent813593a485671e78f17c4e79f1ce12c31dd35aa6 (diff)
downloadserenity-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/Libraries/LibSQL')
-rw-r--r--Userland/Libraries/LibSQL/AST/AST.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/Userland/Libraries/LibSQL/AST/AST.h b/Userland/Libraries/LibSQL/AST/AST.h
index b0ee4c7262..7dbcb7c389 100644
--- a/Userland/Libraries/LibSQL/AST/AST.h
+++ b/Userland/Libraries/LibSQL/AST/AST.h
@@ -943,7 +943,7 @@ public:
bool has_selection() const { return !m_select_statement.is_null(); }
const RefPtr<Select>& select_statement() const { return m_select_statement; }
- RefPtr<SQLResult> execute(ExecutionContext&) const;
+ virtual RefPtr<SQLResult> execute(ExecutionContext&) const override;
private:
RefPtr<CommonTableExpressionList> m_common_table_expression_list;