Skip to content

Commit 46b68ea

Browse files
acdliteacdlite
authored
Delete LanePriority type (#21090)
No longer using LanePriority anywhere, so this deletes the remaining references.
1 parent dcd1304 commit 46b68ea

File tree

2 files changed

+0
-114
lines changed

2 files changed

+0
-114
lines changed

‎packages/react-reconciler/src/ReactFiberLane.new.js

-57
Original file line numberDiff line numberDiff line change
@@ -37,28 +37,6 @@ export type LaneMap<T> = Array<T>;
3737

3838
import {enableCache, enableSchedulingProfiler} from 'shared/ReactFeatureFlags';
3939

40-
export const SyncLanePriority: LanePriority = 12;
41-
42-
const InputContinuousHydrationLanePriority: LanePriority = 11;
43-
export const InputContinuousLanePriority: LanePriority = 10;
44-
45-
const DefaultHydrationLanePriority: LanePriority = 9;
46-
export const DefaultLanePriority: LanePriority = 8;
47-
48-
const TransitionHydrationPriority: LanePriority = 7;
49-
export const TransitionPriority: LanePriority = 6;
50-
51-
const RetryLanePriority: LanePriority = 5;
52-
53-
const SelectiveHydrationLanePriority: LanePriority = 4;
54-
55-
const IdleHydrationLanePriority: LanePriority = 3;
56-
export const IdleLanePriority: LanePriority = 2;
57-
58-
const OffscreenLanePriority: LanePriority = 1;
59-
60-
export const NoLanePriority: LanePriority = 0;
61-
6240
// Lane values below should be kept in sync with getLabelsForLanes(), used by react-devtools-scheduling-profiler.
6341
// If those values are changed that package should be rebuilt and redeployed.
6442

@@ -162,29 +140,19 @@ export const NoTimestamp = -1;
162140
let nextTransitionLane: Lane = TransitionLane1;
163141
let nextRetryLane: Lane = RetryLane1;
164142

165-
// "Registers" used to "return" multiple values
166-
// Used by getHighestPriorityLanes and getNextLanes:
167-
let return_highestLanePriority: LanePriority = DefaultLanePriority;
168-
169143
function getHighestPriorityLanes(lanes: Lanes | Lane): Lanes {
170144
switch (getHighestPriorityLane(lanes)) {
171145
case SyncLane:
172-
return_highestLanePriority = SyncLanePriority;
173146
return SyncLane;
174147
case InputContinuousHydrationLane:
175-
return_highestLanePriority = InputContinuousHydrationLanePriority;
176148
return InputContinuousHydrationLane;
177149
case InputContinuousLane:
178-
return_highestLanePriority = InputContinuousLanePriority;
179150
return InputContinuousLane;
180151
case DefaultHydrationLane:
181-
return_highestLanePriority = DefaultHydrationLanePriority;
182152
return DefaultHydrationLane;
183153
case DefaultLane:
184-
return_highestLanePriority = DefaultLanePriority;
185154
return DefaultLane;
186155
case TransitionHydrationLane:
187-
return_highestLanePriority = TransitionHydrationPriority;
188156
return TransitionHydrationLane;
189157
case TransitionLane1:
190158
case TransitionLane2:
@@ -202,26 +170,20 @@ function getHighestPriorityLanes(lanes: Lanes | Lane): Lanes {
202170
case TransitionLane14:
203171
case TransitionLane15:
204172
case TransitionLane16:
205-
return_highestLanePriority = TransitionPriority;
206173
return lanes & TransitionLanes;
207174
case RetryLane1:
208175
case RetryLane2:
209176
case RetryLane3:
210177
case RetryLane4:
211178
case RetryLane5:
212-
return_highestLanePriority = RetryLanePriority;
213179
return lanes & RetryLanes;
214180
case SelectiveHydrationLane:
215-
return_highestLanePriority = SelectiveHydrationLanePriority;
216181
return SelectiveHydrationLane;
217182
case IdleHydrationLane:
218-
return_highestLanePriority = IdleHydrationLanePriority;
219183
return IdleHydrationLane;
220184
case IdleLane:
221-
return_highestLanePriority = IdleLanePriority;
222185
return IdleLane;
223186
case OffscreenLane:
224-
return_highestLanePriority = OffscreenLanePriority;
225187
return OffscreenLane;
226188
default:
227189
if (__DEV__) {
@@ -230,7 +192,6 @@ function getHighestPriorityLanes(lanes: Lanes | Lane): Lanes {
230192
);
231193
}
232194
// This shouldn't be reachable, but as a fallback, return the entire bitmask.
233-
return_highestLanePriority = DefaultLanePriority;
234195
return lanes;
235196
}
236197
}
@@ -239,12 +200,10 @@ export function getNextLanes(root: FiberRoot, wipLanes: Lanes): Lanes {
239200
// Early bailout if there's no pending work left.
240201
const pendingLanes = root.pendingLanes;
241202
if (pendingLanes === NoLanes) {
242-
return_highestLanePriority = NoLanePriority;
243203
return NoLanes;
244204
}
245205

246206
let nextLanes = NoLanes;
247-
let nextLanePriority = NoLanePriority;
248207

249208
const suspendedLanes = root.suspendedLanes;
250209
const pingedLanes = root.pingedLanes;
@@ -256,24 +215,20 @@ export function getNextLanes(root: FiberRoot, wipLanes: Lanes): Lanes {
256215
const nonIdleUnblockedLanes = nonIdlePendingLanes & ~suspendedLanes;
257216
if (nonIdleUnblockedLanes !== NoLanes) {
258217
nextLanes = getHighestPriorityLanes(nonIdleUnblockedLanes);
259-
nextLanePriority = return_highestLanePriority;
260218
} else {
261219
const nonIdlePingedLanes = nonIdlePendingLanes & pingedLanes;
262220
if (nonIdlePingedLanes !== NoLanes) {
263221
nextLanes = getHighestPriorityLanes(nonIdlePingedLanes);
264-
nextLanePriority = return_highestLanePriority;
265222
}
266223
}
267224
} else {
268225
// The only remaining work is Idle.
269226
const unblockedLanes = pendingLanes & ~suspendedLanes;
270227
if (unblockedLanes !== NoLanes) {
271228
nextLanes = getHighestPriorityLanes(unblockedLanes);
272-
nextLanePriority = return_highestLanePriority;
273229
} else {
274230
if (pingedLanes !== NoLanes) {
275231
nextLanes = getHighestPriorityLanes(pingedLanes);
276-
nextLanePriority = return_highestLanePriority;
277232
}
278233
}
279234
}
@@ -307,8 +262,6 @@ export function getNextLanes(root: FiberRoot, wipLanes: Lanes): Lanes {
307262
) {
308263
// Keep working on the existing in-progress tree. Do not interrupt.
309264
return wipLanes;
310-
} else {
311-
return_highestLanePriority = nextLanePriority;
312265
}
313266
}
314267

@@ -490,9 +443,6 @@ export function getLanesToRetrySynchronouslyOnError(root: FiberRoot): Lanes {
490443
return NoLanes;
491444
}
492445

493-
export function returnNextLanesPriority() {
494-
return return_highestLanePriority;
495-
}
496446
export function includesNonIdleWork(lanes: Lanes) {
497447
return (lanes & NonIdleLanes) !== NoLanes;
498448
}
@@ -579,13 +529,6 @@ export function higherPriorityLane(a: Lane, b: Lane) {
579529
return a !== NoLane && a < b ? a : b;
580530
}
581531

582-
export function higherLanePriority(
583-
a: LanePriority,
584-
b: LanePriority,
585-
): LanePriority {
586-
return a !== NoLanePriority && a > b ? a : b;
587-
}
588-
589532
export function createLaneMap<T>(initial: T): LaneMap<T> {
590533
// Intentionally pushing one by one.
591534
// https://v8.dev/blog/elements-kinds#avoid-creating-holes

‎packages/react-reconciler/src/ReactFiberLane.old.js

-57
Original file line numberDiff line numberDiff line change
@@ -37,28 +37,6 @@ export type LaneMap<T> = Array<T>;
3737

3838
import {enableCache, enableSchedulingProfiler} from 'shared/ReactFeatureFlags';
3939

40-
export const SyncLanePriority: LanePriority = 12;
41-
42-
const InputContinuousHydrationLanePriority: LanePriority = 11;
43-
export const InputContinuousLanePriority: LanePriority = 10;
44-
45-
const DefaultHydrationLanePriority: LanePriority = 9;
46-
export const DefaultLanePriority: LanePriority = 8;
47-
48-
const TransitionHydrationPriority: LanePriority = 7;
49-
export const TransitionPriority: LanePriority = 6;
50-
51-
const RetryLanePriority: LanePriority = 5;
52-
53-
const SelectiveHydrationLanePriority: LanePriority = 4;
54-
55-
const IdleHydrationLanePriority: LanePriority = 3;
56-
export const IdleLanePriority: LanePriority = 2;
57-
58-
const OffscreenLanePriority: LanePriority = 1;
59-
60-
export const NoLanePriority: LanePriority = 0;
61-
6240
// Lane values below should be kept in sync with getLabelsForLanes(), used by react-devtools-scheduling-profiler.
6341
// If those values are changed that package should be rebuilt and redeployed.
6442

@@ -162,29 +140,19 @@ export const NoTimestamp = -1;
162140
let nextTransitionLane: Lane = TransitionLane1;
163141
let nextRetryLane: Lane = RetryLane1;
164142

165-
// "Registers" used to "return" multiple values
166-
// Used by getHighestPriorityLanes and getNextLanes:
167-
let return_highestLanePriority: LanePriority = DefaultLanePriority;
168-
169143
function getHighestPriorityLanes(lanes: Lanes | Lane): Lanes {
170144
switch (getHighestPriorityLane(lanes)) {
171145
case SyncLane:
172-
return_highestLanePriority = SyncLanePriority;
173146
return SyncLane;
174147
case InputContinuousHydrationLane:
175-
return_highestLanePriority = InputContinuousHydrationLanePriority;
176148
return InputContinuousHydrationLane;
177149
case InputContinuousLane:
178-
return_highestLanePriority = InputContinuousLanePriority;
179150
return InputContinuousLane;
180151
case DefaultHydrationLane:
181-
return_highestLanePriority = DefaultHydrationLanePriority;
182152
return DefaultHydrationLane;
183153
case DefaultLane:
184-
return_highestLanePriority = DefaultLanePriority;
185154
return DefaultLane;
186155
case TransitionHydrationLane:
187-
return_highestLanePriority = TransitionHydrationPriority;
188156
return TransitionHydrationLane;
189157
case TransitionLane1:
190158
case TransitionLane2:
@@ -202,26 +170,20 @@ function getHighestPriorityLanes(lanes: Lanes | Lane): Lanes {
202170
case TransitionLane14:
203171
case TransitionLane15:
204172
case TransitionLane16:
205-
return_highestLanePriority = TransitionPriority;
206173
return lanes & TransitionLanes;
207174
case RetryLane1:
208175
case RetryLane2:
209176
case RetryLane3:
210177
case RetryLane4:
211178
case RetryLane5:
212-
return_highestLanePriority = RetryLanePriority;
213179
return lanes & RetryLanes;
214180
case SelectiveHydrationLane:
215-
return_highestLanePriority = SelectiveHydrationLanePriority;
216181
return SelectiveHydrationLane;
217182
case IdleHydrationLane:
218-
return_highestLanePriority = IdleHydrationLanePriority;
219183
return IdleHydrationLane;
220184
case IdleLane:
221-
return_highestLanePriority = IdleLanePriority;
222185
return IdleLane;
223186
case OffscreenLane:
224-
return_highestLanePriority = OffscreenLanePriority;
225187
return OffscreenLane;
226188
default:
227189
if (__DEV__) {
@@ -230,7 +192,6 @@ function getHighestPriorityLanes(lanes: Lanes | Lane): Lanes {
230192
);
231193
}
232194
// This shouldn't be reachable, but as a fallback, return the entire bitmask.
233-
return_highestLanePriority = DefaultLanePriority;
234195
return lanes;
235196
}
236197
}
@@ -239,12 +200,10 @@ export function getNextLanes(root: FiberRoot, wipLanes: Lanes): Lanes {
239200
// Early bailout if there's no pending work left.
240201
const pendingLanes = root.pendingLanes;
241202
if (pendingLanes === NoLanes) {
242-
return_highestLanePriority = NoLanePriority;
243203
return NoLanes;
244204
}
245205

246206
let nextLanes = NoLanes;
247-
let nextLanePriority = NoLanePriority;
248207

249208
const suspendedLanes = root.suspendedLanes;
250209
const pingedLanes = root.pingedLanes;
@@ -256,24 +215,20 @@ export function getNextLanes(root: FiberRoot, wipLanes: Lanes): Lanes {
256215
const nonIdleUnblockedLanes = nonIdlePendingLanes & ~suspendedLanes;
257216
if (nonIdleUnblockedLanes !== NoLanes) {
258217
nextLanes = getHighestPriorityLanes(nonIdleUnblockedLanes);
259-
nextLanePriority = return_highestLanePriority;
260218
} else {
261219
const nonIdlePingedLanes = nonIdlePendingLanes & pingedLanes;
262220
if (nonIdlePingedLanes !== NoLanes) {
263221
nextLanes = getHighestPriorityLanes(nonIdlePingedLanes);
264-
nextLanePriority = return_highestLanePriority;
265222
}
266223
}
267224
} else {
268225
// The only remaining work is Idle.
269226
const unblockedLanes = pendingLanes & ~suspendedLanes;
270227
if (unblockedLanes !== NoLanes) {
271228
nextLanes = getHighestPriorityLanes(unblockedLanes);
272-
nextLanePriority = return_highestLanePriority;
273229
} else {
274230
if (pingedLanes !== NoLanes) {
275231
nextLanes = getHighestPriorityLanes(pingedLanes);
276-
nextLanePriority = return_highestLanePriority;
277232
}
278233
}
279234
}
@@ -307,8 +262,6 @@ export function getNextLanes(root: FiberRoot, wipLanes: Lanes): Lanes {
307262
) {
308263
// Keep working on the existing in-progress tree. Do not interrupt.
309264
return wipLanes;
310-
} else {
311-
return_highestLanePriority = nextLanePriority;
312265
}
313266
}
314267

@@ -490,9 +443,6 @@ export function getLanesToRetrySynchronouslyOnError(root: FiberRoot): Lanes {
490443
return NoLanes;
491444
}
492445

493-
export function returnNextLanesPriority() {
494-
return return_highestLanePriority;
495-
}
496446
export function includesNonIdleWork(lanes: Lanes) {
497447
return (lanes & NonIdleLanes) !== NoLanes;
498448
}
@@ -579,13 +529,6 @@ export function higherPriorityLane(a: Lane, b: Lane) {
579529
return a !== NoLane && a < b ? a : b;
580530
}
581531

582-
export function higherLanePriority(
583-
a: LanePriority,
584-
b: LanePriority,
585-
): LanePriority {
586-
return a !== NoLanePriority && a > b ? a : b;
587-
}
588-
589532
export function createLaneMap<T>(initial: T): LaneMap<T> {
590533
// Intentionally pushing one by one.
591534
// https://v8.dev/blog/elements-kinds#avoid-creating-holes

0 commit comments

Comments
 (0)