Skip to content

Commit f8ef697

Browse files
authored
Merge pull request github#13013 from MathiasVP/fix-todos-in-flow-summary
Swift: Fix TODOs in `FlowSummary.qll`
2 parents a905917 + af18c98 commit f8ef697

File tree

1 file changed

+3
-24
lines changed

1 file changed

+3
-24
lines changed

swift/ql/lib/codeql/swift/dataflow/FlowSummary.qll

Lines changed: 3 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -21,25 +21,11 @@ module SummaryComponent {
2121

2222
predicate content = SummaryComponentInternal::content/1;
2323

24-
/** Gets a summary component for parameter `i`. */
25-
SummaryComponent parameter(int i) {
26-
none() // TODO
27-
}
24+
predicate parameter = SummaryComponentInternal::parameter/1;
2825

29-
/** Gets a summary component for argument `i`. */
30-
SummaryComponent argument(int i) {
31-
none() // TODO
32-
}
26+
predicate argument = SummaryComponentInternal::argument/1;
3327

3428
predicate return = SummaryComponentInternal::return/1;
35-
36-
/** Gets a summary component that represents a qualifier. */
37-
SummaryComponent qualifier() {
38-
none() // TODO
39-
}
40-
41-
/** Gets a summary component that represents the return value of a call. */
42-
SummaryComponent return() { result = return(any(DataFlowDispatch::NormalReturnKind rk)) }
4329
}
4430

4531
class SummaryComponentStack = Impl::Public::SummaryComponentStack;
@@ -52,16 +38,9 @@ module SummaryComponentStack {
5238

5339
predicate push = SummaryComponentStackInternal::push/2;
5440

55-
/** Gets a singleton stack for argument `i`. */
56-
SummaryComponentStack argument(int i) { result = singleton(SummaryComponent::argument(i)) }
41+
predicate argument = SummaryComponentStackInternal::argument/1;
5742

5843
predicate return = SummaryComponentStackInternal::return/1;
59-
60-
/** Gets a singleton stack representing a qualifier. */
61-
SummaryComponentStack qualifier() { result = singleton(SummaryComponent::qualifier()) }
62-
63-
/** Gets a singleton stack representing the return value of a call. */
64-
SummaryComponentStack return() { result = singleton(SummaryComponent::return()) }
6544
}
6645

6746
class SummarizedCallable = Impl::Public::SummarizedCallable;

0 commit comments

Comments
 (0)