Avertissements à la compilation

Version imprimable

Ndm : Déplacé depuis cette discussion.

Bonjour ! J'ai voulu tester la bibliothèque proposée dans l'article. J'ai dû corriger une ligne (tout à la fin du fichier str.c) :

Code:

1
2
      //strng[j] = '\0';
      strip[j] = '\0';

Après cela, j'arrive à compiler mon code mais j'ai un certain nombre d'avertissements. J'aurais voulu savoir si cela venait d'une erreur de ma part. J'utilise le compilateur Borland C++ 5.5.1, sous Windows 8.1. Voici mon code, avec les avertissements.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
#include <stdio.h>
#include "str.h"
#include "str.c"
 
void main(void)
{
  getchar();
}
 
/*
> Executing: C:\Program Files (x86)\ConTEXT\ConExec.exe "C:\BCC55\Bin\bcc32.exe" -nC:\Atelier\C\strings\str\ "C:\Atelier\C\strings\str\test.c"
 
Borland C++ 5.5.1 for Win32 Copyright (c) 1993, 2000 Borland
C:\Atelier\C\strings\str\test.c:
Warning W8004 C:\Atelier\C\strings\str\str.c 58: 'ptr_pos' is assigned a value that is never used in function str_istr
Warning W8012 C:\Atelier\C\strings\str\str.c 80: Comparing signed and unsigned values in function str_sub
Warning W8060 C:\Atelier\C\strings\str\str.c 105: Possibly incorrect assignment in function str_split
Warning W8012 C:\Atelier\C\strings\str\str.c 107: Comparing signed and unsigned values in function str_split
Warning W8004 C:\Atelier\C\strings\str\str.c 109: 'tmp' is assigned a value that is never used in function str_split
Warning W8004 C:\Atelier\C\strings\str\str.c 102: 'cs' is assigned a value that is never used in function str_split
Warning W8004 C:\Atelier\C\strings\str\str.c 143: 'tmp' is assigned a value that is never used in function str_join
Warning W8008 C:\Atelier\C\strings\str\str.c 176: Condition is always true in function str_remplace
Turbo Incremental Link 5.00 Copyright (c) 1997, 2000 Borland
> Execution finished.
*/

27/04/2014, 13h18
neuneutrinos

Code:

Warning W8004 C:\Atelier\C\strings\str\str.c 58: 'ptr_pos' is assigned a value that is never used in function str_istr

regarde à la ligne 58 de str.c et regarde comment tu utilises ptrpos et ce que tu en fait ;)

Citation:

Envoyé par neuneutrinos

Code:

Warning W8004 C:\Atelier\C\strings\str\str.c 58: 'ptr_pos' is assigned a value that is never used in function str_istr

regarde à la ligne 58 de str.c et regarde comment tu utilises ptrpos et ce que tu en fait ;)

Merci pour ta réponse. En fait je doutais s'il y avait vraiment des choses à corriger dans le code ou si le problème venait de l'utilisation que j'en faisais.

Voici la modification qui permet de supprimer l'avertissement en question. Effectivement ce n'était pas trop difficile à trouver.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 /* Borland C++ 5.5.1 */ #include <stdio.h> #include <stdlib.h> #include <string.h> #include <ctype.h> int str_istr (const char *cs, const char *ct) { int index = -1; if (cs != NULL && ct != NULL) { //char *ptr_pos = NULL; // 'ptr_pos' is assigned a value that is never used char *ptr_pos; ptr_pos = strstr(cs, ct); if (ptr_pos != NULL) { index = ptr_pos - cs; } } return index; } void main(void) { char s2[] = "Developpez"; printf("%d\n", str_istr(s2, "opp")); printf("Appuyez sur la touche Entr%ce... ", (char)130); getchar(); }
Je m'attaque au reste. :)

Voici pour le second avertissement. Il ne laisse rien passer, ce compilateur ! :)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
char *str_sub(const char *s, unsigned int start, unsigned int end)
{
  char *new_s = NULL;
 
  if (s != NULL && start < end)
  {
    new_s = malloc(sizeof(*new_s) * (end - start + 2));
 
    if (new_s != NULL)
    {
      //int i; // Comparing signed and unsigned values
      unsigned int i;
 
      for (i = start; i <= end; i++)

27/04/2014, 14h35
neuneutrinos

Le premier warning n'étais pas trop grave.

Mais celui-çi est plus important.
Les opérations entre signed et unsigned sont légales mais peuvent entraîner une faille de sécurité (integer overflow)

EDIT:

j'ai regarder la suite des warnings :p

Citation:

Warning W8060 C:\Atelier\C\strings\str\str.c 105: Possibly incorrect assignment in function str_split

ça sent le code compacte ou alors (par exemple) un '==' transformé accidentellement en '='

Warning W8008 C:\Atelier\C\strings\str\str.c 176: Condition is always true in function str_remplace
la je suis prêt à parier que tu as fais une assignation dans ton if

Citation:

Envoyé par neuneutrinos

Les opérations entre signed et unsigned sont légales mais peuvent entraîner une faille de sécurité (integer overflow).

Merci pour ta remarque. C'est noté !

Pour les quatre avertissements suivants, qui concernent tous la même fonction, il y en a trois qui reviennent aux cas précédents, mais le sens du quatrième est moins clair, du moins pour moi. :roll:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 char **str_split(char *s, const char *ct) { char **tab = NULL; if (s != NULL && ct != NULL) { //int i; unsigned int i; //char *cs = NULL; char *cs; size_t size = 1; for (i = 0; (cs = strtok(s, ct)); i++) // Possibly incorrect assignment (?) { if (size <= i + 1) { //void *tmp = NULL; void *tmp;

27/04/2014, 14h54
neuneutrinos

Code:

for (i = 0; (cs = strtok(s, ct)); i++) // Possibly incorrect assignment (?)

Le compilateur te met en garde pour une possibilité d'érreur dans le traitement d'une condition

Code:

(cs = strtok(s, ct))

mais ici cette compactions est voulue.

Pour signaler que cette écriture est volontaire tu dois rajouter !=0 (ou !=NULL)

Code:

for (i = 0; (cs = strtok(s, ct))!=NULL; i++) // Possibly incorrect assignment (?)

Ce warning peut sûrement être désactivé das les options de compilations
27/04/2014, 15h27
Roland Chastain

Citation:

Envoyé par neuneutrinos

Pour signaler que cette écriture est volontaire tu dois rajouter !=0 (ou !=NULL).

Code:

for (i = 0; (cs = strtok(s, ct))!=NULL; i++) // Possibly incorrect assignment (?)

Ce warning peut sûrement être désactivé dans les options de compilation.

Bien vu. Merci !

Il ne me reste plus qu'à trouver un exemple d'utilisation de cette fonction. Je ne comprends pas bien ce que signifient les deux astérisques. Un pointeur de pointeur ? :roll:

Autrement, il n'y a plus d'avertissements.

char** est un tableau de chaine de caractère

et tu en a déjà vu un :p

Code:

1
2
3
4
 
int main(int argc,char *argv[])
//ou bien
int main(int argc,char **argv)

Code:

1
2
3
4
5
6
 
int main(int argc,char **argv)
{
 printf("Executable lancé avec la commande :%s\n",argv[0]);//ou bien *argv
 return 0;
}

Merci pour ta réponse. Voici mon exemple. Comme il fonctionne, je suppose qu'il est bon. :)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
void main(void)
{
  char s[] = "abc def ghi";
 
  //printf("%s\n", str_split(s, " ")[0]);
 
  char **t = str_split(s, " ");
 
  //printf("%s\n", t[0]);
 
  unsigned int i = 0;
 
  while (t[i] != NULL)
  {
    printf("%s\n", t[i]);
    i++;
  }
 
  printf("Appuyez sur la touche Entr%ce... ", (char)130);
  getchar();
}

Un autre exemple, un peu plus utile.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
void main(void)
{
  char s[2048];
  char **t;
  unsigned int i;
 
  strcpy(s, getenv("PATH"));
  t = str_split(s, ";");
 
  i = 0;
  while (t[i] != NULL)
  {
    printf("%s\n", t[i]);
    i++;
  }
 
  printf("Appuyez sur la touche Entr%ce... ", (char)130);
  getchar();
}

Citation:

Envoyé par neuneutrinos

Warning W8008 C:\Atelier\C\strings\str\str.c 176: Condition is always true in function str_remplace
là je suis prêt à parier que tu as fais une assignation dans ton if

Je n'avais pas vu la suite de ton message. La modification à faire était celle-ci :
Code:

1 2 3 4 5 6 char *str_remplace (const char *s, unsigned int start, unsigned int length, const char *ct) { char *new_s = NULL; //if (s != NULL && ct != NULL && start >= 0 && length > 0) if (s != NULL && ct != NULL && length > 0)
Je précise que le code n'est pas de moi ; c'est celui qui est proposé dans cet article.

L'article est très bien (autant que je puis en juger), mais il y aurait une faute de frappe à corriger dans le code (voir mon premier message). Quant aux autres modifications que j'ai faites pour moi-même et que j'ai rapportées ici, elles ne sont pas vraiment nécessaires (encore que ce soit toujours plus agréable d'avoir un code qui se compile sans avertissement).

C'est dommage aussi, me semble-t-il, qu'aucun exemple d'utilisation de la librairie n'ait été fourni, mais vu la date de l'article, il est sans doute un peu tard pour y revenir. :)

Je marque la discussion comme résolue.

Je reviens vers cette discussion pour ajouter un dernier élément.

Citation:

Envoyé par neuneutrinos

Ce warning peut sûrement être désactivé dans les options de compilation.

J'ai trouvé comment désactiver les avertissements, de façon sélective, à partir du code, et toujours pour le compilateur que j'utilise, à savoir Borland C++ 5.5. Je me suis amusé à faire de petits exemples correspondants aux différents types d'avertissements rencontrés dans cette discussion. Peut-être cela intéressera-t-il quelqu'un ?

Valeur non utilisée :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 // Warning W8004: 'i' is assigned a value that is never used in function main #ifdef __BORLANDC__ #pragma warn -8004 #endif void main(void) { int i = 0; i = 1; }
Condition toujours vraie :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 // Warning W8008: Condition is always true in function main #include <stdio.h> #ifdef __BORLANDC__ #pragma warn -8008 #endif void main(void) { unsigned int i = 0; if (i >= 0) getchar(); }
Comparaison entre types signé et non signé :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 // Warning W8012: Comparing signed and unsigned values in function main #include <stdio.h> #ifdef __BORLANDC__ #pragma warn -8012 #endif void main(void) { int i = 0; unsigned int u = 0; if (i > u) getchar(); }
Possibilité d'une assignation involontaire :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 // Warning W8060: Possibly incorrect assignment in function main #include <stdio.h> #ifdef __BORLANDC__ #pragma warn -8060 #endif void main(void) { int i; if (i = 0) getchar(); }