diff options
| author | eeight <eeight@yandex-team.ru> | 2022-04-19 12:57:12 +0300 |
|---|---|---|
| committer | eeight <eeight@yandex-team.ru> | 2022-04-19 12:57:12 +0300 |
| commit | d46bdf090cf3ebaf644c54cd7ee0dd0691c49b96 (patch) | |
| tree | 308c81808c0660f9295fd8af1b1c202c54925cfa | |
| parent | 466fcce0997221e5b23eef3a0068470544bd6bae (diff) | |
| download | ydb-d46bdf090cf3ebaf644c54cd7ee0dd0691c49b96.tar.gz | |
IGNIETFERRO-1969 Prepare to enable -Wrange-loop-construct
ref:51e077bc539d94cbef090cff37e061d8f7b5120b
| -rw-r--r-- | ydb/library/yql/core/yql_expr_constraint.cpp | 4 | ||||
| -rw-r--r-- | ydb/library/yql/minikql/mkql_program_builder.cpp | 3 |
2 files changed, 3 insertions, 4 deletions
diff --git a/ydb/library/yql/core/yql_expr_constraint.cpp b/ydb/library/yql/core/yql_expr_constraint.cpp index 25a20effd7f..49a010180b5 100644 --- a/ydb/library/yql/core/yql_expr_constraint.cpp +++ b/ydb/library/yql/core/yql_expr_constraint.cpp @@ -3178,7 +3178,7 @@ IGraphTransformer::TStatus UpdateLambdaConstraints(TExprNode::TPtr& lambda, TExp YQL_ENSURE(args->ChildrenSize() == constraints.size()); size_t i = 0; - for (const auto constrList: constraints) { + for (const auto& constrList: constraints) { const auto arg = args->Child(i++); if (arg->GetState() == TExprNode::EState::TypeComplete || arg->GetState() == TExprNode::EState::ConstrPending) { for (const auto c: constrList) { @@ -3197,7 +3197,7 @@ IGraphTransformer::TStatus UpdateLambdaConstraints(TExprNode::TPtr& lambda, TExp TExprNode::TListType argsChildren; argsChildren.reserve(constraints.size()); i = 0; - for (const auto constrList: constraints) { + for (const auto& constrList: constraints) { const auto arg = args->Child(i++); const auto newArg = ctx.ShallowCopy(*arg); newArg->SetTypeAnn(arg->GetTypeAnn()); diff --git a/ydb/library/yql/minikql/mkql_program_builder.cpp b/ydb/library/yql/minikql/mkql_program_builder.cpp index 2314faeb659..ecaf781fd71 100644 --- a/ydb/library/yql/minikql/mkql_program_builder.cpp +++ b/ydb/library/yql/minikql/mkql_program_builder.cpp @@ -5056,10 +5056,9 @@ TRuntimeNode TProgramBuilder::PgResolvedCall(bool useContext, const std::string_ callableBuilder.Add(NewDataLiteral(useContext)); callableBuilder.Add(NewDataLiteral<NUdf::EDataSlot::String>(name)); callableBuilder.Add(NewDataLiteral(id)); - for (const auto arg : args) { + for (const auto& arg : args) { callableBuilder.Add(arg); } - return TRuntimeNode(callableBuilder.Build(), false); } |