Почему я получаю эту ошибку?void * не является указателем на тип объекта. - PullRequest
Новая
       4

Почему я получаю эту ошибку?void * не является указателем на тип объекта.

5 голосов
/ 31 октября 2011 
void *stackAddr[NUM_THREADS];

stackAddr[i] = malloc(STACKSIZE);

Компилятор (g ++ 4.4.3) жалуется, где вызывается malloc ... 

warning: pointer of type ‘void *’ used in arithmetic
error: ‘void*’ is not a pointer-to-object type

Если вы заинтересованы в просмотре всего кода, вот оно ... 

#include <pthread.h>
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#define NUM_THREADS 4

void *stackAddr[NUM_THREADS];
pthread_t thread[NUM_THREADS];
pthread_attr_t attr;

void *BusyWork(void *t)
{
   int i;
   long tid;
   double result=0.0;
   tid = (long)t;

   printf("Thread %ld starting...\n",tid);
   for ( i = 0; i < 1000; i++)
   {
      result = result + sin(i*tid) * tan(i*tid);
   }
   printf("Thread %ld done. Result = %e\n", tid, result);
   pthread_exit((void*) t);
}

void pthread_create_with_stack( pthread_t * pthread, void *(*start_routine) (void *), int tid )
{
    const size_t STACKSIZE = 0xC00000; //12582912
    void *stackAddr;
    int rc;
    size_t i;
    pthread_t thread;
    pid_t pid;

    stackAddr[tid] = malloc(STACKSIZE); // Error here!
    pthread_attr_setstack(&attr, stackAddr[tid], STACKSIZE);

    rc = pthread_create( pthread, &attr, start_routine, (void*)tid );
}

int main (int argc, char *argv[])
{
   int rc;
   long t;
   void *status;

   /* Initialize and set thread detached attribute */
   pthread_attr_init(&attr);
   pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_JOINABLE);

   for(t=0; t<NUM_THREADS; t++) 
   {
      printf("Main: creating thread %ld\n", t);
      rc = pthread_create_with_stack(&thread[t], BusyWork, t); 
      if (rc) 
      {
         printf("ERROR; return code from pthread_create() is %d\n", rc);
         exit(-1);
      }
   }

   /* Free attribute and wait for the other threads */
   pthread_attr_destroy(&attr);
   for(t=0; t<NUM_THREADS; t++) 
   {
      rc = pthread_join(thread[t], &status);
      if (rc) 
      {
         printf("ERROR; return code from pthread_join() is %d\n", rc);
         exit(-1);
      }
      printf("Main: completed join with thread %ld having a status"   
            "of %ld\n",t,(long)status);
    }

    printf("Main: program completed. Exiting.\n");
    pthread_exit(NULL);
}

Ответы [ 7 ]

12 голосов
/ 31 октября 2011

Вы объявляете локальную переменную void *stackAddr, которая затеняет глобальный массив stackAddr.

Более того, это не массив, и применение оператора индекса [] пытается сместить и разыменовать указатель void, следовательно, ошибка компиляции.

Разыменование и арифметика указателей на пустые указатели не разрешены стандартом, поскольку sizeof(void) не определено.

4 голосов
/ 31 октября 2011

Вы испортили свою декларацию: 

void *stackAddr;

должно быть: 

void *stackAddr[];

(Вам также может понадобиться установить размер для массива.)

Тогда вы попытаетесь сделать это: 

stackAddr[tid] = malloc(STACKSIZE);

Итак, вы получаете доступ к элементу массива void*.

2 голосов
/ 31 октября 2011

В pthread_create_with_stack у вас есть этот код: 

void *stackAddr;
...
stackAddr[tid] = malloc(STACKSIZE)

Выражение stackAddr[tid] пытается сделать арифметику для void*.

2 голосов
/ 31 октября 2011

То, что вы опубликовали первым, это не проблема. 

void *stackAddr[NUM_THREADS];

Но в коде у вас есть что-то еще: 

void *stackAddr;

, поэтому компилятор пытаетсяиспользовать эту локальную переменную, и, конечно, не удается скомпилировать ..

2 голосов
/ 31 октября 2011

Теперь вы разместили реальный код, я предполагаю, что ошибка в pthread_create_with_stack, где у вас есть локальная переменная void * stackAddr, которая скрывает глобальный массив.

Похоже, вы должны просто удалить локальную переменную.

0 голосов
/ 31 октября 2011

В C ++ указатели void не имеют атрибутов, включая размер. В результате вы не можете рассчитать адрес смещения для пустот i-номера от начала массива. Именно спецификация определенного индекса в массиве вызывает у вас математические проблемы.

0 голосов
/ 31 октября 2011

В опубликованной вами версии stackAddr не является массивом, но вы присваиваете stackAddr[tid]. Компилятор заменяет это на *(stackAddr + tid) выражением в скобках, которое оценивается как stackAddr + tid * sizeof(void), поэтому компилятор предупреждает вас о sizeof(void).

...