c++ float array as default parameter

Question

I'm quite new to c++ and I don't manage to make this works. Sorry but i have always worked with languages that didn't helped me to think in terms of memory pointers ans so maybe this is a fooly question.

I want to pass a float array as a default parameter. Like this:

void getHistogram(const Mat& src, MatND& hist, float range[]=NULL) {

    if(range==NULL) {
        double maxPixel=0;
        minMaxLoc(src, 0, &maxPixel, 0, 0);
        range = { 0, maxPixel +1 };
    }

    // now calculate histogram with the right range
    // something something
}

I have tried with some different syntax but i'm always in front on some errors like

histogram.cpp:21: warning: extended initializer lists only available with -std=c++0x or -std=gnu++0x
histogram.cpp:21: error: cannot convert ‘<brace-enclosed initializer list>’ to ‘float*’ in assignment

EDIT (but with memory leak):

Ok, thatnks to this answer i have resolved in this way:

void imHist(const Mat& src, MatND& hist, float range[]=NULL) {

    if(range==NULL) {
        double maxPixel=0;
        minMaxLoc(src, 0, &maxPixel, 0, 0);

        range = new float[2];
        range[0] = 0;
        range[1] = maxPixel +1;
    }

}

some pros or cons?

EDIT 2

see the accepted answer

Answer 1

Replace your current code …

void getHistogram(const Mat& src, MatND& hist, float range[]=NULL) {

    if(range==NULL) {
        double maxPixel=0;
        minMaxLoc(src, 0, &maxPixel, 0, 0);
        range = { 0, maxPixel +1 };
    }

    // now calculate histogram with the right range
    // something something
}

with this:

void getHistogram(const Mat& src, MatND& hist, float range[] ) {
    assert( range != 0 );
    // now calculate histogram with the right range
    // something something
}

void getHistogram(const Mat& src, MatND& hist ) {
    double maxPixel=0;
    minMaxLoc(src, 0, &maxPixel, 0, 0);
    float range[] = { 0, maxPixel +1 };
    getHistogram( src, hist, range );
}

That said, why are you using float instead of double?

---

EDIT: the OP explains that the array of float is required by OpenCV.

He further explains in his answer that he's resolved the problem as follows:

void imHist(const Mat& src, MatND& hist, float range[]=NULL) {

    if(range==NULL) {
        double maxPixel=0;
        minMaxLoc(src, 0, &maxPixel, 0, 0);

        range = new float[2];
        range[0] = 0;
        range[1] = maxPixel +1;
    }
}

This leaks memory, and is needlessly inefficient.

EDIT 2: the reason that the above code leaks memory is because there is a new (which allocates memory), but no corresponding delete expression (which frees the memory), and there is no indication of whether the memory was allocated by new or provided by the caller.

The reason that it's inefficient is that a dynamic memory allocation typically is orders of magnitude slower than e.g. a basic assignment or stack allocation, because it has to do a search for a suitable small free chunk of memory.

The stack allocation (C++ “automatic memory”) avoids that inefficiency by always deallocating in reverse order of allocation, so that it can always use the start of the free stack memory area for the next allocation (modulo direction: in practice the stack grows downward in memory on all machines I have known about).

Answer 2

When you pass an array, you pass a pointer to the first element, not all the array. For give you a responce, could you edit your post and add the code when you call the function?