Skip to content

feat(net): optimize validSignature function#5159

Merged
xxo1shine merged 1 commit intotronprotocol:release_v4.7.2from
xxo1shine:optimize_sign
Apr 18, 2023
Merged

feat(net): optimize validSignature function#5159
xxo1shine merged 1 commit intotronprotocol:release_v4.7.2from
xxo1shine:optimize_sign

Conversation

@xxo1shine
Copy link
Copy Markdown
Collaborator

What does this PR do?
optimize validSignature function

Why are these changes required?

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 18, 2023

Codecov Report

Merging #5159 (9d5d0e0) into release_v4.7.2 (0a3f2aa) will increase coverage by 0.00%.
The diff coverage is 0.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@                Coverage Diff                @@
##             release_v4.7.2    #5159   +/-   ##
=================================================
  Coverage             55.81%   55.82%           
- Complexity             8357     8361    +4     
=================================================
  Files                   836      836           
  Lines                 49592    49593    +1     
  Branches               5517     5518    +1     
=================================================
+ Hits                  27682    27684    +2     
  Misses                19693    19693           
+ Partials               2217     2216    -1     
Impacted Files Coverage Δ
...c/main/java/org/tron/core/net/TronNetDelegate.java 41.26% <0.00%> (-0.34%) ⬇️

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@xxo1shine xxo1shine merged commit 3f1b2a5 into tronprotocol:release_v4.7.2 Apr 18, 2023
@xxo1shine xxo1shine deleted the optimize_sign branch July 14, 2023 09:09
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