diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-11-12 12:44:32 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-11-12 13:59:50 +0100 |
commit | 1a2926a995fdbdcdae0ca6407877084f3520e539 (patch) | |
tree | 4ac33d3d0f1d3941c6fe782633fe6e555c157c13 /connectivity/source | |
parent | a07c637a0e4fe0ab81db70c69decbc946ad37da9 (diff) |
use std::move when popping stuff off stacks
Change-Id: I6ba0ee8afee1a9579045643cd0118cf19599d5b9
Reviewed-on: https://gerrit.libreoffice.org/82497
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'connectivity/source')
-rw-r--r-- | connectivity/source/commontools/RowFunctionParser.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/connectivity/source/commontools/RowFunctionParser.cxx b/connectivity/source/commontools/RowFunctionParser.cxx index 811e0fb7c70f..44bd04a080dd 100644 --- a/connectivity/source/commontools/RowFunctionParser.cxx +++ b/connectivity/source/commontools/RowFunctionParser.cxx @@ -206,9 +206,9 @@ public: throw ParseError( "Not enough arguments for binary operator" ); // retrieve arguments - std::shared_ptr<ExpressionNode> pSecondArg( rNodeStack.top() ); + std::shared_ptr<ExpressionNode> pSecondArg( std::move(rNodeStack.top()) ); rNodeStack.pop(); - std::shared_ptr<ExpressionNode> pFirstArg( rNodeStack.top() ); + std::shared_ptr<ExpressionNode> pFirstArg( std::move(rNodeStack.top()) ); rNodeStack.pop(); // create combined ExpressionNode @@ -257,7 +257,7 @@ public: throw ParseError( "Not enough arguments for unary operator" ); // retrieve arguments - std::shared_ptr<ExpressionNode> pArg( rNodeStack.top() ); + std::shared_ptr<ExpressionNode> pArg( std::move(rNodeStack.top()) ); rNodeStack.pop(); rNodeStack.push( std::shared_ptr<ExpressionNode>( new UnaryFunctionExpression( pArg ) ) ); |