SakulFlee/Orbital
1
0
Fork 0
Code Issues 51 Pull requests 24 Projects Releases 1 Packages Wiki Activity

perf: use fixed [u8; 64] array for light buffer data #709

Open
SakulFlee wants to merge 1 commit from perf/light-descriptor-fixed-array into main
pull from: perf/light-descriptor-fixed-array
merge into: SakulFlee:main
Conversation 0 Commits 1 Files changed 1 +38 -34
SakulFlee commented 2026-07-17 22:04:51 +02:00
Owner
Copy link

LightDescriptor::to_buffer_data() was allocating a Vec and extending
it 16 times per light. Since the output is always exactly 64 bytes,
now returns a fixed-size array with no heap allocation.

LightDescriptor::to_buffer_data() was allocating a Vec and extending it 16 times per light. Since the output is always exactly 64 bytes, now returns a fixed-size array with no heap allocation.
perf: use fixed [u8; 64] array for light buffer data
All checks were successful
ci/woodpecker/push/linting Pipeline was successful
Details
a1f69f2a46
 
LightDescriptor::to_buffer_data() was allocating a Vec and extending
it 16 times per light. Since the output is always exactly 64 bytes,
now returns a fixed-size array with no heap allocation.
Some checks are pending
ci/woodpecker/push/linting Pipeline was successful
Details
ci/woodpecker/pr/test
Required
ci/woodpecker/pr/linting
Required
This pull request has changes conflicting with the target branch.
  • Crates/orbital_resources/src/light/descriptor.rs
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin perf/light-descriptor-fixed-array:perf/light-descriptor-fixed-array
git switch perf/light-descriptor-fixed-array
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Context: Async
Async specific, be it bug or feature

  
Context: Bug
Something isn't working as intended

  
Context: CI
GitHub Actions / CI related tasks

  
Context: Dependencies
Pull requests that update a dependency file

  
Context: Documentation
Improvements or additions to documentation

  
Context: Enhancement
New feature or request

  
Context: Example
Example related issues

  
Context: Macro

   
Context: Runtime
Runtime related issues

  
Difficulty: Easy

   
Difficulty: Hard

   
Difficulty: Medium

   
Platform: Android
ONLY if this is platform specific!

  
Platform: iOS
ONLY if this is platform specific!

  
Platform: Linux
ONLY if this is platform specific!

  
Platform: macOS
ONLY if this is platform specific!

  
Platform: Web
ONLY if this is platform specific!

  
Platform: Windows
ONLY if this is platform specific!

  
Type: Discussion

   
Type: Evaluation
Further evaluation is needed

  
Type: Tracker
Issues acting as trackers; I.e. a combination of issues of the same overall topic.

No labels
Context: Async
Context: Bug
Context: CI
Context: Dependencies
Context: Documentation
Context: Enhancement
Context: Example
Context: Macro
Context: Runtime
Difficulty: Easy
Difficulty: Hard
Difficulty: Medium
Platform: Android
Platform: iOS
Platform: Linux
Platform: macOS
Platform: Web
Platform: Windows
Type: Discussion
Type: Evaluation
Type: Tracker
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
SakulFlee
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
SakulFlee/Orbital!709
No description provided.