Warning avec <list>

Version imprimable

Warning avec <list>

Bonjour,

j'ai ce warning bizarre avec MinGW:

Code:

1
2
3
Compiling: utilities.cpp
C:/Program Files/CodeBlocks/MinGW/bin/../lib/gcc/mingw32/3.4.5/../../../../include/c++/3.4.5/bits/stl_list.h: In member function `void FileArchiver::add_file(const char*)':
C:/Program Files/CodeBlocks/MinGW/bin/../lib/gcc/mingw32/3.4.5/../../../../include/c++/3.4.5/bits/stl_list.h:435: warning: '__p' might be used uninitialized in this function

Voilà mon code:

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
/* classe pour stocker plusieurs fichiers dans un seul */
class FileArchiver
{
    public:
        struct entete
        {
            std::string nom;
            unsigned long long pos;
            unsigned long long len;
            entete(std::string nom, unsigned long long pos, unsigned long long len) : nom(nom), pos(pos), len(len) {}
        };
 
        std::ofstream out;
        std::list<entete> entetes;
        unsigned long long cur_pos;
 
        /*prend en paramètre le nom de l'archive */
        FileArchiver(const char *path);
 
        //rajoute un fichier à l'archive
        void add_file(const char *path);
        //ajoute les entêtes à la fin
        void finish();
};

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
36
37
38
39
40
41
42
43
44
45
46
47
48
FileArchiver::FileArchiver(const char *path) :out(path, ios::binary), entetes()
{
    if (!out)
    {
        throw string("Erreur lors du chargment dans FileArchiver: " +string (path));
    }
 
    cur_pos = 0;
}
 
void FileArchiver::add_file(const char *path)
{
    unsigned long long size;
 
    ifstream in(path, ios::binary);
 
    if (!in)
    {
        throw string ("Erreur lors de l'addition d'un fichier: " + string(path));
    }
 
    // get pointer to associated buffer object
    filebuf *pbuf=in.rdbuf();
 
    // get file size using buffer's members
    size=pbuf->pubseekoff (0,ios::end,ios::in);
    pbuf->pubseekpos (0,ios::in);
 
    char buffer[size];
    pbuf->sgetn (buffer,size);
 
    out.write(buffer, size);
 
    entetes.push_back(entete(path, cur_pos, size));
    cur_pos += size;
}
 
void FileArchiver::finish()
{
    list<entete>::iterator it;
 
    for (it = entetes.begin(); it != entetes.end(); ++it)
    {
        out << it->nom << " " << it->pos << " " << it->len << endl;
    }
 
    out << ((entetes.back()).pos + (entetes.back()).len) << flush;
}

Je pense que le warning se produit à cette ligne:

Code:

entetes.push_back(entete(path, cur_pos, size));

entetes est une list<entete> (voir la définition de la classe).

A l'exécution, donc j'ai le warning montré plus haut. Le warning parle de cette fonction ci de la STL (en disant __p might not be initialized here...):

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
      _Node*
      _M_create_node(const value_type& __x)
      {
	_Node* __p = this->_M_get_node();
	try
	  {
	    std::_Construct(&__p->_M_data, __x);
	  }
	catch(...)
	  {
	    _M_put_node(__p);
	    __throw_exception_again;
	  }
	return __p;
      }

Faut-il que j'initialise ma liste quelque part? sachant que pour l'instant tout marche très bien.

Pour ceux qui me diraient de ne pas réinventer la roue, je n'ai trouvé aucune lib faisant ça (il y a bien zzip, mais j'arrive pas à l'installer...).

Merci!

24/04/2008, 14h06
Mat007
Salut,

C'est un warning pour lequel tu ne peux pas grand chose...
Tout ce que tu peux faire c'est mettre l'inclusion à <list> dans un .h séparé avec juste avant un truc du style :
Code:

1 2 3 #ifdef __GNUC__ # pragma GCC system_header #endif
Ce qui a pour effet de désactiver les warnings pour le fichier courant.

MAT.

Citation:
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 void FileArchiver::add_file(const char *path) { unsigned long long size; ifstream in(path, ios::binary); if (!in) { throw string ("Erreur lors de l'addition d'un fichier: " + string(path)); } // get pointer to associated buffer object filebuf *pbuf=in.rdbuf(); // get file size using buffer's members size=pbuf->pubseekoff (0,ios::end,ios::in); pbuf->pubseekpos (0,ios::in); char buffer[size]; pbuf->sgetn (buffer,size); out.write(buffer, size); entetes.push_back(entete(path, cur_pos, size)); cur_pos += size; }
Les VLAs ne font pas partie du C++, étant donné qu'ils posent quelques problèmes.
Ensuite, pourquoi tu déclares size sans la définir ? Attends de pouvoir définir une variable avant de la déclarer.
Tu devrais initialiser cur_pos dans le constructeur-initialiseur, pas dans le constructeur.
Pas de using namespace std à la portée d'un fichier, surtout dans une entête... Et inutile de copier la string encore une fois quand tu lèves une exception...

Citation:

Faut-il que j'initialise ma liste quelque part? sachant que pour l'instant tout marche très bien.

valgrind te dit rien ?

24/04/2008, 15h56
coyotte507

Salut,

les VLAs c'est trop pratique pour que je m'en passe.
Par contre la string oui c'est trop bête je vais changer ça.

Le using namespace std, je ne l'ai pas mis dans l'entête (tu peux remarquer que dans l'entête j'utilise std::string, etc.)

Merci pour la méthode pour enlever le warning (je n'ai pas de valgrind)

Edit: La méthode pour enlever le warning avec minGW ça marche pas pour moi :s

Je vois pas le problème avec cur_pos, j'ai qu'un constructeur?

Merci de votre aide en tout cas ;)
24/04/2008, 16h07
Mat007

Citation:

Envoyé par coyotte507

La méthode pour enlever le warning avec minGW ça marche pas pour moi :s

C'est pas forcément étonnant en fait à la réflexion...
Le warning apparait lors de la compilation et non de l'inclusion de <list>, c'est ça ? Si tu commentes la ligne avec le push_back il disparait ?

MAT.
24/04/2008, 16h15
coyotte507

Oui exactement, j'ai 0 warnings dans ce cas ^^
@loufoque: Pour cur_pos, je vais le mettre dans la liste d'initialisation (j'avais pas compris)
24/04/2008, 16h57
loufoque

valgrind a lui seul vaut bien l'installation d'un linux