LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-01-08 12:19:37 +03:00
Code Issues Packages Projects Releases Wiki Activity

Shell: Allow non-exhaustive 'case' statements in POSIX

Browse Source
This commit is contained in:
Ali Mohammad Pur 2023-04-19 12:16:23 +03:30 committed by Ali Mohammad Pur
parent ef551a045d
commit 8a042cd9cb
Notes: sideshowbarker 2024-07-17 06:51:40 +09:00 
Author: https://github.com/alimpfard
Commit: https://github.com/SerenityOS/serenity/commit/8a042cd9cb
Pull-request: https://github.com/SerenityOS/serenity/pull/18429
Reviewed-by: https://github.com/nico ✅
2 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Userland/Shell/AST.cpp
View File

@ -2363,7 +2363,9 @@ ErrorOr<RefPtr<Value>> MatchExpr::run(RefPtr<Shell> shell)
return move(result).get<RefPtr<Value>>();
}
shell->raise_error(Shell::ShellError::EvaluatedSyntaxError, "Non-exhaustive match rules!", position());
// Non-exhaustive 'case' statements are valid in POSIX.
if (!shell || !shell->posix_mode())
shell->raise_error(Shell::ShellError::EvaluatedSyntaxError, "Non-exhaustive match rules!", position());
return make_ref_counted<AST::ListValue>({});
}

2
Userland/Shell/Shell.h
View File

@ -308,6 +308,8 @@ public:
void notify_child_event();
bool posix_mode() const { return m_in_posix_mode; }
struct termios termios;
struct termios default_termios;
bool was_interrupted { false };