Skip to content

Conversation

@hero5512
Copy link
Contributor

@hero5512 hero5512 commented Jun 9, 2023

It seems that there is no error return when marshal block to rpc type. Can we remove it?

@fjl fjl changed the title internal/ethapi: rm error when marshal block to rpc type internal/ethapi: remove error return value of RPCMarshalBlock Jun 9, 2023
@hero5512 hero5512 requested a review from zsfelfoldi as a code owner June 12, 2023 19:23
@hero5512 hero5512 closed this Jun 12, 2023
@hero5512 hero5512 reopened this Jun 12, 2023
Copy link
Contributor

@holiman holiman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@holiman holiman added this to the 1.12.1 milestone Jun 13, 2023
@holiman holiman merged commit 5ac4da3 into ethereum:master Jun 13, 2023
devopsbo3 pushed a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
devopsbo3 added a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
devopsbo3 added a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Sep 29, 2025
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Sep 29, 2025
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Sep 30, 2025
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Oct 8, 2025
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Oct 8, 2025
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Oct 30, 2025
AnilChinchawale pushed a commit to XinFinOrg/XDPoSChain that referenced this pull request Oct 31, 2025
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.

3 participants