Skip to content

feat(*): remove redundant ret for transaction #5839

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

Merged

Conversation

lxcmyf
Copy link
Contributor

@lxcmyf lxcmyf commented May 30, 2024

What does this PR do?
remove redundant ret for transaction
Why are these changes required?

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@lxcmyf lxcmyf force-pushed the feature/remove_redundant_ret branch 2 times, most recently from 1d5085b to daf2943 Compare May 30, 2024 08:41
@lxcmyf lxcmyf force-pushed the feature/remove_redundant_ret branch 2 times, most recently from 23b1fb9 to b1b6c0c Compare May 30, 2024 09:05
@lxcmyf lxcmyf force-pushed the feature/remove_redundant_ret branch from b1b6c0c to 271e3d0 Compare May 30, 2024 09:06
…emove_redundant_ret

# Conflicts:
#	common/src/main/java/org/tron/core/Constant.java
#	framework/src/main/java/org/tron/core/db/Manager.java
#	framework/src/test/java/org/tron/core/db/ManagerTest.java
@lvs007 lvs007 merged commit fcae37a into tronprotocol:release_v4.7.5 May 30, 2024
5 checks passed
Copy link

@Sr2g4u2CMe1 Sr2g4u2CMe1 left a comment

Choose a reason for hiding this comment

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

Uploading 17187770889205191642704792131882.jpg…

@zeusoo001 zeusoo001 deleted the feature/remove_redundant_ret branch June 25, 2024 02:56
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.

4 participants