Skip to content

Commit ae9b492

Browse files
authored
Merge pull request #277 from jbj/missing-return-high
C++: Make cpp/missing-return visible on LGTM again
2 parents bea86e5 + 9c0ba51 commit ae9b492

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

change-notes/1.19/analysis-cpp.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
|----------------------------|------------------------|------------------------------------------------------------------|
1515
| Resource not released in destructor | Fewer false positive results | Placement new is now excluded from the query. |
1616
| Wrong type of arguments to formatting function | Fewer false positive results | False positive results involving typedefs have been removed. |
17+
| Missing return statement (`cpp/missing-return`) | Visible by default | The precision of this query has been increased from 'medium' to 'high', which makes it visible by default in LGTM. It was 'medium' in release 1.17 and 1.18 because it had false positives due to an extractor bug that was fixed in 1.18. |
1718

1819
## Changes to QL libraries
1920

cpp/ql/src/jsf/4.13 Functions/AV Rule 114.ql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
* @description All functions that are not void should return a value on every exit path.
44
* @kind problem
55
* @problem.severity error
6-
* @precision medium
6+
* @precision high
77
* @id cpp/missing-return
88
* @tags reliability
99
* readability

0 commit comments

Comments
 (0)