summaryrefslogtreecommitdiff
path: root/connectivity/source/commontools
diff options
context:
space:
mode:
authorNoel Grandin <noel.grandin@collabora.co.uk>2019-11-12 12:44:32 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2019-11-12 13:59:50 +0100
commit1a2926a995fdbdcdae0ca6407877084f3520e539 (patch)
tree4ac33d3d0f1d3941c6fe782633fe6e555c157c13 /connectivity/source/commontools
parenta07c637a0e4fe0ab81db70c69decbc946ad37da9 (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/commontools')
-rw-r--r--connectivity/source/commontools/RowFunctionParser.cxx6
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 ) ) );