perf: use slice instead of Vec for render model list #708

Open

SakulFlee wants to merge 1 commit from perf/render-slice-argument into main

SakulFlee commented

2026-07-17 22:04:46 +02:00

Owner

The render function was taking Vec<&Model> by value, requiring the
caller to own the Vec. Now takes &[&Model] which avoids the move
and works with any contiguous slice of references.

The render function was taking Vec<&Model> by value, requiring the caller to own the Vec. Now takes &[&Model] which avoids the move and works with any contiguous slice of references.

SakulFlee self-assigned this

2026-07-17 22:04:46 +02:00

SakulFlee added 1 commit

2026-07-17 22:04:46 +02:00

perf: use slice instead of Vec for render model list

ci/woodpecker/push/linting Pipeline was successful

Details

961200b73b

The render function was taking Vec<&Model> by value, requiring the
caller to own the Vec. Now takes &[&Model] which avoids the move
and works with any contiguous slice of references.

ci/woodpecker/push/linting Pipeline was successful

Details

ci/woodpecker/pr/test

Required

ci/woodpecker/pr/linting

Required

Some required checks are missing.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin perf/render-slice-argument:perf/render-slice-argument

git switch perf/render-slice-argument

No reviewers

No milestone

No project

No assignees

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!708

No description provided.

Rows
Columns