openPMD / openPMD-api / 1097

82%

dev: 83%

Pull #305

travis-ci

Path Arguments: Pass-by-Value

Pass `std::string`s by value and `std::move` them where needed,
e.g. in constructors.

At worst, same copy cost as before for member init.
At best, add rvalue movability.

Seen via `modernize-pass-by-value` clang-tidy pass:
  https://clang.llvm.org/extra/clang-tidy/checks/modernize-pass-by-value.html

14 of 14 new or added lines in 5 files covered. (100.0%)

3708 of 4540 relevant lines covered (81.67%)

0.82 hits per line