Discussion :

[corrector]

Au passage:

• En C, il est déconseillé de caster explicitement l'adresse retournée par malloc()

Pourquoi?

[Thierry Chappuis]

Pourquoi?

Voici quelques explications:

• C'est inutile!
• En cas d'oubli de l'inclusion de stdlib.h, la malloc() n'est pas déclarée. Dans ce cas, le compilateur suppose que malloc() retourne un entier de type int. Or, la conversion implicite d'un entier de type int en un pointeur n'est pas légale et le compilateur renvoie une erreur. Par contre, si une conversion explicite est effectuer, le compilateur comprendra "je sais que je fais quelque chose de pas catholique, mais je le sais" et ne renverra pas d'erreur. On se prive ainsi qu'une possibilité de vérification automatique par le compilateur.
• Plus grave (toujours en cas d'oubli de stdlib.h), sur les systèmes 64 bits où un pointeur est représenté sur 64 bits et un int sur 32, la valeur retournée par malloc(), considérée comme un entier de type int par le compilateur, sera tronquée à 32 bits.

Thierry

[corrector]

En cas d'oubli de l'inclusion de stdlib.h, la malloc() n'est pas déclarée. Dans ce cas, le compilateur suppose que malloc() retourne un entier de type int.

C'est bien là le problème, et pas la présence ou l'absence de cast. Il ne faut pas appeler une fonction non-déclarée, qu'on utilise la valeur de retour comme un entier, ou qu'on l'ignore, enfin qu'on suppose ou pas que c'est un pointeur.

Ou alors il faut bannir toutes les fonctions qui renvoient un entier, quelque chose qui se converti implicitement en entier, ou dont la valeur de retour est généralement ignorée.

La solution est de demander au compilateur de raller dès qu'il voit un appel à une fonction non déclarée. Et laisser malloc tranquille.

[Médinoc]

La solution est de demander au compilateur de raller dès qu'il voit un appel à une fonction non déclarée. Et laisser malloc tranquille.

En effet, l'erreur décrite par Thierry est généralement due à une combinaison de facteurs (cast + oubli d'inclusion + oubli des warnings).

Mais ce n'est pas si difficile que ça à obtenir: Après avoir grandi avec des EDI, j'ai moi-même appris à mes dépens que gcc en ligne de commande n'affiche pas de warnings à moins qu'on les lui demande explicitement...

[corrector]

Après avoir grandi avec des EDI, j'ai moi-même appris à mes dépens que gcc en ligne de commande n'affiche pas de warnings à moins qu'on les lui demande explicitement...

Je trouve assez humiliant d'avoir à préciser -pedantic pour avoir un comportement conforme.

C'est d'autant plus difficile de convaincre les utilisateurs, utilisateurs déjà pas très chauds pour ajouter des options dont ils ne voient pas a priori l'utilité.

[Garulfo]

C'est bien là le problème, et pas la présence ou l'absence de cast.[...]

Mais si le fait de ne pas mettre de cast permet de la détecter, alors que le cast n'est pas requis et n'aide en rien, c'est une bonne chose. Pourquoi vouloir s'en priver ?

Ton « ou alors » n'a pas de sens pour le développeur. Ce n'est pas lui qui choisi la spécification du langage.

Par contre, c'est clair qu'il faudrait penser à toujours demander le maximum de warning en phase de développement. C'est clair que j'ai toujours trouvé qu'il serait plus intelligent que le compilateur donne tous les avertissements par défaut. Dans le cas où l'utilisateur les trouve génant, ce serait explicitement qu'il activerait une option de limitation. Mais ce n'est pas ainsi.

[souviron34]

Mais si le fait de ne pas mettre de cast permet de la détecter, alors que le cast n'est pas requis et n'aide en rien, c'est une bonne chose. Pourquoi vouloir s'en priver ?

et pourquoi le "déconseiller", et même "fortement" comme c'est dit dans plusieurs posts et/ou threads ??

Je trouve que ces arguments sont un peu faibles, car en fait cela montre les déficiences du compilo, pas du programme ou de l'utilisation du langage...

Et d'abord le "cast" peut être requis dans plusieurs utilisations (passages de paramètres à une fonction par exemple, où le -pedantic -Wall gueulera qu'il n'y a pas le bon type de paramètres..).

Et donc mettre au rencart une instruction partiellement me semble plus dangereux...

[Médinoc]

Et d'abord le "cast" peut être requis dans plusieurs utilisations (passages de paramètres à une fonction par exemple, où le -pedantic -Wall gueulera qu'il n'y a pas le bon type de paramètres..).

?
Tu pourrais préciser et mettre un exemple ? J'ai du mal à voir de quoi tu veux parler, et je n'ai pas de gcc sous la main non plus...