Skip to content

[exo] Fix exception throw for unknown dialect - #16483

Merged
mhs4670go merged 1 commit into
Samsung:masterfrom
arkq:throw-patch-2
Apr 21, 2026
Merged

[exo] Fix exception throw for unknown dialect#16483
mhs4670go merged 1 commit into
Samsung:masterfrom
arkq:throw-patch-2

Conversation

@arkq

@arkq arkq commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

This commit fixes a typo causing the exception not being thrown.

Copilot AI review requested due to automatic review settings April 20, 2026 07:42

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes the exo::set(Dialect) default-case behavior so an exception is actually thrown when an unknown dialect is provided.

Changes:

  • Replace construction of std::runtime_error with throw std::runtime_error(...) in the default: branch.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread compiler/exo/src/Knob.cpp Outdated
Comment thread compiler/exo/src/Knob.cpp Outdated
This commit fixes a typo causing the exception not being thrown.

ONE-DCO-1.0-Signed-off-by: Arkadiusz Bokowy <a.bokowy@samsung.com>

@mhs4670go mhs4670go left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mhs4670go
mhs4670go merged commit de7f473 into Samsung:master Apr 21, 2026
6 checks passed
@arkq
arkq deleted the throw-patch-2 branch April 21, 2026 05:59
glistening pushed a commit to glistening/ONE that referenced this pull request Jul 5, 2026
This commit fixes a typo causing the exception not being thrown.

ONE-DCO-1.0-Signed-off-by: Arkadiusz Bokowy <a.bokowy@samsung.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants