Skip to content

Commit fab8bb6

Browse files
sanityclaude
andcommitted
refactor: remove unused location field from InboundConnection event
As suggested in PR review, the location field was marked with #[allow(dead_code)] and not being used. The connection is only added to ConnectionManager after StartJoinReq is accepted, not when InboundConnection event is processed. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 5eb0396 commit fab8bb6

File tree

2 files changed

+0
-4
lines changed

2 files changed

+0
-4
lines changed

crates/core/src/node/network_bridge/handshake.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,6 @@ pub(super) enum Event {
6262
id: Transaction,
6363
conn: PeerConnection,
6464
joiner: PeerId,
65-
#[allow(dead_code)]
66-
location: Location,
6765
op: Option<Box<ConnectOp>>,
6866
forward_info: Option<Box<ForwardInfo>>,
6967
},
@@ -463,7 +461,6 @@ impl HandshakeHandler {
463461
id,
464462
conn,
465463
joiner,
466-
location,
467464
op: ok.map(|ok_value| Box::new(ConnectOp::new(id, Some(ok_value), None, None))),
468465
forward_info: forward_info.map(Box::new),
469466
})

crates/core/src/node/network_bridge/p2p_protoc.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -782,7 +782,6 @@ impl P2pConnManager {
782782
id,
783783
conn,
784784
joiner,
785-
location: _, // Not used - connection only added after StartJoinReq accepted
786785
op,
787786
forward_info,
788787
} => {

0 commit comments

Comments
 (0)