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

aya-bpf-macros: appease clippy #823

Merged
merged 1 commit into from
Oct 22, 2023
Merged

aya-bpf-macros: appease clippy #823

merged 1 commit into from
Oct 22, 2023

Conversation

tamird
Copy link
Member

@tamird tamird commented Oct 22, 2023

warning: accessing first element with `args.args.get(0)`
  --> aya-bpf-macros/src/args.rs:71:24
   |
71 |     if let Some(arg) = args.args.get(0) {
   |                        ^^^^^^^^^^^^^^^^ help: try: `args.args.first()`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_first
   = note: `#[warn(clippy::get_first)]` on by default

Appears rust-lang/rust-clippy@31fd282732e15811 has just landed in nightly.

```
warning: accessing first element with `args.args.get(0)`
  --> aya-bpf-macros/src/args.rs:71:24
   |
71 |     if let Some(arg) = args.args.get(0) {
   |                        ^^^^^^^^^^^^^^^^ help: try: `args.args.first()`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_first
   = note: `#[warn(clippy::get_first)]` on by default
```

Appears rust-lang/rust-clippy@31fd282732e15811
has just landed in nightly.
@netlify
Copy link

netlify bot commented Oct 22, 2023

Deploy Preview for aya-rs-docs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit a31332f
🔍 Latest deploy log https://app.netlify.com/sites/aya-rs-docs/deploys/653550396390b40008c2ff48
😎 Deploy Preview https://deploy-preview-823--aya-rs-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tamird tamird requested a review from dave-tucker October 22, 2023 16:39
@mergify mergify bot added the aya-bpf This is about aya-bpf (kernel) label Oct 22, 2023
@tamird
Copy link
Member Author

tamird commented Oct 22, 2023

Merging without waiting for a macOS worker because this is trivial.

@tamird tamird merged commit 847f410 into aya-rs:main Oct 22, 2023
@tamird tamird deleted the clippy branch October 22, 2023 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aya-bpf This is about aya-bpf (kernel)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants