fix redirect with error - Githubissues

habvi / 42_minishell

Re-implemented a simple shell.

4 stars 0 forks source link

fix redirect with error #326

Closed ak0327 closed 1 year ago

ak0327 commented 1 year ago

原因

redirectはexpand_filename() -> open_file() の順で実行しているが、expand_filename(), open_file()でそれぞれredirectをiterate(1 ~ n番目）をしている
redirect 1 ~ n番目全てに対しexpand(check: ambiguous)の後、1 ~ n番目をopen (check: no such file, etc...) していたため、1つでもambiguousがあるとopenが実行されていなかった

変更

expand_filename_and_open_files()を作成しredirect 1 ~ n番目をiterateする
i番目のexpand -> open完了後、i+1番目を実行するよう変更した
テスト追加

minishell ls # nothing

minishell >$a
minishell: $a: ambiguous redirect

minishell >out1 >out2 >$a >out3
minishell: $a: ambiguous redirect
minishell ls
out1  out2  # dekita!

habvi commented 1 year ago

ありがとうございます！！良い感じですね✨ エラー+正常ケース試し忘れ他にもありそうですね…

彷徨いしコメントだけ移動してみました a1897e1f6eea755787ba9f3a1432d0144a85b9e4 大丈夫そうなら merge お願いしますー！

(私が消し忘れていた test の残党を消去、もついでに追加しました) 152ee08a78f92902853514ac68ffe1cb405801dd