widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #02477
Re: [Merge] lp:~widelands-dev/widelands/bug-1203629 into lp:widelands
> The problem in building.cc was the constness of the string.
Actually, it wasn't but I solved it anyway :)
We have a problem with a crash in pathfield-cc, so I reverted the new code. I had a go at it, but can't get it to work. Here's one of my attempts:
for (boost::shared_ptr<Pathfields> pathfield : m_list) {
if (pathfield.use_count() == 1) {
++(pathfield.get())->cycle;
if (!(pathfield.get())->cycle) {
clear(pathfield);
}
return pathfield;
}
}
And the crash is:
Map_Saver::save() took 73ms
Game: Writing Map Data took 75ms
Game: Writing Player Economies Info ... took 0ms
Game: Writing Command Queue Data ... took 972ms
Game: Writing Interactive Player Data ... took 0ms
Game_Saver::save() took 1087ms
SaveHandler::save_game() took 1119ms
terminate called after throwing an instance of '_wexception'
what(): [/home/bratzbert/sources/widelands/bug-1203629_2/src/logic/pathfield.cc:99] PathfieldManager::allocate: unbounded nesting?
Aborted (core dumped)
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1203629/+merge/228203
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1203629 into lp:widelands.
References