← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/bug-1644553-buildhelp-button into lp:widelands

 

I looked into this because I was confused why this crashed. I found that the problem was that the registry was used by two windows at the same time: the old one, because die() would only kill it in the next frame and the new one, because FieldActionWindow() would use the same registry.

The solution I came up with is to distinguish if a new window was created - in which case, the new unique_window will take care of killing the old one - or if the current window should just be close. I the second case, using destroy() is correct. ptal and test if my reasoning is correct.
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1644553-buildhelp-button/+merge/312429
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1644553-buildhelp-button into lp:widelands.


References