Project

General

Profile

Actions

Optimization #4599

closed

Optimization #4591: Fix Rust clippy lints

Fix warning about "ptr_arg"

Added by Juliana Fajardini Reichow over 1 year ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Effort:
Difficulty:
Label:
Beginner, Good First Issue, Outreachy, Rust

Description

We are working on cleaning our Rust code from lint warnings.

For resolving this issue, please check the rust code and fix warnings related to "ptr_arg", or along the lines of "writing `&Vec<_>` instead of `&[_]` involves one more reference and cannot be used with non-Vec-based slices"

For more details on this warning, check
https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg

More information on the parent issue.

Actions #1

Updated by Anonymous about 2 months ago

Hello,

Can I claim this issue?

Actions #2

Updated by Juliana Fajardini Reichow about 2 months ago

Hello!

Sure, thanks. We'll assign it to you. Meanwhile, feel free to start working on it. :)

Actions #3

Updated by Juliana Fajardini Reichow about 2 months ago

  • Assignee set to Anonymous
Actions #4

Updated by Shivani Bhardwaj about 2 months ago

  • Target version set to 7.0.0-beta1
Actions #5

Updated by Anonymous about 1 month ago

  • Status changed from New to In Review
Actions #6

Updated by Juliana Fajardini Reichow about 1 month ago

A PR was done for this, although I think the person has decided not to work on this anymore (can't find their account anymore):

https://github.com/OISF/suricata/pull/8002

Actions #7

Updated by Victor Julien about 1 month ago

  • Status changed from In Review to New
  • Target version deleted (7.0.0-beta1)
Actions #8

Updated by Victor Julien about 1 month ago

  • Status changed from New to In Review
  • Assignee set to Jason Ish
  • Target version set to 7.0.0-beta1

Guess this is part of Jason's staging branch.

Actions #9

Updated by Shivani Bhardwaj about 1 month ago

  • Status changed from In Review to Closed
Actions

Also available in: Atom PDF