-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Feat/add on rpc server started hook #4947
feat: Feat/add on rpc server started hook #4947
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is great, only a few style nits
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is great, I only made one smol change by introducing a new type
aaa |
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Hey!
@mattsseI work on #4917
I really need your help!
I prepared a new function, I use the new type RethRpcComponents but I've got a borrowing issue on the line below which I don't know how to resolve in a clear manner YET:
cannot move out of
modulesbecause it is borrowed move out of
modulesoccurs hererustc[Click for full compiler diagnostic](rust-analyzer-diagnostics-view:/diagnostic%20message%20%5B0%5D?0#file%3A%2F%2F%2FUsers%2Fnil.medvedev%2FProjects%2Fmy%2Freth%2Fbin%2Freth%2Fsrc%2Fargs%2Frpc_server_args.rs) rpc_server_args.rs(206, 82): borrow of
modulesoccurs here rpc_server_args.rs(198, 14): binding
modulesdeclared here rpc_server_args.rs(233, 54): borrow later used here
https://github.com/allnil/reth/blob/2fc27313e4b2c381514168d2731af20eaa9c3c33/bin/reth/src/args/rpc_server_args.rs#L211C6-L211C6
I played with it in different ways but still don't know how to do it better
I will continue to work on it but maybe you could just give me a hint or show me a direction to proceed?
Thank you!