Skip to content

Commit 58fcbc1

Browse files
committed
Ruby: Re-factor getComponent.
1 parent 811eee1 commit 58fcbc1

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

ruby/ql/lib/codeql/ruby/dataflow/internal/FlowSummaryImpl.qll

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -166,15 +166,10 @@ module Public {
166166
SummaryComponentStack return(ReturnKind rk) { result = singleton(SummaryComponent::return(rk)) }
167167
}
168168

169-
private predicate noComponentSpecific(SummaryComponent sc) {
170-
not exists(getComponentSpecific(sc))
171-
}
172-
173169
/** Gets a textual representation of this component used for flow summaries. */
174170
private string getComponent(SummaryComponent sc) {
175171
result = getComponentSpecific(sc)
176172
or
177-
noComponentSpecific(sc) and
178173
(
179174
exists(ArgumentPosition pos |
180175
sc = TParameterSummaryComponent(pos) and
@@ -185,9 +180,9 @@ module Public {
185180
sc = TArgumentSummaryComponent(pos) and
186181
result = "Argument[" + getParameterPosition(pos) + "]"
187182
)
188-
or
189-
sc = TReturnSummaryComponent(getReturnValueKind()) and result = "ReturnValue"
190183
)
184+
or
185+
sc = TReturnSummaryComponent(getReturnValueKind()) and result = "ReturnValue"
191186
}
192187

193188
/** Gets a textual representation of this stack used for flow summaries. */

0 commit comments

Comments
 (0)