delete videocapture

edit

I did try x->release(); before delete x and it still crashes when deleting x.

Did you tried to call the destructor? ~VideoCapture();?

@Immi: delete x is effectively calling the destructor, no need to call it explicitly!

@Guanta (hope it get linked): I've played a little bit with this in OpenCV 2.4.4: When I'm trying it like described in the Question -> It works. I get a crash with: int main (void) { cout << "Creating" << endl; VideoCapture cap(0); cout << "Created" << endl; delete &cap; cout << "Deleted" << endl;

waitKey();
return 0;

}

jep, please see my comment above

@Guanta May you tell my, why it crashes with Delete and not with calling destructor directly?

Honestly, I don't know why it crashes exactly without spending too much time on the source code. delete makes two steps: 1. it calls the destructor of the object 2. the object deleted is removed from memory. And you should always call delete if you create your object via new - otherwise this is not necessary. So, some weired configuration of the classes make it fail using delete on a non-initialized VideoCapture object.

This line of code results in the compiler error:

error C2440: 'initializing' : cannot convert from 'cv::VideoCapture *' to 'cv::VideoCapture'

NOTE: initializing on the stack does work well for us. This question pertains to initializing on the heap.

Stack:

function foo()

{

VideoCapture x;

x.open(movieFile);

}

I edited my code. It should be x('location of file);

Without digging too much into the code: The problem is that the default constructor of VideoCapture doesn't initialize the internal CVCapture-struct properly and if you want to delete it now, it tries to delete this uninitialized struct and crashes somehow <- this is a clear OpenCV-bug and needs to be fixed. Maybe you want to write a bug-report here.

You can circumvent it either by calling the other constructor as Steven suggested, or if you call the open() - method after creating it to initialize the internal struct. It shouldn't matter if you create it on the heap or not, also the deletion of your VideoCapture-pointer should work than fine - in contrast to Steven's answer: you should also do it (i.e. call delete if you are done) when you created VideoCapture via new.

Thank you all for the quick responses.

I tried passing the movie file in to the constructor with no luck:

VideoCapture* x = new VideoCapture("test.avi");

delete x;

If I run as is, it crashes. If I comment delete x; then the movie loads fine and all is well, but the memory doesn't get freed properly.

Could you switch to the recently released 2.4.5 version and see if it still keeps giving the same error?

Yup, I'll give that a shot, thanks!

Hmm, this destroys my whole theory :D . Please report if you have more luck using a newer version!