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.

Sign up for GitHub

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

Jump to bottom

Replace usages of Array.Clear(arr, 0, arr.Length) with Array.Clear(arr) #13136

Closed
Tracked by #8597
Youssef1313 opened this issue Aug 5, 2023 · 0 comments · Fixed by #14978
Closed
Tracked by #8597

Replace usages of Array.Clear(arr, 0, arr.Length) with Array.Clear(arr) #13136

Youssef1313 opened this issue Aug 5, 2023 · 0 comments · Fixed by #14978
Assignees
@Youssef1313
Labels
area/performance 📈 Categorizes an issue or PR as relevant to performance difficulty/starter 🚀 Categorizes an issue for which the difficulty level is reachable by newcomers kind/enhancement New feature or request low-hanging-🍒 Categorizes an issue that might be a quick win with meaningful positive impact.

Comments

@Youssef1313
Copy link
Member

Per dotnet/runtime#51548, it's supposed to be more optimized.

It might still be good to have a benchmark though.

cc @ebariche
@Youssef1313 Youssef1313 added kind/enhancement New feature or request triage/untriaged Indicates an issue requires triaging or verification difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. area/performance 📈 Categorizes an issue or PR as relevant to performance labels Aug 5, 2023
@Youssef1313 Youssef1313 self-assigned this Aug 5, 2023
@jeromelaban jeromelaban mentioned this issue Aug 14, 2023
Open
71 tasks
@MartinZikmund MartinZikmund added difficulty/starter 🚀 Categorizes an issue for which the difficulty level is reachable by newcomers low-hanging-🍒 Categorizes an issue that might be a quick win with meaningful positive impact. and removed difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. triage/untriaged Indicates an issue requires triaging or verification labels Sep 20, 2023
@Youssef1313 Youssef1313 mentioned this issue Jan 6, 2024
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Youssef1313 Youssef1313

Labels
area/performance 📈 Categorizes an issue or PR as relevant to performance difficulty/starter 🚀 Categorizes an issue for which the difficulty level is reachable by newcomers kind/enhancement New feature or request low-hanging-🍒 Categorizes an issue that might be a quick win with meaningful positive impact.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

perf: Use Array.Clear(arr) instead of Array.Clear(arr, 0, arr.Length) Youssef1313/uno
2 participants
@MartinZikmund @Youssef1313