search

aya-rs / aya

Aya is an eBPF library for the Rust programming language, built with a focus on developer experience and operability.
https://aya-rs.dev/book/
Apache License 2.0
3.25k stars 291 forks source link

feat: Add `set` for `Array<T>` #1005

Open fslongjin opened 4 months ago

fslongjin commented 4 months ago

Add set() function for Array<T>

This change is Reviewable

netlify[bot] commented 4 months ago

Deploy Preview for aya-rs-docs ready!

Built without sensitive environment variables

Name Link
Latest commit 5458de0760361f26a10695a2102cda8fe1d50b9f
Latest deploy log https://app.netlify.com/sites/aya-rs-docs/deploys/67485d1ab70acb00082d3baf
Deploy Preview https://deploy-preview-1005--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.

fslongjin commented 3 months ago

ping

tamird commented 3 months ago

ping

Needs tests.

vadorovsky commented 2 months ago

To be precise - a great place to add tests would be https://github.com/aya-rs/aya/blob/main/test/integration-ebpf/src/map_test.rs

A simple set call on any kind of map with some dummy values would be enough :slightly_smiling_face:

fslongjin commented 2 months ago

Ok, I'll add tests tomorrow

fslongjin commented 1 day ago

To be precise - a great place to add tests would be https://github.com/aya-rs/aya/blob/main/test/integration-ebpf/src/map_test.rs

A simple set call on any kind of map with some dummy values would be enough 🙂

I've already added a test and rebased the branch to the latest version. Could you please review it? Thank you!