Skip to content

rust: fixes for new clippy warnings#12643

Closed
jasonish wants to merge 1 commit intoOISF:masterfrom
jasonish:20250220/clippy/v1
Closed

rust: fixes for new clippy warnings#12643
jasonish wants to merge 1 commit intoOISF:masterfrom
jasonish:20250220/clippy/v1

Conversation

@jasonish
Copy link
Member

Fixes provided by cargo clippy --fix.

Fixes provided by cargo clippy --fix.
@codecov
Copy link

codecov bot commented Feb 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.77%. Comparing base (d61f36c) to head (41ed559).

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #12643   +/-   ##
=======================================
  Coverage   80.76%   80.77%           
=======================================
  Files         932      932           
  Lines      259381   259381           
=======================================
+ Hits       209484   209505   +21     
+ Misses      49897    49876   -21     
Flag Coverage Δ
fuzzcorpus 57.01% <83.33%> (+<0.01%) ⬆️
livemode 19.36% <16.66%> (+<0.01%) ⬆️
pcap 44.17% <83.33%> (+0.01%) ⬆️
suricata-verify 63.48% <83.33%> (+<0.01%) ⬆️
unittests 58.33% <33.33%> (-0.01%) ⬇️

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

@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline 24828

* an order of 3 which guarantee we have some room in the block compared
* to standard frame size */
#define AFP_BLOCK_SIZE_DEFAULT_ORDER 3
#define AFP_BLOCK_SIZE_DEFAULT_ORDER 5
Copy link
Member

Choose a reason for hiding this comment

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

did clippy warn about this too? ;-)

@victorjulien
Copy link
Member

#12647 addresses the same, will go with that.

@jasonish jasonish deleted the 20250220/clippy/v1 branch May 21, 2025 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants