Skip to content

Conversation

@cacapon
Copy link
Owner

@cacapon cacapon commented May 2, 2025

fixed #198

メモリの確保が正しくできていなかったので、parserの内部処理を作り直しました。
インターフェース部分に変更はないのでinvoke_cmdはそのまま使えます。

メモリリークが発生していますが、
こちらの対応については #207 でまとめて行うので、ここでは残したままとさせてください。

@cacapon cacapon added the WIP まだ作業が完了していないプルリクなどに label May 2, 2025
@cacapon cacapon self-assigned this May 2, 2025
@cacapon cacapon requested a review from dayano74 May 2, 2025 12:55
@cacapon cacapon removed the WIP まだ作業が完了していないプルリクなどに label May 2, 2025
@cacapon cacapon marked this pull request as ready for review May 2, 2025 13:01
@cacapon cacapon changed the title Feature/198 fix cmds argv argvのメモリ領域確保対応 May 2, 2025
@cacapon
Copy link
Owner Author

cacapon commented May 2, 2025

対応しました。
./minishellで動作確認できるので、セグフォなどが起きないか確認をお願いします。

Copy link
Collaborator

@dayano74 dayano74 left a comment

Choose a reason for hiding this comment

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

特に問題ないと思います!

@dayano74 dayano74 merged commit ce96ad8 into main May 4, 2025
2 checks passed
@dayano74 dayano74 deleted the feature/198-fix-cmds-argv branch May 4, 2025 07:10
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.

parserのargvの確保が正しくできていない

3 participants