Thread: delete memory in a struct destructor

Where do you allocate the memory for theOptions that you're deleting in the destructor? In addHead? Can you show the definition for that? Also, can you explain why you decided to define this as a struct not a class?

Oops... didn't read the part about not wanting an answer...

Here's a hint. You are using C++, so use C++. If you consider it "lazy" to use standard C++ features, you are more likely to end up with problems like this that are difficult to debug and fix. If you choose to use standard C++ features instead of doing it the un-"lazy" way, your problem will be fixed.

If you want to fix your problem without using those language tools, look at what happens when getMenu returns a copy of a new menu (or read the answer that I originally posted).

>> I don't need an answer on how to fix it, just need a clue as to where to look

given that you have several raw pointers in your struct...you could start by redefining everything as stl containers (it's usually better to initially use 'known good' containers before optimizing with hand-coded versions).

Code:

struct menu
{
	struct option
	{
		string name;
		bool isMenu;
		string fileName;
	};
	
	string name;
	vector <option> theOptions;

	menu(void) { }
	~menu(void) { }
};

// Prototypes
const menu getMenu(const char* fileName);
void showMenu(const menu& theMenu, unsigned int currentSelection = 0);
void gameLoop(void);

int main(void)
{
	list<menu> menus;
	menus.push_back(getMenu("main.mnu"));

	gameLoop();

	return 0;
}