hyudai28 / minishell_42

0 stars 1 forks source link

norm未修正の部分とか気になるとこ #13

Closed fujishit closed 2 years ago

fujishit commented 2 years ago

builtinsのexitはatoiが3関数あるため、normerror。まとめた奴に差し替えれるか試す。

fujishit commented 2 years ago

builtins_exportの13行目がなんかきもい、normが変

fujishit commented 2 years ago

excuteは関数上限ばっか、ファイル分けしよう。

fujishit commented 2 years ago

excute: 一部中にあるコメントが修正していいか不明、いったん残す。

fujishit commented 2 years ago

expansion_utils: builtins_exportの13行目がきもい奴。

if
  if
    if1();
  else
    if2();
else
  else1()

を{}なしでやるとif内のelseが一個左に下げさせられる、normのバージョンの問題かも ifの次に出てきたelseはifと同じインデントにいないといけないnormのバグ

fujishit commented 2 years ago

mainって引数ついてても実行でいいのか??

hyudai28 commented 2 years ago

本来であればオプションを取るからコマンドラインは必須だけど、今回は対象外。 わざわざ蹴らなくてもいいんじゃない?って考え

fujishit commented 2 years ago

^cの表示がいやだったら、writeに\bをつけたらいいらしい https://note.com/syamashi/n/na31efe927b41

hyudai28 commented 2 years ago

writeで改行だけ出力してるやつがあると思うけど、前そこでテストしたんだよね。 その時うまくいかなかったから試してほしい

fujishit commented 2 years ago

struct: ファイル分け、malloc失敗処理してないところがあるからコメントが残ってる

fujishit commented 2 years ago

ttoken.h: normエラーになるため、enumがtypedef enumだったのをenumにした。 命名規則の都合でtypedef enum tがenum e_になってる。

fujishit commented 2 years ago

t_token.cの関数分けがちょっとだけ不安 あとヘッダーをやってない

fujishit commented 2 years ago

writeで改行だけ出力してるやつがあると思うけど、前そこでテストしたんだよね。 その時うまくいかなかったから試してほしい

普通になおったからpushした

なおったと思ったら、前の行のminishellのmiだけ消えてたから修正してpush