Question

I fear that I am running into memory leak issues by doing the following:

(Sample code)

class myItem //random container stuff mostly. All primatives.
{
    int index;
    char* name;
    int val1;
    int val2;
};

class vecList
{

    vector< myitem* > *myVec;

    void delete()
    { 
        MyVec->erase(std::remove_if(myVec->begin(), MyVec->end(), IsMarkedToDelete), MyVec->end()); //leak here?
    }
};

Erase doesn’t free the memory if it’s a pointer, right? If I wasn’t using remove_if, I could call delete on the pointer before destroying it. How would I do it in this case? Smart Pointers? I’d prefer not to re-implement everything with them and I don’t really want to add the boost library.

Thanks!

Answer 1

You could just delete the item in your IsMarkedToDelete function when it returns true.