diff options
| author | mikhnenko <[email protected]> | 2025-09-26 16:05:56 +0300 |
|---|---|---|
| committer | mikhnenko <[email protected]> | 2025-09-26 16:36:44 +0300 |
| commit | 5f06d989f8c9230743fef30f7a9f70809b7d07a3 (patch) | |
| tree | 2e855d912a6ada03658da395f470e075e085c79d /yql/essentials/sql/v1 | |
| parent | ee38d4a04d0b4226b36bd9d548c35739f11e88f9 (diff) | |
Fix -Wunused-but-set-variable in yql
If you think that this pr has broken something for you, roll it back
If the problem is not trivial, come to [support](https://nda.ya.ru/t/a1JIzEbo78mtfQ)
Original errors you can see [here](https://nda.ya.ru/t/pFJQM0ho7KSaTN
commit_hash:77d426cfbc646733517e14e69ac61a1afedaf82f
Diffstat (limited to 'yql/essentials/sql/v1')
| -rw-r--r-- | yql/essentials/sql/v1/builtin.cpp | 2 | ||||
| -rw-r--r-- | yql/essentials/sql/v1/select.cpp | 2 | ||||
| -rw-r--r-- | yql/essentials/sql/v1/source.cpp | 4 | ||||
| -rw-r--r-- | yql/essentials/sql/v1/sql_select.cpp | 4 |
4 files changed, 6 insertions, 6 deletions
diff --git a/yql/essentials/sql/v1/builtin.cpp b/yql/essentials/sql/v1/builtin.cpp index 077b3d9fed5..78fdd702ad7 100644 --- a/yql/essentials/sql/v1/builtin.cpp +++ b/yql/essentials/sql/v1/builtin.cpp @@ -1826,7 +1826,7 @@ public: const auto memberPos = Args_[0]->GetPos(); TVector<TNodePtr> repackArgs = {BuildAtom(memberPos, "row", NYql::TNodeFlags::Default)}; - if (auto literal = Args_[1]->GetLiteral("String")) { + if (/* auto literal = */ Args_[1]->GetLiteral("String")) { TString targetType; if (!GetDataTypeStringNode(ctx, *this, 1, &targetType)) { return false; diff --git a/yql/essentials/sql/v1/select.cpp b/yql/essentials/sql/v1/select.cpp index f20668ddbc6..c6ba387e7bd 100644 --- a/yql/essentials/sql/v1/select.cpp +++ b/yql/essentials/sql/v1/select.cpp @@ -2430,7 +2430,7 @@ public: } if (ListCall_) { - if (auto atom = dynamic_cast<TTableRows*>(term.Get())) { + if (/* auto atom = */ dynamic_cast<TTableRows*>(term.Get())) { listPosIndex = termIndex; } } diff --git a/yql/essentials/sql/v1/source.cpp b/yql/essentials/sql/v1/source.cpp index 1a6038b5e6a..613c521da6e 100644 --- a/yql/essentials/sql/v1/source.cpp +++ b/yql/essentials/sql/v1/source.cpp @@ -166,7 +166,7 @@ bool ISource::AddExpressions(TContext& ctx, const TVector<TNodePtr>& expressions } if (exprSeat == EExprSeat::GroupBy) { - if (auto sessionWindow = dynamic_cast<TSessionWindow*>(expr.Get())) { + if (/* auto sessionWindow = */ dynamic_cast<TSessionWindow*>(expr.Get())) { if (SessionWindow_) { ctx.Error(expr->GetPos()) << "Duplicate session window specification:"; ctx.Error(SessionWindow_->GetPos()) << "Previous session window is declared here"; @@ -174,7 +174,7 @@ bool ISource::AddExpressions(TContext& ctx, const TVector<TNodePtr>& expressions } SessionWindow_ = expr; } - if (auto hoppingWindow = dynamic_cast<THoppingWindow*>(expr.Get())) { + if (/* auto hoppingWindow = */ dynamic_cast<THoppingWindow*>(expr.Get())) { if (HoppingWindow_) { ctx.Error(expr->GetPos()) << "Duplicate hopping window specification:"; ctx.Error(HoppingWindow_->GetPos()) << "Previous hopping window is declared here"; diff --git a/yql/essentials/sql/v1/sql_select.cpp b/yql/essentials/sql/v1/sql_select.cpp index 720bf607f17..ea769eb5385 100644 --- a/yql/essentials/sql/v1/sql_select.cpp +++ b/yql/essentials/sql/v1/sql_select.cpp @@ -823,7 +823,7 @@ TSourcePtr TSqlSelect::ProcessCore(const TRule_process_core& node, const TWriteS auto args = call.GetArgs(); for (auto& arg: args) { - if (auto placeholder = dynamic_cast<TTableRows*>(arg.Get())) { + if (/* auto placeholder = */ dynamic_cast<TTableRows*>(arg.Get())) { if (listCall) { Ctx_.Error() << "Only one TableRows() argument is allowed."; return nullptr; @@ -946,7 +946,7 @@ TSourcePtr TSqlSelect::ReduceCore(const TRule_reduce_core& node, const TWriteSet auto args = call.GetArgs(); for (auto& arg: args) { - if (auto placeholder = dynamic_cast<TTableRows*>(arg.Get())) { + if (/* auto placeholder = */ dynamic_cast<TTableRows*>(arg.Get())) { if (listCall) { Ctx_.Error() << "Only one TableRows() argument is allowed."; return nullptr; |
