Fix compiler warnings by not std::move()ing where redundant

Author: Al2Klimov

lib/base/object-packer.cpp

@@ -242,5 +242,5 @@ String icinga::PackObject(const Value& value)
 	std::string builder;
 	PackAny(value, builder);
 
-	return std::move(builder);
+	return builder;
 }

lib/config/config_parser.yy

@@ -280,7 +280,7 @@ std::unique_ptr<Expression> ConfigCompiler::Compile()
 
 	std::unique_ptr<DictExpression> expr{new DictExpression(std::move(dlist))};
 	expr->MakeInline();
-	return std::move(expr);
+	return expr;
 }
 
 #define scanner (context->GetScanner())

lib/config/configcompiler.cpp

@@ -146,7 +146,7 @@ std::unique_ptr<Expression> ConfigCompiler::HandleInclude(const String& relative
 
 	std::unique_ptr<DictExpression> expr{new DictExpression(std::move(expressions))};
 	expr->MakeInline();
-	return std::move(expr);
+	return expr;
 }
 
 /**
@@ -174,7 +174,7 @@ std::unique_ptr<Expression> ConfigCompiler::HandleIncludeRecursive(const String&
 
 	std::unique_ptr<DictExpression> dict{new DictExpression(std::move(expressions))};
 	dict->MakeInline();
-	return std::move(dict);
+	return dict;
 }
 
 void ConfigCompiler::HandleIncludeZone(const String& relativeBase, const String& tag, const String& path, const String& pattern, const String& package, std::vector<std::unique_ptr<Expression> >& expressions)