Code review comment for lp:~ghugesss/xpad/xpad

Revision history for this message
Sagar Ghuge (ghugesss) wrote :

Hi,

If I am not wrong then pads are actually our toplevel widgets and we are destroying it. so if we destroy the toplevel widget according to documentation those will gets finalized automatically so I don't think so that we need to set pads to NULL. That's why I removed the code. And if I introduce that line again then it will give me the seg fault. ( as it is already finalized and still we were trying to set it to NULL ) that's why may be.

« Back to merge proposal