Skip to content

Commit 249d209

Browse files
authored
QL: fix
1 parent e456098 commit 249d209

2 files changed

Lines changed: 9 additions & 1 deletion

File tree

ql/src/codeql_ql/ast/Ast.qll

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -288,8 +288,16 @@ class ClassPredicate extends TClassPredicate, Predicate {
288288

289289
override Class getParent() { result.getAClassPredicate() = this }
290290

291+
/**
292+
* Holds if this predicate is private.
293+
*/
291294
predicate isPrivate() { hasAnnotation(this, "private") }
292295

296+
/**
297+
* Holds if this predicate is annotated as overriding another predicate.
298+
*/
299+
predicate isOverride() { hasAnnotation(this, "override") }
300+
293301
override VarDecl getParameter(int i) {
294302
toGenerated(result) =
295303
rank[i](Generated::VarDecl decl, int index | decl = pred.getChild(index) | decl order by index)

ql/src/queries/style/MissingOverride.ql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,6 @@ string getQualifiedName(ClassPredicate p) {
1515
}
1616

1717
from ClassPredicate pred, ClassPredicate sup
18-
where pred.overrides(sup) and not pred.hasAnnotation("override")
18+
where pred.overrides(sup) and not pred.isOverride()
1919
select pred, getQualifiedName(pred) + "overrides $@ but does not have an override annotation.", sup,
2020
getQualifiedName(sup)

0 commit comments

Comments
 (0)