New game memory leaks

[jwatson1993]

In my game, if the user presses escape, they will return to the main menu, where they can choose to make a new game. I'm not deleting the textures and meshes from memory because Irrlicht checks for them beforehand. What all do I have to do to make sure memory doens't leak, because for each iteration ~400k of memory(according to the task manager) is being created. Here's the code for the quitting:

cpp Code: Select all

 
        //clearPhysics();
                gameState = G_MAIN_MENU;
                smgr->clear();
               
                SKeyMap* keyMap = new SKeyMap[6];//changes the keymap to something usable
                        keyMap[5].Action = EKA_MOVE_FORWARD;
                 keyMap[5].KeyCode = KEY_UP;
                 keyMap[0].Action = EKA_MOVE_FORWARD;
                 keyMap[0].KeyCode = KEY_KEY_W;
                 keyMap[1].Action = EKA_MOVE_BACKWARD;
                 keyMap[1].KeyCode = KEY_KEY_S;
                 keyMap[2].Action = EKA_STRAFE_LEFT;
                 keyMap[2].KeyCode = KEY_KEY_A;
                 keyMap[3].Action = EKA_STRAFE_RIGHT;
                 keyMap[3].KeyCode = KEY_KEY_D;
                                 keyMap[4].Action = EKA_JUMP_UP;
                 keyMap[4].KeyCode = KEY_SPACE;
                camera = smgr->addCameraSceneNodeFPS(smgr->getRootSceneNode(), 50.0f,.1f, -1, keyMap, 6, false, 10.f);
                device->getCursorControl()->setVisible(true);
                guienv->getRootGUIElement()->setVisible(true);
 

If my variables are self-documenting ask me to explain what they are. Whenever this is called there isn't a drop in memory usage, either, so something isn't being cleared.

[hendu]

I don't see you deleting keyMap anywhere. Other than that, a valgrind log please (or your favorite memory leak detector tool).

[hybrid]

If you add cameras and other scene graph elements each time without removing them, you'll also see an increase in memory usage.

[jwatson1993]

Cameras are nodes, though, and shouldn't clear() delete their memory? Keymap shouldn't be an issue, should it?

[hybrid]

Oh yes, seem to have missed the early lines of the code fragment. That should be enough to remove all scene nodes, including cameras

[hendu]

Well if you're creating a new keymap each time, and not deleting it, that's an obvious leak?