Discussion :

[Gamall]

Bonjour,

J'ai un petit problème en essayant de compiler libxml2, je tombe sur cette erreur:

[zaour@joker libxml2-2.7.8]$ i486-mingw32-gcc -DHAVE_CONFIG_H -I. -I./include -I./include -D_REENTRANT -DWIN32 -march=x86-64 -mtune=generic -O2 -pipe -mms-bitfields -pedantic -W -Wformat -Wunused -Wimplicit -Wreturn-type -Wswitch -Wcomment -Wtrigraphs -Wformat -Wchar-subscripts -Wuninitialized -Wparentheses -Wshadow -Wpointer-arith -Wcast-align -Wwrite-strings -Waggregate-return -Wstrict-prototypes -Wmissing-prototypes -Wnested-externs -Winline -Wredundant-decls -MT testThreads.o -MD -MP -MF .deps/testThreads.Tpo -c -o testThreads.o testThreads.c
testThreads.c: In function 'main':
testThreads.c:110:6: error: conversion to non-scalar type requested
[zaour@joker libxml2-2.7.8]$

Voici la portion de code qui pose problème. Je ne comprends pas ligne 24 le '(pthread_t) -1'. Qu'est ce que ça signifie ? pthread_t est un type, ça n'est même pas un objet, ni même un nombre, alors pourquoi soustraire ce 1 ?

Code :

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

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
49
50
51
52
#ifdef HAVE_PTHREAD_H
static pthread_t tid[MAX_ARGC];
#elif defined HAVE_BEOS_THREADS
static thread_id tid[MAX_ARGC];
#endif
 
/* ... */
 
#ifdef HAVE_PTHREAD_H
int
main(void)
{
    unsigned int i, repeat;
    unsigned int num_threads = sizeof(testfiles) / sizeof(testfiles[0]);
    void *results[MAX_ARGC];
    int ret;
 
    xmlInitParser();
    for (repeat = 0;repeat < 500;repeat++) {
	xmlLoadCatalog(catalog);
 
	for (i = 0; i < num_threads; i++) {
	    results[i] = NULL;
	    tid[i] = (pthread_t) -1;
	}
 
	for (i = 0; i < num_threads; i++) {
	    ret = pthread_create(&tid[i], NULL, thread_specific_data,
				 (void *) testfiles[i]);
	    if (ret != 0) {
		perror("pthread_create");
		exit(1);
	    }
	}
	for (i = 0; i < num_threads; i++) {
	    ret = pthread_join(tid[i], &results[i]);
	    if (ret != 0) {
		perror("pthread_join");
		exit(1);
	    }
	}
 
	xmlCatalogCleanup();
	for (i = 0; i < num_threads; i++)
	    if (results[i] != (void *) Okay)
		printf("Thread %d handling %s failed\n", i, testfiles[i]);
    }
    xmlCleanupParser();
    xmlMemoryDump();
    return (0);
}
#elif defined HAVE_BEOS_THREADS

Quelqu'un a -t-il une explication à ça ?

Merci


PS. Ah, oui, je suis bête, ils ont voulu caster -1 en pthread_t, j'avais pas compris au début
Par contre, c'est tout aussi obscure puisque pthread_t est un type opaque (une structure chez moi, en l’occurrence), donc je ne vois pas pourquoi caster 1 en un pthread_t

[Sve@r]

PS. Ah, oui, je suis bête, ils ont voulu caster -1 en pthread_t, j'avais pas compris au début
Par contre, c'est tout aussi obscure puisque pthread_t est un type opaque (une structure chez moi, en l’occurrence), donc je ne vois pas pourquoi caster 1 en un pthread_t

Salut
pthread_t ne peut pas être une structure car tu tentes de lui insérer -1 (de type int). Moi, je verrais pthread_t de type entier, comme size_t. Donc si vraiment c'est une structure, c'est qu'il y a un décalage entre tes headers et ton code...

Quand au cast, c'est juste pour être plus parlant. Comme ceux qui écrivent

Code c :

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

1
2
FILE *fp=fopen("toto", "w");
if (fp == (FILE*)NULL) ...

alors que le cast ne se nécessite pas.

[Gamall]

Euh, si, c'est bien une structure, d'ailleurs, elle est définie comme ça:

Code :

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

1
2
3
4
5
6
typedef struct {
    void * p;                   /* Pointer to actual object */
    unsigned int x;             /* Extra information - reuse count etc */
} ptw32_handle_t;
 
typedef ptw32_handle_t pthread_t;

[jouana]

Bonjour,

Voici un lien sur les threads en espérant que ca puisse t'aider.
pthread

Cordialement

[Sve@r]

Euh, si, c'est bien une structure, d'ailleurs, elle est définie comme ça:

Code :

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

1
2
3
4
5
6
typedef struct {
    void * p;                   /* Pointer to actual object */
    unsigned int x;             /* Extra information - reuse count etc */
} ptw32_handle_t;
 
typedef ptw32_handle_t pthread_t;

Ben ça correspond bien au message d'erreur. Vu sa définition, tu ne peux pas écrire

Code c :

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

pthread_t toto=-1;

. Donc gros décalage entre header et source...

[Gamall]

Ça a une grosse incidence si j'enlève la ligne en question ? En fait, je ne vois même pas l'intérêt, vu que de toute façon, les threads sont créés par la suite, et la valeur de tid[i] sera écrasée:

Code :

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

1
2
3
4
5
6
7
8
9
 
for (i = 0; i < num_threads; i++) {
    ret = pthread_create(&tid[i], NULL, thread_specific_data,
			 (void *) testfiles[i]);
    if (ret != 0) {
	perror("pthread_create");
	exit(1);
    }
}

[Sve@r]

Ça a une grosse incidence si j'enlève la ligne en question ? En fait, je ne vois même pas l'intérêt, vu que de toute façon, les threads sont créés par la suite, et la valeur de tid[i] sera écrasée:

Code :

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

1
2
3
4
5
6
7
8
9
 
for (i = 0; i < num_threads; i++) {
    ret = pthread_create(&tid[i], NULL, thread_specific_data,
			 (void *) testfiles[i]);
    if (ret != 0) {
	perror("pthread_create");
	exit(1);
    }
}

A priori non. Mais on peut penser que si t'as une discordance entre header et code, tu pourras trouver d'autres erreurs plus tard.
De plus, on ne sait pas ce que fait la fonction pthread_create(). Peut-être qu'elle va tester, à un moment donné, si tid[i] vaut -1.
De toute façon tu ne risques rien à mettre la ligne 24 en commentaires et recompiler voir ce que ça donne...

[sharky.fr]

Bonsoir,

J'ai jeté un coup d'oeil aux sources de pthread et les 3 lignes du dessus sont intéressantes:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
#if defined(_UWIN) && PTW32_LEVEL >= PTW32_LEVEL_MAX
#   include     <sys/types.h>
#else
/*
 * Generic handle type - intended to extend uniqueness beyond
 * that available with a simple pointer. It should scale for either
 * IA-32 or IA-64.
 */
typedef struct {
    void * p;                   /* Pointer to actual object */
    unsigned int x;             /* Extra information - reuse count etc */
} ptw32_handle_t;
 
typedef ptw32_handle_t pthread_t;

source

On jète donc un coup d'oeil dans sys/types.h:

Code :

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

1
2
3
4
5
 
/* Now add the thread types.  */
#if defined __USE_POSIX199506 || defined __USE_UNIX98
# include <bits/pthreadtypes.h>
#endif

Puis de même dans bits/pthreadtypes.h:

Code :

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

1
2
3
4
 
/* Thread identifiers.  The structure of the attribute type is not                                                                                                                                         
   exposed on purpose.  */
typedef unsigned long int pthread_t;

Il semble que pthread_t puisse être défini comme alias de unsigned long.
Affecter -1 positionne donc tous les bit à 1.

A la rigueur tu peux utiliser memset().
Mais comme l'a justement relevé Sve@r, il y aura surement d'autres problèmes plus tard.

Au pire, ne peux tu pas te servir d'une version pré-compilée ?

Bon courage.