warnings being treated as errors

Version imprimable

bonjour, je n'arrive pas à résoudre une erreur concernant un thread, j'ai une erreur de compilation qui concerne le constructeur de ma classe, et je ne voit pas le rapport entre le constructeur de la classe et le thread. J'aimerais qu'on m'explique pourquoi j'ai cette erreur de compilation.

voici le code de ma classe :

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
 
#include "Interpretor.h"
#include "Control.h"
 
using namespace std;
 
 
Control :: Control(Chlist * clist)
{
	this->clist = clist;
	cout << "Création de l'objet Thread "<< endl;
	Start();
}
 
 
Control :: ~Control(void)
{
 
}
 
void Control::Start(void)
{
        cout << "Démarrage du thread"<<": debut.\n";
        pthread_create(thread,NULL,&Control::Run,(void *)&clist);
        cout << "Démarrage du thread"<< ": fait.\n";
} 
 
void Control::Join(void)
{
        pthread_join(*thread,NULL);
}
 
void * Control::Run(void * clist)
{
       int i=0;
       Chlist *m_ctrl= (Chlist*)clist;
       m_ctrl->printAll();
       while(1)
       {
              i++;
              cout << " execution n°" << i << endl;
              sleep(1);
       }
       return NULL;
}

voici le point h:

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
 
#ifndef __CONTROL_H__
#define __CONTROL_H__
 
#include <iostream>
#include <string>
#include <sstream>
#include <vector>
 
extern "C"
{
	#include <unistd.h>
	#include <stdio.h>
	#include <stdlib.h>
	#include <sys/socket.h>
	#include <sys/types.h>
	#include <sys/stat.h>
        #include <fcntl.h>
        #include <dirent.h>
        #include <pthread.h>
}
#include <pthread.h>
#ifndef BUFFER_SIZE
#define BUFFER_SIZE 1024
#endif
 
using namespace std;
 
 
class Control
{
	public:
	   Control (Chlist * clist);
	   ~Control (void);
	   void Start(void);
	   void Join(void);
	   static void *Run(void * clist);
 
    private:
	  pthread_t * thread;
	  pthread_attr_t * attr;
	  Chlist * clist;
 
};
 
#endif

voici mon erreur de compilation :

Code:

1
2
3
4
5
 
cc1plus: warnings being treated as errors
Control.cpp: In constructor ‘Control::Control(Chlist*)’:
Control.cpp:7: error: declaration of ‘clist’ shadows a member of 'this'
make: *** [Control.o] Erreur 1

Merci de votre compréhension.

06/04/2012, 07h14
DakM
this-> ? :cfou:

1. Supprime dans ton cpp le constructeur
2. Dans le header,
Code:

1 2 3 4 5 6 7 8 class Control { public: Control (Chlist * clistarg) : clist(clistarg) {}; void Init(); };
3. A nouveau dans le cpp
Code:

1 2 3 4 void Control::Init() { Start(); }
Le probleme est simple, tu essayes d'acceder aux membres de ta classe avec this-> alors que c'est inutile (:weird:)
Suffit simplement de la meme facon que tu accedes a la fonction membre void Start(); d'écrire Start(); clist = X; etc.
06/04/2012, 09h19
Bousk
Bonjour,

Ton paramètre du constructeur s'apelle clist et... une variable membre s'apelle clist.
Inutile d'aller plus loin pour se rendre compte de l'erreur.:aie:
En théorie, il me semblait que le this->clist = clist; permettait de réaliser cette opération.
En pratique, ne jamais avoir de paramètre avec le même nom qu'un membre. C'est évident : comment savoir si tu veux utiliser le paramètre ou le membre ?!

Enfin, il est mieux d'utiliser les listes d'initialisation pour le constructeur :
Code:

1 2 3 4 5 Control::Control(Chlist* _clist) : clist(_clist) { cout << "Création de l'objet Thread "<< endl; Start(); }

Citation:
Envoyé par l'unique
voici mon erreur de compilation :
Code:

1 2 3 4 5 cc1plus: warnings being treated as errors Control.cpp: In constructor ‘Control::Control(Chlist*)’: Control.cpp:7: error: declaration of ‘clist’ shadows a member of 'this' make: *** [Control.o] Erreur 1
En fait, ce n'est pas exactement une erreur, mais un avertissement, que ton compilateur "promeut" en erreur ("cc1plus: warnings being treated as errors"). Tu dois avoir un paramètre de compilation qui fait cela.

Comme d'autres l'ont dit, l'avertissement porte sur le fait que tu passes au constructeur un paramètre qui a le même nom qu'un de ses membres. Ce n'est pas incorrect, et ton code fonctionnera (this->clist et les règles du langage lèvent l'ambiguité), mais ça reste une pratique dangereuse: quand plusieurs choses différentes ont le même nom, une erreur est vite arrivée.

C'est pour cela que le compilateur émet un avertissement.

Francois

bonjour, j'ai modifié le code , il compile bien, cependant il me fait une erreur de segmentation au niveau du pthread_create(), j'ai utilisé gdb pour connaître où se situe l'erreur. Cela me paraît aussi blizzard puisque si j'aurais mis les mauvais arguments dans pthread_create(), j'aurais une erreur de compilation, et ce n'est pas le cas. Pouvez vous m'expliquer pourquoi j'ai cette erreur de segmentation?

Voici le nouveau code cpp:

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
 
#include "Interpretor.h"
#include "Control.h"
 
using namespace std;
 
 
Control::Control(Chlist* _clist) : clist(_clist)
{
	cout << "Création de l'objet Thread "<< endl;
	Init();
}
 
Control :: ~Control(void)
{
}
 
void Control::Init()
{
	   Start();
}
void Control::Start(void)
{
	cout << "Démarrage du thread"<<": debut.\n";
	pthread_create(thread,NULL,&Control::Run,(void *)clist);
	cout << "Démarrage du thread"<< ": fait.\n";
} 
 
void Control::Join(void)
{
	pthread_join(*thread,NULL);
}
 
void * Control::Run(void * clist)
{
	int i=0;
	Chlist *m_ctrl= (Chlist*)clist;
	m_ctrl = NULL;
	while(1)
	{
		i++;
		cout << " execution n°" << i << endl;
		sleep(1);
	}
	return NULL;
}

Voici le nouveau point h :

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
 
#ifndef __CONTROL_H__
#define __CONTROL_H__
 
#include <iostream>
#include <string>
#include <sstream>
#include <vector>
 
extern "C"
{
	#include <unistd.h>
	#include <stdio.h>
	#include <stdlib.h>
	#include <sys/socket.h>
	#include <sys/types.h>
	#include <sys/stat.h>
        #include <fcntl.h>
        #include <dirent.h>
	#include <pthread.h>
}
#include <pthread.h>
#ifndef BUFFER_SIZE
#define BUFFER_SIZE 1024
#endif
 
using namespace std;
 
 
class Control
{
	public:
 
		Control(Chlist* _clist);
		~Control (void);
		void Init();
		void Start(void);
		void Join(void);
		static void *Run(void * clist);
 
    private:
	  pthread_t * thread;
	  pthread_attr_t * attr;
	  Chlist * clist;
 
};
 
#endif

Voici ce que me dit gdb :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Program received signal SIGSEGV, Segmentation fault.
0x00135454 in __pthread_create_2_1 (newthread=0x0, attr=0x0, start_routine=0x804a650 <Control::Run(void*)>, 
    arg=0x8059170) at pthread_create.c:558
558     pthread_create.c: Aucun fichier ou dossier de ce type.
        in pthread_create.c
(gdb) bt
#0  0x00135454 in __pthread_create_2_1 (newthread=0x0, attr=0x0, 
    start_routine=0x804a650 <Control::Run(void*)>, arg=0x8059170) at pthread_create.c:558
#1  0x0804a753 in Control::Start (this=0x8059210) at Control.cpp:25
#2  0x0804e76b in Interpretor::Interpretor (this=0x8059180, clist=0x8059170) at Interpretor.cpp:34
#3  0x08054229 in Site::Site (this=0x8059028, filename=..., p_port=10000) at site.cpp:24
#4  0x080528cb in main (argc=3, argv=0xbffff324) at main.cpp:45

Merci de votre compréhension.

08/04/2012, 18h54
Bousk

Où est initialisé thread ?

bonjour, pour pouvoir utiliser mon thread je l'ait déclarer dans le .h comme cela: "pthread_t * thread;", une fois que mon thread est déclaré, il est lié à une fonction avec "pthread_create()".J'ai vu des exemples qui font comme cela.
Je ne sais pas si ma façon de faire est correcte?

Voici comment j'ai fais dans le .cpp:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 Control::Control(Chlist* _clist) : clist(_clist) { cout << "Création de l'objet Thread "<< endl; Init(); } Control :: ~Control(void) { } void Control::Init() { Start(); } void Control::Start(void) { cout << "Démarrage du thread"<<": debut.\n"; pthread_create(thread,NULL,&Control::Run,(void *)clist); cout << "Démarrage du thread"<< ": fait.\n"; }
Dois je initialisé le thread et où et comment?

16/04/2012, 11h02
Kaamui
je ne sais pas ce que fait pthread_create, mais définir un pointeur sans l'initialiser est une erreur. Cela dépend de ce que fait pthread_create, mais tu devrais mettre dans ton constructeur :
Code:

1 2 3 4 5 6 7 8 9 10 Control::Control(Chlist* _clist) : clist(_clist), //thread(null) //ou //thread( new thread() ) { cout << "Création de l'objet Thread "<< endl; Init(); }
Mais je penses que Bousk pourra t'aiguiller mieux que moi ;)
16/04/2012, 23h34
l'unique

bonjour, mon problème est résolu il fallait le définir sans pointeur de la façon suivante pthread_t thread;, et utiliser pthread_create de la façon suivante : pthread_create(&thread,NULL,&Control::Run,(void *)&clist);
:ccool:
26/04/2012, 10h17
Médinoc

Normalement, tu ne devrais même pas avoir besoin du cast explicite: Dans ce cas, retire-le.