tree ff98263babe999d00287d4b5c360bdc488eb53e7
parent c3b093bc58b32cfd6e0444adde5f706cda7e956f
author Zesstra <zesstra@zesstra.de> 1617971186 +0200
committer Zesstra <zesstra@zesstra.de> 1620381898 +0200

Revert "MNPC: Walk() in zwei Funktionen gesplittet"

This reverts commit 8ecb43e259b9e0cfe4e960fb21f065081717e1fb.

Meines Erachtens ist die Aenderung nicht ausgereift:

1) Sie macht es zwar einfacher, das Walk() zu ueberschreiben,
aber immer noch noetig fuer das Ersetzen der Zielermittlung
und damit unnoetig komplex. Will man nur das Ziel aendern,
ist das Ueberladen des Walk() immer noch eine unnoetig
aufwendige Massnahme.

2) Die Umbenennung der Schedulerfunktion erfordert in der
Praxis einen Reboot - unnoetigerweise.

3) Die aktuelle Implementation erfordert die Pruefung
aller ueberladenen Walk() in erbenden Objekten. Denn wenn
jemand das Walk mit einer Kopie ueberladen hat, kommt es
bei jeder Callout-Multiplikation (in jeder Bewegung
werden 2 Callouts gestartet).

4) Die Aenderung enthaelt einen Bug bzgl. der Ausgabe der
Bewegungsmeldung vom Exit - der war zwar aelter, aber wurde
auch nicht korrigiert. Die Meldung beim Nutzen eines Ausganges
ist in P_EXIT nicht mehr mit # getrennt am Zielpfad.

Change-Id: Ie0b59f8a5f09806cc7a17d80d2740371951e451a