Bug #8

Code issues in HandleSharingChange

Added by Hirundo almost 11 years ago. Updated almost 11 years ago.

Status:ClosedStart date:2009-04-29
Priority:NormalDue date:
Assignee:Hirundo% Done:

100%

Category:-
Target version:IS2.0-beta3

Description

These are a couple of relatively minor issues to look at, in order to
perfect the sharing change code:

  • Correctly set reservations for tunnels and bridges when removing a
    train (they can have more than one train on them!)
  • Update level crossings when removing trains, but not when another
    nearby train should keep them closed.
  • Avoid doing unnecessary work, or doing the same work twice, to save CPU time.
  • The behaviour of crashed trains, also w.r.t. the first two points

Apart from this, a lot of testing is needed. Some debugging code to see the results of these tests may be helpful.

Associated revisions

Revision 1563:625908e22169
Added by dominik about 15 years ago

(svn r1586) Fix: [ 1107340 ] Graphs don't show comapany info for company #8

Revision 5419:25bd8121f09c
Added by truelight over 13 years ago

(svn r5487) -Fix: [#8] UDP sockets were used even if network-availability was set to false (tokay)

Revision 5420:1e4a1a5d72b9
Added by truelight over 13 years ago

(svn r5488) -Backport (r5487) -Fix: [#8] UDP sockets were used even if network-availability was set to false (tokai)

Revision 14509:9529e386f67c
Added by SmatZ about 10 years ago

(svn r18636) -Codechange: make TextEffect::duration a value in ticks instead of ticks * 8

History

#1 Updated by Hirundo almost 11 years ago

point 3, avoiding unnecessary work, is fixed in the recent commits.

#2 Updated by Hirundo almost 11 years ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

These issues should be fixed in r12125

Also available in: Atom PDF