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

[EPIC] Optimisation improvements #33

Open
2 tasks
jonblack opened this issue Jun 10, 2016 · 0 comments
Open
2 tasks

[EPIC] Optimisation improvements #33

jonblack opened this issue Jun 10, 2016 · 0 comments

Comments

@jonblack
Copy link
Contributor

Looking at the code there are a lot of improvements that can be made to optimise the code. Profile each case before implementing and check that it indeed improves performance.

Let's chat before implementing anything.

  • Don't return copies of objects (alternatives are pointers, references and rvalue references, be careful with the latter. Move semantics take time to get used to.)
  • Don't pass by value. Pass by either const reference or pointer (raw or smart) or rvalue reference (move semantics if transferring ownership)
@jonblack jonblack changed the title Optimisation improvements [EPIC] Optimisation improvements Jun 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant