Error in nobHarborBuilding::Destroy

Bug #1022396 reported by Åsmund T
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Return To The Roots
Status tracked in S25client
S25client
Fix Released
Medium
FloSoft

Bug Description

There seems to be an error in nobHarborBuilding::Destroy() that might cause game crashes. In line 155 figures_for_ships.clear() should clearly be soldiers_for_ships.clear() (figures were cleared a couple of lines above).

The error is present in the current version (rev7921)

Tags: rev7921
Revision history for this message
Spike (fvv1) wrote :

Well, I got no Idea how the seafaring was coded :s

tags: added: rev7921
Revision history for this message
NastX (jtrampe) wrote : Re: [Bug 1022396] Re: Error in nobHarborBuilding::Destroy

He already proposed a solution^^
Now somebody should fix it^^

2012/7/13 Spike <email address hidden>:
> Well, I got no Idea how the seafaring was coded :s
>
> ** Tags added: rev7921
>
> --
> You received this bug notification because you are subscribed to Return
> To The Roots.
> https://bugs.launchpad.net/bugs/1022396
>
> Title:
> Error in nobHarborBuilding::Destroy
>
> Status in Return To The Roots:
> New
>
> Bug description:
> There seems to be an error in nobHarborBuilding::Destroy() that might
> cause game crashes. In line 155 figures_for_ships.clear() should
> clearly be soldiers_for_ships.clear() (figures were cleared a couple
> of lines above).
>
> The error is present in the current version (rev7921)
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/s25rttr/+bug/1022396/+subscriptions

Revision history for this message
FloSoft (flosoft) wrote :

commited patch in rev8040

Changed in s25rttr:
importance: Undecided → Medium
status: New → Fix Released
assignee: nobody → FloSoft (flosoft)
milestone: none → nightly
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.