diff options
| author | vitya-smirnov <[email protected]> | 2025-10-20 15:43:33 +0300 |
|---|---|---|
| committer | vitya-smirnov <[email protected]> | 2025-10-20 16:32:51 +0300 |
| commit | 2031733477ed2cb915aa37e7eb14bd9eb3d5d8f9 (patch) | |
| tree | 36e3f7184ade6465c96931ec9a792c94f6504f08 /yql/essentials/sql/v1/sql.cpp | |
| parent | dbc560aad32360d83f45d7854b01617b185acae4 (diff) | |
Use Y_UNREACHABLE for ALT_NOT_SET
It is not convinient to grep "You should change implementation according
to grammar changes" every time you need it during new translation
feature. AI does not always suggests it. More convinient is to use
just `Y_UNREACHABLE` as `ALT_NOT_SET` is actually unreachable.
commit_hash:132f264e9f781b8c2433fb1f12374f7684466370
Diffstat (limited to 'yql/essentials/sql/v1/sql.cpp')
| -rw-r--r-- | yql/essentials/sql/v1/sql.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/yql/essentials/sql/v1/sql.cpp b/yql/essentials/sql/v1/sql.cpp index 1fb88c6089b..c34e25112d4 100644 --- a/yql/essentials/sql/v1/sql.cpp +++ b/yql/essentials/sql/v1/sql.cpp @@ -125,7 +125,6 @@ bool NeedUseForAllStatements(const TRule_sql_stmt_core::AltCase& subquery) { case TRule_sql_stmt_core::kAltSqlStmtCore14: // export case TRule_sql_stmt_core::kAltSqlStmtCore18: // define action or subquery return true; - case TRule_sql_stmt_core::ALT_NOT_SET: case TRule_sql_stmt_core::kAltSqlStmtCore2: // select case TRule_sql_stmt_core::kAltSqlStmtCore4: // create table case TRule_sql_stmt_core::kAltSqlStmtCore5: // drop table @@ -188,6 +187,8 @@ bool NeedUseForAllStatements(const TRule_sql_stmt_core::AltCase& subquery) { case TRule_sql_stmt_core::kAltSqlStmtCore67: // alter secret case TRule_sql_stmt_core::kAltSqlStmtCore68: // drop secret return false; + case TRule_sql_stmt_core::ALT_NOT_SET: + Y_UNREACHABLE(); } } |