The International Simutrans Forum

Is that patch match for a bus route like a-b-c-d-c-b-a ?
And if destination doesn't contain on route anymore, where they to go ?

If the old route is a-b-c-d-e-a-b-c then the first 'a' etc will be chosen. But only if the exact coordinate matches.

Many of the cases, for example you replace one stop by stop moving tool, order of other stop don't change. So, you must check first, position of next stop is the same or not. In this case, you don't need to change next stop.

And also you need to search best match stop.

In this case, schedule of each convoi is still version 0, so difference of version 1 to version 2 is meaningless.

Please freely test it, but it may not read your mind.

goto is evil -  but it may help at mind-reading

But it seems you don't like this approach, I don't work for this anymore.

So I think you did the difficult part ... I will have a look at it.

         if(aktuell<new_fpl->get_count()  &&  current==new_fpl->eintrag[aktuell].pos  ) {
            // next pos is the same => keep the convoi state
            is_same = true;
         }

            int quality =
               (new_fpl->eintrag[ i ].pos==current)*8 +
               (new_fpl->eintrag[(i+1)%new_count].pos==next)*4 +
               (new_fpl->eintrag[(i+2)%new_count].pos==nextnext)*2 +
               (new_fpl->eintrag[(i+3)%new_count].pos==nextnextnext);

                  aktuell %= new_count;
                  how_good_matching = quality;
               }
            }

            is_same = new_fpl->eintrag[aktuell].pos==current;

            if(how_good_matching==0) {
               // nothing matches => take the one from the line
               aktuell = new_fpl->get_aktuell();
            }

                  aktuell %= new_count;
                  how_good_matching = quality;
               }
               // if we go to same, then we do not need route recalculation ...
               is_same = new_fpl->eintrag[aktuell].pos==current;
            }

            if(how_good_matching==0) {
               // nothing matches => take the one from the line
               aktuell = new_fpl->get_aktuell();
            }
            // if we go to same, then we do not need route recalculation ...
            is_same = new_fpl->eintrag[aktuell].pos==current;

         }