Discussion :

[yan]

Pour vous, doit on virer ton les warning d'une compilation? et donc les considérer comme une erreur?

[ram-0000]

Je configure mon compilateur est au niveau super sensible
Je n'active pas l'option warning = erreur
mais de toutes façons, je ne continue pas tant qu'il y a des warnings

[BainE]

bonjour,

assez d'accord, quand le temps et le client le permettent j essaye de tout virer c'est toujours plus tranquilisant, excepter sous visual studio les trucs du genre sprintf deprecated utiliser sprintf_s

[yan]

Pour la plupart ok.
Mais certain j'hésite.
Par exemple l'utilisation de int et uint dans un calcul.
Un warning est donné, et je le trouve important de le garder. Justement parce qu'un cast pour virer le warning n'est peut être pas la solution...


Je trouve que ça permet de donner une trace d'un problème potentielle. Et cela bien mieux qu'un commentaire que l'on ne lira pas

[BainE]

et changer l une des 2 variables en signée ou non pour etre conforme ?

[yan]

et changer l une des 2 variables en signée ou non pour etre conforme ?

parce que cast de -1 en unsigned int est si vite arrivé

[Mat007]

Par exemple l'utilisation de int et uint dans un calcul.
Un warning est donné, et je le trouve important de le garder. Justement parce qu'un cast pour virer le warning n'est peut être pas la solution...

Et garder un warning n'est pas forcément le plus malin non plus, cf. par ex quelques explications.
Et un peu de code pour illustrer :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
BOOST_AUTO_TEST_CASE( IntegralPromotion )
{
    int i = -1;
    unsigned int j = 0;
    BOOST_CHECK( i < j );
}

Produit le fameux warning :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

warning C4018: '<' : signed/unsigned mismatch

Et donne comme résultat :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

error in "IntegralPromotion": check i < j failed

(le test échoue, i n'est pas < à j)

parce que cast de -1 en unsigned int est si vite arrivé

Pas besoin de transtypage, le mécanisme de promotion/conversion sur les entiers s'en charge très bien.

MAT.

[yan]

Et garder un warning n'est pas forcément le plus malin non plus, cf. par ex quelques explications.

C'est typiquement mon exemple en faite
Ce n'est pas toujours évident de le résoudre. ET surtout pas toujours nécessaires...
Je trouve que garder le warning est utile pour garder la trace de ce risque potentiel

[khayyam90]

Je suis tenté de dire oui.
Si un warning est affiché, c'est pas pour rien, il faut en tenir compte, c'est pas pour les chiens. Bien souvent les warnings sont sur ces broutilles, des erreurs d'inattention, ça ne te tuera pas de les corriger.
Si vraiment il y a certains warnings que tu ne peux pas éviter, soit, on pourrait alors préconiser un réglage du compilo pour inhiber ces warnings là et toujours bénéficier de la levée de tous les autres, toujours dans l'idée d'avoir une compilation "sans warning".

Et comme on dit toujours, un bon réglage de ton compilo t'évitera déjà pas mal de bugs