Skip to content

Commit

Permalink
Merge pull request #2224 from cloudflare/jlee/remove-unused-waituntil
Browse files Browse the repository at this point in the history
Remove unused waitUntilTasks fields
  • Loading branch information
jclee authored Jun 6, 2024
2 parents b5b54a9 + 439a5f8 commit ba344bc
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 5 deletions.
4 changes: 2 additions & 2 deletions src/workerd/api/hibernatable-web-socket.c++
Original file line number Diff line number Diff line change
Expand Up @@ -213,13 +213,13 @@ HibernatableWebSocketCustomEventImpl::HibernatableWebSocketCustomEventImpl(
kj::TaskSet& waitUntilTasks,
kj::Own<HibernationReader> params,
kj::Maybe<Worker::Actor::HibernationManager&> manager)
: typeId(typeId), waitUntilTasks(waitUntilTasks), params(kj::mv(params)) {}
: typeId(typeId), params(kj::mv(params)) {}
HibernatableWebSocketCustomEventImpl::HibernatableWebSocketCustomEventImpl(
uint16_t typeId,
kj::TaskSet& waitUntilTasks,
HibernatableSocketParams params,
Worker::Actor::HibernationManager& manager)
: typeId(typeId), waitUntilTasks(waitUntilTasks), params(kj::mv(params)), manager(manager) {}
: typeId(typeId), params(kj::mv(params)), manager(manager) {}

HibernatableSocketParams HibernatableWebSocketCustomEventImpl::consumeParams() {
KJ_IF_SOME(p, params.tryGet<kj::Own<HibernationReader>>()) {
Expand Down
1 change: 0 additions & 1 deletion src/workerd/api/hibernatable-web-socket.h
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,6 @@ class HibernatableWebSocketCustomEventImpl final: public WorkerInterface::Custom
HibernatableSocketParams consumeParams();

uint16_t typeId;
kj::TaskSet& waitUntilTasks;
kj::OneOf<HibernatableSocketParams, kj::Own<HibernationReader>> params;
kj::Maybe<uint32_t> timeoutMs;
kj::Maybe<Worker::Actor::HibernationManager&> manager;
Expand Down
3 changes: 1 addition & 2 deletions src/workerd/api/trace.h
Original file line number Diff line number Diff line change
Expand Up @@ -589,7 +589,7 @@ class TraceCustomEventImpl final: public WorkerInterface::CustomEvent {
public:
TraceCustomEventImpl(
uint16_t typeId, kj::TaskSet& waitUntilTasks, kj::Array<kj::Own<Trace>> traces)
: typeId(typeId), waitUntilTasks(waitUntilTasks), traces(kj::mv(traces)) {}
: typeId(typeId), traces(kj::mv(traces)) {}

kj::Promise<Result> run(
kj::Own<IoContext::IncomingRequest> incomingRequest,
Expand All @@ -608,7 +608,6 @@ class TraceCustomEventImpl final: public WorkerInterface::CustomEvent {

private:
uint16_t typeId;
kj::TaskSet& waitUntilTasks;
kj::Array<kj::Own<workerd::Trace>> traces;
};

Expand Down

0 comments on commit ba344bc

Please sign in to comment.