Skip to content

Commit baa646e

Browse files
committed
Ruby: Remove unused UnknownMember from API graphs
1 parent c307a12 commit baa646e

File tree

1 file changed

+1
-23
lines changed

1 file changed

+1
-23
lines changed

ruby/ql/lib/codeql/ruby/ApiGraphs.qll

Lines changed: 1 addition & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -180,20 +180,11 @@ module API {
180180
bindingset[result]
181181
Node getMember(string m) { result = this.getASuccessor(Label::member(m)) }
182182

183-
/**
184-
* Gets a node representing a member of this API component where the name of the member is
185-
* not known statically.
186-
*/
187-
Node getUnknownMember() { result = this.getASuccessor(Label::unknownMember()) }
188-
189183
/**
190184
* Gets a node representing a member of this API component where the name of the member may
191185
* or may not be known statically.
192186
*/
193-
Node getAMember() {
194-
result = this.getASuccessor(Label::member(_)) or
195-
result = this.getUnknownMember()
196-
}
187+
Node getAMember() { result = this.getASuccessor(Label::member(_)) }
197188

198189
/**
199190
* Gets a node representing an instance of this API component, that is, an object whose
@@ -718,10 +709,6 @@ module API {
718709
or
719710
succ = MkMethodAccessNode(entry.getACall())
720711
)
721-
or
722-
// to get rid of an incompatible types error in `getUnknownMember`
723-
lbl = Label::unknownMember() and
724-
none()
725712
}
726713

727714
/**
@@ -737,7 +724,6 @@ module API {
737724
cached
738725
newtype TLabel =
739726
MkLabelMember(string member) { member = any(ConstantReadAccess a).getName() } or
740-
MkLabelUnknownMember() or
741727
MkLabelMethod(string m) { m = any(DataFlow::CallNode c).getMethodName() } or
742728
MkLabelReturn() or
743729
MkLabelSubclass() or
@@ -780,11 +766,6 @@ module API {
780766
override string toString() { result = "getMember(\"" + member + "\")" }
781767
}
782768

783-
/** A label for a member with an unknown name. */
784-
class LabelUnknownMember extends ApiLabel, MkLabelUnknownMember {
785-
override string toString() { result = "getUnknownMember()" }
786-
}
787-
788769
/** A label for a method. */
789770
class LabelMethod extends ApiLabel, MkLabelMethod {
790771
private string method;
@@ -852,9 +833,6 @@ module API {
852833
/** Gets the `member` edge label for member `m`. */
853834
LabelMember member(string m) { result.getMember() = m }
854835

855-
/** Gets the `member` edge label for the unknown member. */
856-
LabelUnknownMember unknownMember() { any() }
857-
858836
/** Gets the `method` edge label. */
859837
LabelMethod method(string m) { result.getMethod() = m }
860838

0 commit comments

Comments
 (0)