Skip to content

Commit 1ea843f

Browse files
authored
Merge pull request #20323 from asgerf/js/remove-totalorder
JS: Remove totalorder()
2 parents ab3ad20 + 45b8158 commit 1ea843f

File tree

1 file changed

+0
-58
lines changed

1 file changed

+0
-58
lines changed

javascript/ql/lib/semmle/javascript/dataflow/internal/DataFlowPrivate.qll

Lines changed: 0 additions & 58 deletions
Original file line numberDiff line numberDiff line change
@@ -408,12 +408,6 @@ class DataFlowCallable extends TDataFlowCallable {
408408

409409
/** Gets the corresponding `LibraryCallable` if this is a library callable. */
410410
LibraryCallable asLibraryCallable() { this = MkLibraryCallable(result) }
411-
412-
int totalorder() {
413-
result = TotalOrdering::astNodeId(this.asSourceCallable()).bitShiftLeft(1)
414-
or
415-
result = TotalOrdering::libraryCallableId(this.asLibraryCallable()).bitShiftLeft(1) + 1
416-
}
417411
}
418412

419413
/** A callable defined in library code, identified by a unique string. */
@@ -797,47 +791,6 @@ private newtype TDataFlowCall =
797791
FlowSummaryImpl::Private::summaryCallbackRange(c, receiver)
798792
}
799793

800-
private module TotalOrdering {
801-
private predicate astNodeRefl(AstNode x, AstNode y) { x = y }
802-
803-
int astNodeId(AstNode n) = equivalenceRelation(astNodeRefl/2)(n, result)
804-
805-
predicate dataFlowNodeId(DataFlow::Node node, int cls, int content) {
806-
exists(AstNode n |
807-
node = TValueNode(n) and cls = 1 and content = astNodeId(n)
808-
or
809-
node = TReflectiveCallNode(n, _) and cls = 2 and content = astNodeId(n)
810-
)
811-
}
812-
813-
predicate callId(DataFlowCall call, int cls, int child, int extra) {
814-
exists(DataFlow::Node node |
815-
call = MkOrdinaryCall(node) and dataFlowNodeId(node, cls - 1000, child) and extra = 0
816-
or
817-
call = MkPartialCall(node, _) and dataFlowNodeId(node, cls - 2000, child) and extra = 0
818-
or
819-
call = MkBoundCall(node, extra) and dataFlowNodeId(node, cls - 3000, child)
820-
or
821-
call = MkAccessorCall(node) and dataFlowNodeId(node, cls - 4000, child) and extra = 0
822-
)
823-
or
824-
exists(Function f |
825-
call = MkImpliedLambdaCall(f) and cls = 5000 and child = astNodeId(f) and extra = 0
826-
)
827-
or
828-
exists(
829-
FlowSummaryImpl::Public::SummarizedCallable c, FlowSummaryImpl::Private::SummaryNode receiver
830-
|
831-
call = MkSummaryCall(c, receiver) and
832-
cls = 6000 and
833-
c = rank[child](FlowSummaryImpl::Public::SummarizedCallable cs) and
834-
extra = 0
835-
)
836-
}
837-
838-
int libraryCallableId(LibraryCallable callable) { callable = rank[result](LibraryCallable c) }
839-
}
840-
841794
class DataFlowCall extends TDataFlowCall {
842795
DataFlowCallable getEnclosingCallable() { none() } // Overridden in subclass
843796

@@ -861,15 +814,6 @@ class DataFlowCall extends TDataFlowCall {
861814
}
862815

863816
Location getLocation() { none() } // Overridden in subclass
864-
865-
int totalorder() {
866-
this =
867-
rank[result](DataFlowCall call, int x, int y, int z |
868-
TotalOrdering::callId(call, x, y, z)
869-
|
870-
call order by x, y, z
871-
)
872-
}
873817
}
874818

875819
private class OrdinaryCall extends DataFlowCall, MkOrdinaryCall {
@@ -1653,8 +1597,6 @@ abstract class NodeRegion extends Unit {
16531597

16541598
/** Holds if this region contains `n`. */
16551599
predicate contains(Node n) { none() }
1656-
1657-
int totalOrder() { none() }
16581600
}
16591601

16601602
/**

0 commit comments

Comments
 (0)