Discussion :

[blazed]

Bonjour,

en netoyant quelques unes de mes fonctions favorites, j'ai voulu optimiser un peu et debuger avec Valgrind ma petite librairie.

Mais la c'est le drame, je tombe donc sur un probleme mineur mais je vois pas quoi faire.

Le message de valgrind est celui-ci :
Conditional jump or move depends on uninitialised value(s).

Et voici mon code xD:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int     wtablen(char **wtab)
{
  int   i;
 
  i = 0;
  if (wtab)
    while (wtab[i])
      i++;
  return (i);
}

i est initialise, je ne pense pas aller trop loin dans le tableau...

Merci pour votre aide

[Obsidian]

Bonsoir,

Essaie

Code C :

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

  if (wtab==NULL)

… si c'est bien cette macro que tu utilises. A priori, NULL est résolue en ((void *)0) mais ce n'est pas une garantie. Cela dit, ce code est valide quand même valide, même si transtyper directement un pointeur en entier ou en booléen n'est pas très propre. Je ne sais pas pourquoi valgrind se plaint.

Et voici mon code xD:

Code :

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

1
2
3
4
5
6
7
 
int     wtablen(char **wtab)
{
    …
 
    return (i);
}

« return » est un mot-clé, pas une fonction. Les parenthèses sont inutiles.

[orfix]

Le problème est sûrement ailleurs, peut-on avoir un code compilable reproduisant le problème ?

[LittleWhite]

J'utilise moi aussi de plus en plus valgrind, et je me demande si avec les options -v pour valgrind lors de l'executon, et l'option -g pour gcc à la compilation, on ne peut pas avoir un message plus précis. Du genre, indication de la ligne

Sinon, mais je pense que j'ai tord,

Code :

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

1
2
3
4
 
int i = 0;
 
i = 0;

Expliciter la définition de i, lors de sa déclaration ...

[blazed]

Bonjour,

merci de toutes ces reponces, je n'ai toujours pas trouver de solution pour resoudre mon probleme. Je reposterais dans ce topic si je trouve au file des programmes

Pour les parentheses sur le retrun c'est parce que j'aime bien ^^

J'utilise ces flags pour valgrind :
valgrind --leak-check=yes --show-reachable=yes

Merci encore et au plaisir.

[Obsidian]

Pour les parentheses sur le retrun c'est parce que j'aime bien ^^

Tu fais comme tu veux, mais tu te forces ainsi à prendre une mauvaise habitude. Ce n'est pas forcément ce qu'il y a de plus sensé. Cela risque de te conduire à terme à des effets de bords de ceux que tu rencontres aujourd'hui.

[LittleWhite]

Et puis peut être que vos collègues ne vont pas apprécié ( compagnie ). Du coup, vous allez vite changé. ( Surtout qu'un gars comme moi, il va vous le dire de changer, et risque de corriger ... )

[diogene]

Le message de valgrind est celui-ci :
Conditional jump or move depends on uninitialised value(s).

Ce message ne peut-il pas provenir du fait que l'argument x correspondant au paramètre wtab de la fonction n'est pas initialisé au moment de l'appel ?
Du genre :

Code :

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

1
2
3
char ** x;
....
wtablen(x);

bien que, dans ce cas, l'erreur devrait être signalée sur la ligne de l'appel (mais c'est peut-être lié à un niveau d'optimisation du code ?).

[blazed]

diogene, merci pour l'idée je vais regarder dans ce sens car en effet j'utilise une moulinette de test pour différent cas de figure et cela pourrait provenir des tests.

Mais ne vous inquiétez pas, si je devais travailler avec vous je changerais mon mode opératoire sans problème (et puis même si j'aime bien, c'est dans ma norme :p )