Skip to content
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

rust: Convert to use Default/Self #6136

Closed
wants to merge 7 commits into from
Closed

Conversation

jlucovsky
Copy link
Contributor

This PR updates the Rust source modules to use the more "modern" Self/Default constructs.

Link to redmine ticket: 4497

Describe changes:

  • Modify to use Self/Default constructs within Rust source code modules

#suricata-verify-pr:
#suricata-verify-repo:
#suricata-verify-branch:
#suricata-update-pr:
#suricata-update-repo:
#suricata-update-branch:
#libhtp-pr:
#libhtp-repo:
#libhtp-branch:

@codecov
Copy link

codecov bot commented May 22, 2021

Codecov Report

Merging #6136 (91a8a25) into master (533c6ff) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6136      +/-   ##
==========================================
+ Coverage   76.87%   76.88%   +0.01%     
==========================================
  Files         613      613              
  Lines      186344   186344              
==========================================
+ Hits       143248   143270      +22     
+ Misses      43096    43074      -22     
Flag Coverage Δ
fuzzcorpus 52.82% <ø> (+<0.01%) ⬆️
suricata-verify 50.97% <ø> (+0.01%) ⬆️
unittests 63.07% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@victorjulien victorjulien mentioned this pull request May 25, 2021
@victorjulien
Copy link
Member

Merged in #6139, thanks!

@jlucovsky jlucovsky deleted the 4497/1 branch June 27, 2021 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants