Conversion vers un type non-scalaire

Version imprimable

Bonjour,

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

Citation:

[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 ? 8O
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 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 :ccool:

PS. Ah, oui, je suis bête, ils ont voulu caster -1 en pthread_t, j'avais pas compris au début :aie:
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 8O

06/07/2011, 07h05
Sve@r
Citation:

Envoyé par artificier59

PS. Ah, oui, je suis bête, ils ont voulu caster -1 en pthread_t, j'avais pas compris au début :aie:
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 8O

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:

1 2 FILE *fp=fopen("toto", "w"); if (fp == (FILE*)NULL) ...
alors que le cast ne se nécessite pas.

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

Code:

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;

06/07/2011, 18h01
jouana

Bonjour,

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

Cordialement

Citation:
Envoyé par artificier59
Euh, si, c'est bien une structure, d'ailleurs, elle est définie comme ça:
Code:

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:

pthread_t toto=-1;

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

Ç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:

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); } }

Citation:
Envoyé par artificier59
Ç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:

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...

quelques corrections

Bonsoir,

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

Code:

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:

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:

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.