faboussard / 42_minishell

1 stars 0 forks source link

pour discussion a loral : necessqire de faire des ifndef partout ? pour moi cest pas necessaire #80

Closed faboussard closed 4 months ago

faboussard commented 4 months ago

ifndef MINISHELL_H

define MINISHELL_H

ifndef MALLOC_FAILED

define MALLOC_FAILED -1

endif

ifndef SUCCESSFULLY_ADDED

define SUCCESSFULLY_ADDED 0

endif

ifndef HERE_DOC_TMP_FILE

define HERE_DOC_TMP_FILE "/tmp/.tmp_heredoc"

endif

ifndef PATH_MAX

define PATH_MAX 4096

endif

ifndef SHLVL

define SHLVL_LIMIT 1000

endif

melobern commented 4 months ago

➡️ Considéré comme une bonne pratique de toujours mettre ifndef même si dans un seul fichier header ➡️ Certains sont vraiment nécessaires même si uniquement dans ce projet, comme le ifndef PATH_MAX ➡️ Ceux-ci n'empêchent pas le projet de fonctionner, il vaut mieux réfléchir sur d'autres issues (notamment le Ctrl \ sur une entree standard, pour lequel le minishell rentre dans WIFEXITED au lieu de WIFSIGNALED)