私はPthreadsを学んでいます。私のコードは私が望むように実行し、私はそれを使用することができます。しかし、コンパイルに関する警告が表示されます。
私は次を使用してコンパイルします:
gcc test.c -o test -pthread
gCC 4.8.1で。そして、私は警告を受け取ります
test.c: In function ‘main’:
test.c:39:46: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
pthread_create(&(tid[i]), &attr, runner, (void *) i);
^
test.c: In function ‘runner’:
test.c:54:22: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
int threadnumber = (int) param;
^
このエラーは次のコードで発生します。
#include <pthread.h>
#include <stdlib.h>
#include <stdio.h>
#define MAX_THREADS 10
int sum; /* this data is shared by the thread(s) */
void *runner(void * param);
int main(int argc, char *argv[])
{
int num_threads, i;
pthread_t tid[MAX_THREADS]; /* the thread identifiers */
pthread_attr_t attr; /* set of thread attributes */
if (argc != 2) {
fprintf(stderr, "usage: test <integer value>\n");
exit(EXIT_FAILURE);
}
if (atoi(argv[1]) <= 0) {
fprintf(stderr,"%d must be > 0\n", atoi(argv[1]));
exit(EXIT_FAILURE);
}
if (atoi(argv[1]) > MAX_THREADS) {
fprintf(stderr,"%d must be <= %d\n", atoi(argv[1]), MAX_THREADS);
exit(EXIT_FAILURE);
}
num_threads = atoi(argv[1]);
printf("The number of threads is %d\n", num_threads);
/* get the default attributes */
pthread_attr_init(&attr);
/* create the threads */
for (i=0; i<num_threads; i++) {
pthread_create(&(tid[i]), &attr, runner, (void *) i);
printf("Creating thread number %d, tid=%lu \n", i, tid[i]);
}
/* now wait for the threads to exit */
for (i=0; i<num_threads; i++) {
pthread_join(tid[i],NULL);
}
return 0;
}
/* The thread will begin control in this function */
void *runner(void * param)
{
int i;
int threadnumber = (int) param;
for (i=0; i<1000; i++) printf("Thread number=%d, i=%d\n", threadnumber, i);
pthread_exit(0);
}
この警告を修正するにはどうすればよいですか?
簡単なハック修正は、long
ではなくint
にキャストするだけです。多くのシステムでは、sizeof(long) == sizeof(void *)
。
より良いアイデアは、intptr_t
。
int threadnumber = (intptr_t) param;
そして
pthread_create(&(tid[i]), &attr, runner, (void *)(intptr_t)i);
_pthread_create(&(tid[i]), &attr, runner, (void *) i);
_
i
、sizeof(void*) == 8
、およびsizeof(int) == 4
(64ビット)の引数としてローカル変数runner
を渡します。
i
を渡したい場合は、ポインタまたは何かとしてラップする必要があります。
_void *runner(void * param) {
int id = *((int*)param);
delete param;
}
int tid = new int; *tid = i;
pthread_create(&(tid[i]), &attr, runner, tid);
_
単にi
が必要な場合がありますが、その場合、次のことは安全なはずです(ただし、推奨にはほど遠い)。
_void *runner(void * param) {
int id = (int)param;
}
pthread_create(&(tid[i]), &attr, runner, (void*)(unsigned long long)(i));
_
私も同じ警告を受けていました。そのため、警告を解決するためにintをlongに変換すると、この警告は消えました。 「異なるサイズの整数からポインターにキャストする」警告については、64xのポインターは64ビットであり、32xのポインターは32ビットであるため、ポインターは任意の変数の値を保持できるため、この警告を残すことができます。