mirror of
https://github.com/github/codeql.git
synced 2025-12-16 16:53:25 +01:00
Update alert messages
This commit is contained in:
@@ -25,11 +25,11 @@ where
|
||||
(
|
||||
// Simple case: the method that should be called is directly overridden
|
||||
mroStart = base.getScope() and
|
||||
msg = "This deletion method does not call $@, which may leave $@ not properly cleaned up."
|
||||
msg = "This delete method does not call $@, which may leave $@ not properly cleaned up."
|
||||
or
|
||||
// Only alert for a different mro base if there are no alerts for direct overrides
|
||||
not missingCallToSuperclassDel(base, _, base.getScope()) and
|
||||
msg =
|
||||
"This deletion method does not call $@, which follows it in the MRO of $@, leaving it not properly cleaned up."
|
||||
"This delete method does not call super().__del__, which may cause $@ to be missed during the cleanup of $@."
|
||||
)
|
||||
select base, msg, shouldCall, shouldCall.getQualifiedName(), mroStart, mroStart.getName()
|
||||
|
||||
@@ -29,6 +29,6 @@ where
|
||||
// Only alert for a different mro base if there are no alerts for direct overrides
|
||||
not missingCallToSuperclassInit(base, _, base.getScope()) and
|
||||
msg =
|
||||
"This initialization method does not call $@, which follows it in the MRO of $@, leaving it partially initialized."
|
||||
"This initialization method does not call super().__init__, which may cause $@ to be missed during the initialization of $@."
|
||||
)
|
||||
select base, msg, shouldCall, shouldCall.getQualifiedName(), mroStart, mroStart.getName()
|
||||
|
||||
@@ -26,5 +26,5 @@ where
|
||||
multipleCallsToSuperclassInit(meth, subMulti) and
|
||||
calledMulti.getScope() = getADirectSuperclass+(subMulti.getScope())
|
||||
)
|
||||
select meth, "This initializer method calls $@ multiple times.", calledMulti,
|
||||
select meth, "This initialization method calls $@ multiple times.", calledMulti,
|
||||
calledMulti.getQualifiedName()
|
||||
|
||||
Reference in New Issue
Block a user