summaryrefslogtreecommitdiff
path: root/test/event-dispatcher.cpp
diff options
context:
space:
mode:
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>2019-01-22 15:31:22 +0200
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2019-01-22 20:44:04 +0200
commita22dcaaa786de67786eeaeb47598ed76339249ad (patch)
tree81a9cf50def556cc97f64861cdc0bba8b55e3ba7 /test/event-dispatcher.cpp
parent29c3b54f0c8ec7e3aeed8971d24f939bc4bfab04 (diff)
cam: options: Don't implement move semantics for OptionsParser::Options
The compiler creates a move constructor automatically when none is supplied, and it does the right thing by default in this case. Using std::move() inside the function prevents the compiler from doing return value optimization and actually hinders performances. Using std::move() in the caller is unnecessary, the move constructor is used automatically by the compiler. For all these reasons remove the tentative optimization that resulted in worse performances and worse code. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>
Diffstat (limited to 'test/event-dispatcher.cpp')
0 files changed, 0 insertions, 0 deletions