Error: initialization makes pointer from integer without a cast

Question

I have had trouble with this piece of code returning the error:

assgTest2.c: In function 'Integrate':
assgTest2.c:12: warning: initialization makes pointer from integer without a cast
assgTest2.c:15: error: expected ';' before ')' token

I've had a look around and haven't been able to make sense of the answer to similar questions, any help would be appreciated.

1    void SamplePoint(double *point, double *lo, double *hi, int dim)
2    {
3       int i = 0;
4       for (i = 0; i < dim; i++)
5          point[i] = lo[i] + rand() * (hi[i] - lo[i]);
6    }
7
8    double Integrate(double (*f)(double *, int), double *lo, double *hi, int dim, 
9                     double N)
10    {
11       double * point = alloc_vector(dim);
12       double sum = 0.0, sumsq = 0.0;
13
14       int i = 0;
15       for (i = 0.0, i < N; i++)
16       {
17         SamplePoint(point, lo, hi, dim);
18
19         double fx = f(point, dim);
20         sum += fx;
21         sumsq += fx * fx;
22       }
23
24       double volume = 1.0;
25       i = 0;
26       for (i = 0; i < dim; i++)
27         volume *= (hi[i] - lo[i]);
28
29       free_vector(point, dim);
30       return volume * sum / N;
31    }

Edit: Fixed some mistakes, still giving the same error

Fix for (i = 0.0, i < N; i++) to for (i = 0.0; i < N; i++) — Balu
– Balu, Commented Mar 15, 2014 at 10:44
possible duplicate of Assignment makes pointer from integer without cast — jww
– jww, Commented Sep 14, 2014 at 17:58

pmg · Accepted Answer · 2014-03-15 10:54:19Z

9

I guess this is your line 12

    double * point = alloc_vector(dim);

The text of the warning is

warning: initialization makes pointer from integer without a cast

What this means is that the integer returned from alloc_vector() is being automatically converted to a pointer and you shouldn't do that (also you should not cast despite what the warning hints at).

Correction: add the proper #include where alloc_vector() is declared so that the compiler knows it returns a pointer and doesn't need to guess (incorrectly) it returns an integer.

Or, if you don't have the include file, add the prototype yourself at the top of your file

double *alloc_vector(int); // just guessing

line 15

     for (i = 0.0, i < N; i++)

The text for the error is

assgTest2.c:15: error: expected ';' before ')' token

Each for statement has two semicolons in the control structure (between parenthesis). Your control structure only has 1 semicolon. Change that to

     for (i = 0.0; i < N; i++)
     //          ^ <-- semicolon

edited Mar 15, 2014 at 10:54

answered Mar 15, 2014 at 10:30

pmg

109k14 gold badges132 silver badges203 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

user3422805 Over a year ago

Thanks heaps, that fixed the first error, however the "assgTest2.c:15: error: expected ';' before ')' token" error persists.

pmg Over a year ago

@user3422805: I've edited my post with the 2nd error.

Collectives™ on Stack Overflow

Error: initialization makes pointer from integer without a cast

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related