Skip to content

Conversation

@Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Oct 25, 2024

There are either set to None or Some(vec![]) these days, but we might as well initialize them with an empty static array instead and never return null.

There are either set to `None` or `Some(vec![])` these days, but we might as well initialize them with an empty static array instead and never return `null`.
@Turbo87 Turbo87 added C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear A-backend ⚙️ labels Oct 25, 2024
@Turbo87 Turbo87 requested a review from a team October 25, 2024 10:16
@codecov
Copy link

codecov bot commented Oct 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.80%. Comparing base (8b42804) to head (2ad917c).
Report is 9 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9755      +/-   ##
==========================================
- Coverage   88.81%   88.80%   -0.01%     
==========================================
  Files         289      289              
  Lines       29817    29808       -9     
==========================================
- Hits        26482    26472      -10     
- Misses       3335     3336       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Turbo87 Turbo87 merged commit 7bce876 into rust-lang:main Oct 27, 2024
10 checks passed
@Turbo87 Turbo87 deleted the badges branch October 27, 2024 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant