Merge pull request #4973 from nyalldawson/std_reverse

Use std::reverse instead of custom array reverse
qgis · Aug 6, 2017 · 3d425fe · 3d425fe
2 parents a612a68 + 7f1fa54
commit 3d425fe
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/core/expression/qgsexpressionfunction.cpp b/src/core/expression/qgsexpressionfunction.cpp
@@ -3605,7 +3605,7 @@ static QVariant fcnArrayCat( const QVariantList &values, const QgsExpressionCont
 static QVariant fcnArrayReverse( const QVariantList &values, const QgsExpressionContext *, QgsExpression *parent )
 {
   QVariantList list = QgsExpressionUtils::getListValue( values.at( 0 ), parent );
-  for ( int k = 0, s = list.size(), max = ( s / 2 ); k < max; k++ ) list.swap( k, s - ( 1 + k ) );
+  std::reverse( list.begin(), list.end() );
   return list;
 }