News:

Do you need help?
Simutrans Wiki Manual can help you to play and extend Simutrans. In 9 languages.

Changed so that multiple halt and convoy dialogs are saved

Started by Ranran(retired), September 24, 2022, 08:59:59 AM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Ranran(retired)

Many players are inconvenienced by the fact that they have multiple convoy info open but only 1 is restored when they reboot simutrans from an auto-save. (´・ω・`)
This seems like a bug. I guess that the implementation of some code was forgotten for a long time.
Because there is a large gap between magic_convoi_info/magic_halt_info and the next magic number, it seems that save multiple dialogs is intended.
    // magic numbers with big jumps between them
    magic_convoi_info,
    magic_UNUSED_convoi_detail = magic_convoi_info          + 0x10000, // unused range
    magic_halt_info            = magic_UNUSED_convoi_detail + 0x10000,
    magic_UNUSED_halt_detail  = magic_halt_info            + 0x10000, // unused range
I made a patch to change this and tested it with extended, but it doesn't seem to cause any problems. It doesn't look like the save size will be large.
May I ask if there was any intention?

I think the patch I attached does that in standard.
ひめしという日本人が開発者達の助言を無視して自分好みの機能をextendedに"強引に"実装し、
コードをぐちゃぐちゃにしてメンテナンスを困難にし(とりわけ道路と建物関連)、
挙句にバグを大量に埋め込み、それを知らんぷりして放置し(隠居するなどと言って)別のところに逃げ隠れて自分のフォーク(OTRP)は開発を続けている
その事実と彼の無責任さに日本人プレイヤーは目を向けるべき。らんらんはそれでやる気をなくした(´・ω・`)
他人の振り見て我が振り直せ。ひめしのようにならないために、らんらんが生み出したバグや問題は自分で修正しなくちゃね(´・ω・`)

prissi

I think indeed all convoi dialogs should survive a svae/load circle, when a new client joins.

Incorporated in r 10759, thanks.