Skip to content

fix: correct duplicate field in SharedState Debug impl#24

Merged
maurges merged 1 commit into
LFDT-Lockness:mfrom
phraakture:debug
Jun 11, 2026
Merged

fix: correct duplicate field in SharedState Debug impl#24
maurges merged 1 commit into
LFDT-Lockness:mfrom
phraakture:debug

Conversation

@phraakture

@phraakture phraakture commented May 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Closes #23

Befor fix

Debug impl was printing:
.field("wants_recv_msg", &self.wants_recv_msg)
.field("wants_recv_msg", &self.wants_recv_msg)
.field("yielded", &self.yielded)
This prints wants_recv_msg twice and skips wants_send_msg.

Fix

Replace second wants_recv_msg with wants_send_msg.

Signed-off-by: acegikmoo <rubairakib11@gmail.com>
PROWLERx15

This comment was marked as low quality.

@maurges

maurges commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Looks good, I'll have to close and reopen it to run the CI

@maurges maurges closed this Jun 11, 2026
@maurges maurges reopened this Jun 11, 2026
@maurges maurges merged commit 76e74a9 into LFDT-Lockness:m Jun 11, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Duplicate field in Debug impl, missing wants_send_msg in SharedState

3 participants