search

tkomatsu / minishell

my own shell like bash
MIT License
3 stars 0 forks source link

itoaでアロケーション済みでは？ #151

Closed tkomatsu closed 3 years ago

tkomatsu commented 3 years ago

https://github.com/tkomatsu/minishell/blob/274ea1f2fe3b9e93c680c2c3ac6e4f6d71a67199/srcs/parse/expand_environ.c#L64-L67 コードを見ていてふと気になりました。 ft_itoa()は内部でmallocしているはずなので、ft_strdup()及びそれに対するエラー処理は不要になったりしませんか？

tkomatsu commented 3 years ago

リファクタリングにあたるので、無理に修正は不要です。

kefujiwa commented 3 years ago

確かに、、指摘ありがとうございます！