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.

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

Add const lvalue ref to platform/* container parameters #88971

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Muller-Castro
Copy link
Contributor

@Muller-Castro Muller-Castro commented Feb 28, 2024

Adds const lvalue reference to the core container types specified in the documentation that are declared in function parameters in platform/* files

initial: #51156
core/*: #86966
editor/*: #88368
modules/*: #88915
servers/*: #88972
scene/*: #88974
others: #88975

@Muller-Castro Muller-Castro requested review from a team as code owners February 28, 2024 17:46
@Muller-Castro Muller-Castro changed the title Add const lvalue ref to platform/* container parameters Add const lvalue ref to platform/* container parameters Feb 28, 2024
@AThousandShips AThousandShips added this to the 4.x milestone Feb 28, 2024
@Muller-Castro Muller-Castro requested a review from a team as a code owner February 28, 2024 18:32
@Muller-Castro Muller-Castro requested a review from a team as a code owner February 28, 2024 18:44
@Muller-Castro Muller-Castro requested a review from a team as a code owner February 28, 2024 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants