Discussion :

[uriotcea]

Bonjour,

J'ai un bout de code en C++ compilé avec gcc et des warnings.
Je voudrait supprimer les messages de warning à un endroit particulier du code C++. Mais je ne trouve pas les commande appropriées dy type "#pragma..." qu'ils faut inclure.
Merci de votre aide

[Médinoc]

Je ne crois pas qu'il y ait d'équivalent à #pragma warning sous gcc: Il n'y a que les options pour désactiver les warnings à l'échelle d'une compilation entière.

[3DArchi]

Je voudrait supprimer les messages de warning à un endroit particulier du code C++.

Salut,
Si c'est dans ton code, tu as plutôt intérêt à voir pourquoi ils sont levés et comment les corrigés. Un warning génère souvent un bug ... au moment de la livraison.

[Goten]

Code :

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

1
2
3
 
 
#pragma GCC diagnostic warning "-Wformat"

http://gcc.gnu.org/onlinedocs/gcc/Di...c-Pragmas.html


Mais comme l'a dit 3DArchi, il vaut mieux les résoudre plutôt que les masqué. A moins que ça soit une bibliothèque tiers?

[nikko34]

Même si c'est une librarie tiers, je les laisserais.

[uriotcea]

merci de vos réponses

J'ai des warning sous windows avec ce type de lignes

Code :

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

1
2
3
4
5
6
 
        #ifdef _WIN32
        fscanf(f,"%d %I64i %lf ",&n,&N0,&ms);
        #else
        fscanf(f,"%d %llu %lf ",&n,&N0,&ms);
        #endif

je ne vois pas moyen de les éviter (pas faute d'avoir essayé)

maintenant la command

Code :

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

1
2
 
#pragma GCC diagnostic warning "-Wformat"

ne semble pas empecher les warnings

Code :

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

1
2
 
warning: format '%I64i' expects type 'int', but argument 3 has type 'long long int

Et malheureusement j'en ai des centaines.

[3DArchi]

J'ai l'impression d'avoir déjà vu ce bout de code il n'y a pas si longtemps

Et même la question. Ca frôle le mutlipost à trois mois de distance

[uriotcea]

Non pas de multipost ici
Aujourd'hui, je m'interroge sur les commandes #pragram et comment supprimer les warnings

[koala01]

Salut,

Déjà, pourquoi utiliser fscanf alors que tu travailles en C++

Ne pourrais tu pas, tout simplement, utiliser un flux classique et les opérateurs de flux classiques

Sous réserve d'utiliser un flux C++ (parmi ifstream, stringstream,...) tu pourrais te contenter de

Code :

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

f>>n>>NO>>ms;

qui s'adaptera quoi qu'il arrive au type correspondant aux variables n, NO et ms (y compris s'il s'agit de classes, à ceci près qu'il faudra définir l'opérateur >> pour celles-ci)

[3DArchi]

Salut,

Déjà, pourquoi utiliser fscanf alors que tu travailles en C++

Ne pourrais tu pas, tout simplement, utiliser un flux classique et les opérateurs de flux classiques

Un lourd passé historique hérité des intervenants précédents

[koala01]

Mais, le fait est que, en modifiant les "milliers de lignes de code" (qui, au passage, pourraient en partie du moins être modifiées automatiquement ), il s'éviterait bien des soucis par la suite...

Et puis, rien n'empêche d'utiliser les fichiers "C style" à l'écriture et les flux à la lecture, si toutes les écritures sont faites (même si j'admets volontiers que ca n'est pas forcément l'idéal )

[3DArchi]

Mais, le fait est que, en modifiant les "milliers de lignes de code" (qui, au passage, pourraient en partie du moins être modifiées automatiquement ), il s'éviterait bien des soucis par la suite...

Et puis, rien n'empêche d'utiliser les fichiers "C style" à l'écriture et les flux à la lecture, si toutes les écritures sont faites (même si j'admets volontiers que ca n'est pas forcément l'idéal )

Sur le fond, je partage ton avis : probablement que les lignes seraient déjà remplacées au vue des questions qu'elles ont générées. Mais, après, il n'a peut être pas le choix : question de risque + coût -> + de code modifié = + de risque de bug = couverture de test plus importante = + cher .

[koala01]

Sur le fond, je partage ton avis : probablement que les lignes seraient déjà remplacées au vue des questions qu'elles ont générées. Mais, après, il n'a peut être pas le choix : question de risque + coût -> + de code modifié = + de risque de bug = couverture de test plus importante = + cher .

Honnetement, je ne crois pas...

Les risques sont largement moindre par l'utilisation des flux, ne serait-ce que parce qu'il supprime celui de se tromper de format dans les fprintf fscanf...

En utilisant les flux, tu supprime la compilation conditionnelle selon un ratio minimum de 5 lignes supprimées pour une ligne écrite, ce qui rend le code bien moins complexe

Les tests d'écriture et de leture ne sont au final pas modifié: tu crées une structure donnée, tu l'écris dans le fichier, tu teste la récupération de la structure en lecture

De plus, j'ai du mal à concevoir que cela puisse représenter des "milliers de lignes de code" à modifier: à coup de 6 lignes (la déclaration et l'ouverture du fichier qui devient une déclaration du flux correspondant et les 5 lignes de compilation conditionnelles) à modifier par fonction, de deux fonction par structure, il faudrait déjà arriver à 100 structures à modifier pour arriver seulement à 1200 lignes (qui se transformeraient en 400 lignes de code)

Le risque de problèmes dans un programme tient bien plus de la complexité du code (qui est exponentielle par rapport au nombre d'instructions) que de la modifcation du code...

De ce point de vue, tu limite bien plus les risques en modifiant correctement ton code qu'en ne le le faisant pas

[uriotcea]

Ok, mon but n'est pas de faire un code super propre pur C++, il conteint du C++ du C du fortran95 et même du fortran77.
Non, actuellement il est robus, presque exent de bug (long long travail de plusieur années) et surtout il est trop utilisé par la communauté (scientifiques) pour que je prennent le risque aujourd'hui de de le remanier completement ou en partie.
Je l'ai optimisé pour enlever tous les warning de maniére à le maintenair plus facilement, il ne me reste que celui que je vous ai indiqué précedemment (des milliers). J'aimerai votre aide pour le retirer le plus proprement possible.

[3DArchi]

Ok, mon but n'est pas de faire un code super propre pur C++, il conteint du C++ du C du fortran95 et même du fortran77.
Non, actuellement il est robus, presque exent de bug (long long travail de plusieur années) et surtout il est trop utilisé par la communauté (scientifiques) pour que je prennent le risque aujourd'hui de de le remanier completement ou en partie.
Je l'ai optimisé pour enlever tous les warning de maniére à le maintenair plus facilement, il ne me reste que celui que je vous ai indiqué précedemment (des milliers).

C'est un peu ce que je voulais dire. Quand ça marche, remplacer du code risqué par du code sûr c'est quand même prendre le risque d'une régression. C'est pour ça qu'en général, c'est assez mal vu dans les boîtes. En même temps, c'est dommage car on conserve du code bancal année après année de peur de "faire s'écrouler le château de carte".

J'aimerai votre aide pour le retirer le plus proprement possible.

Goten t'a donné (presque) la solution :

Code :

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

#pragma GCC diagnostic ignored "-Wformat"

[Médinoc]

Pas de push ou pop sous gcc?

[3DArchi]

Je n'en n'ai pas vu. C'est effectivement gênant.

[loufoque]

Encore heureux que GCC te donne un warning, tu invoques un comportement indéfini.

Ce n'est pas

Code :

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

1
2
3
4
5
#ifdef _WIN32
        fscanf(f,"%d %I64i %lf ",&n,&N0,&ms);
        #else
        fscanf(f,"%d %llu %lf ",&n,&N0,&ms);
        #endif

qu'il faut faire... Le premier cas n'est pas à utiliser sous win32, mais sous Visual C++.

[uriotcea]

C'est un peu ce que je voulais dire. Quand ça marche, remplacer du code risqué par du code sûr c'est quand même prendre le risque d'une régression. C'est pour ça qu'en général, c'est assez mal vu dans les boîtes. En même temps, c'est dommage car on conserve du code bancal année après année de peur de "faire s'écrouler le château de carte".

Oui je suis d'accord, je mene des campagne pour réactualiser et moderniser des parties de code régulierement quand ma charge de travail me le permet, mais aujourd'hui ce n'est pas le cas, c'est la raison de ma demande.

Goten t'a donné (presque) la solution :

Oui mais encore, effectivement cela n'a aucun effet et j'ai beau naviguer sur le web, je ne vois pas la syntax nécessaire

[3DArchi]

Oui mais encore, effectivement cela n'a aucun effet et j'ai beau naviguer sur le web, je ne vois pas la syntax nécessaire

En mettant ignored, j'ai bien la disparition du warning.
Cependant, loufoque a raison :

Encore heureux que GCC te donne un warning, tu invoques un comportement indéfini.

Ce n'est pas [...] qu'il faut faire... Le premier cas n'est pas à utiliser sous win32, mais sous Visual C++.

Il faut que tu changes ta directive de compilation pour modifier selon GCC/Visual et non selon WIN32 ou pas :

Code :

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

1
2
3
4
5
6
 
#ifdef _MSC_VER
        fscanf(f,"%d %I64i %lf ",&n,&N0,&ms);
        #else
        fscanf(f,"%d %llu %lf ",&n,&N0,&ms);
        #endif