web-dev-qa-db-ja.com

警告:異なるサイズの整数への/からのポインターへのキャスト

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

この警告を修正するにはどうすればよいですか?

25
user159

簡単なハック修正は、longではなくintにキャストするだけです。多くのシステムでは、sizeof(long) == sizeof(void *)

より良いアイデアは、intptr_t

int threadnumber = (intptr_t) param;

そして

pthread_create(&(tid[i]), &attr, runner, (void *)(intptr_t)i);
45
tangrs
_pthread_create(&(tid[i]), &attr, runner, (void *) i);
_

isizeof(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));
_
1
Juan Ramirez

私も同じ警告を受けていました。そのため、警告を解決するためにintをlongに変換すると、この警告は消えました。 「異なるサイズの整数からポインターにキャストする」警告については、64xのポインターは64ビットであり、32xのポインターは32ビットであるため、ポインターは任意の変数の値を保持できるため、この警告を残すことができます。

0
newbie