Skip to content

Fixes for pull requests #38, #41, and #42#46

Merged
akleshchev merged 2 commits intosecondlife:contributefrom
Fawrsk:fix-prs
Jan 10, 2023
Merged

Fixes for pull requests #38, #41, and #42#46
akleshchev merged 2 commits intosecondlife:contributefrom
Fawrsk:fix-prs

Conversation

@Fawrsk
Copy link
Contributor

@Fawrsk Fawrsk commented Jan 10, 2023

Eliminate unnecessary copies, and remove uses of auto

Eliminate unnecessary copies, and remove uses of auto
@Fawrsk Fawrsk marked this pull request as ready for review January 10, 2023 02:43
@akleshchev
Copy link
Contributor

akleshchev commented Jan 10, 2023

Thank you.
Wasn't aware it generated a copy (even if declaration implies that), I thought C++11's 'for' loop was smarter than that and would optimize copies away.

@akleshchev akleshchev merged commit 7419037 into secondlife:contribute Jan 10, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jan 10, 2023
@Fawrsk Fawrsk deleted the fix-prs branch January 10, 2023 21:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants