This sounds fair enough, but I would think that what should really be done is 
free `vf` *after* destroying the VTE widget, as the callbacks use the `vf` 
global.
However, it makes sense not to restart the child when destroying the widget, so 
the patch here seems also not bad.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2634#issuecomment-715886335

Reply via email to