Memory Leak when creating AddGameWizard about client HOT 8 OPEN

On further inspection, various widgets in AddGameWizard need parents as well.

from client.

Noted, will amend in the refactor inbound. Muchas gracias, senor.

from client.

Just curious about design decisions, why aren't managed pointers such as unique_ptr<T> and shared_ptr<T> being used? I don't see the reason behind using raw pointers in 2015 unless one is working with legacy code. In terms of performance, compilers can optimize unique_ptrs to the same machine code as raw pointers.

If it's just a matter of somebody doing it, let me know.

from client.

Because there's no need for ownership or sharing and the objects won't outlive the parent. Not relevant to the issue.

Sent from my Android device with K-9 Mail. Please excuse my brevity.

from client.

To elaborate on the response by @elken above, Qt by nature favors heap-allocation of local QObjects for a number of reasons, opting to manage memory hierarchically throughout the application when respective QObjects are parented to one another. In this issue, some parents were never set, hence the memory leakage.

The sheer amount of QObjects we're initializing programmatically now and in the future favors raw pointers as a form of brevity, though yes, in any other case we would be doing otherwise.

from client.

To further elaborate, we also want raw pointers as arguments and as local variables for the speed and leanness of just an address, we don't need the overhead *_ptr types provide.

from client.

Issue seems to be resolved, reopen if not.

from client.

The memory leak still occurs on both wizards: http://i.imgur.com/Jd5Ztm0.gifv

from client.