← Back to team overview

widelands-dev team mailing list archive

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