penberg / helix

Helix, a market data feed handler for C and C++.
Other
110 stars 35 forks source link

Small cleanups #31

Closed jlaire closed 8 years ago

jlaire commented 8 years ago

Change the code to follow two good C++11 idioms consistently:

  1. Make unary constructors explicit.
  2. When a copy of a parameter is needed, pass it by value and use std::move() as appropriate.

In the second case, using T&& instead of T means the function can't be called with an lvalue. Typically there is no meaningful difference in performance between the two.