The constructor failed because you were initializing with a reference in the
other constructor. I have changed that to a pointer, since the object has a
pointer internally anyway.
For the loop, it's not clear to me how the loop above the end condition
actually ensures that eventually there's nothing found.
Your team Widelands Developers is subscribed to branch
Mailing list: https://launchpad.net/~widelands-dev
Post to : firstname.lastname@example.org
Unsubscribe : https://launchpad.net/~widelands-dev
More help : https://help.launchpad.net/ListHelp