Skip to content

Conversation

@dayano74
Copy link
Collaborator

@dayano74 dayano74 commented May 9, 2025

fix #232

is_unit_builtin

  • 可読性が低かったので、リファクタリングしました。
  • exec_unit_builtinでリダイレクトを2つまでしか処理できませんが、is_unit_builtinでは拡張性を考慮してリダイレクトが2つ以上合ってもtrueを返すようにしています。

exec_unit_builtin

以下のようなリダイレクトが2つまでのシンプルな処理は実行できるように修正しました。./minishellで検証お願いします。
echo hello
echo hello > outfile
echo hello<< EOF > outfile

@dayano74 dayano74 requested a review from cacapon May 9, 2025 07:15
Copy link
Owner

@cacapon cacapon left a comment

Choose a reason for hiding this comment

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

動作確認しましたがOKだと思います。
口頭で共有した前方のリダイレクトについてはWANTとしましょう。
こちらはマージします。お疲れ様でした!

@cacapon cacapon merged commit 01b69d3 into main May 9, 2025
2 checks passed
@cacapon cacapon deleted the feature/fix_exec_unit_builtin branch May 9, 2025 07:31
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.

exec_unit_builtinの処理の修正

3 participants