News:

Simutrans Wiki Manual
The official on-line manual for Simutrans. Read and contribute.

[patch] preserve position of public stop when join other stops

Started by Dwachs, February 24, 2013, 06:39:55 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Dwachs

Currently, if one makes an own stop public, and the stop gets joined to an existing public halt, the label of the public halt is moved to the position of the players stop. Is this intended?

Patch changes this: The label with the public stop's name stays at its position.
Parsley, sage, rosemary, and maggikraut.

Fabio

Sounds great.

Please consider also ctrl+ stop moving tool to move the label to a different station tile.

prissi

I am ignoring Fabios always greedy extension requests and would suggest that this is indeed a good idea.

However, wasn't there somewhere an assumption that the basis pos is number 0 in the tile array?

Fabio

Quote from: prissi on February 24, 2013, 10:38:03 PM
I am ignoring Fabios always greedy extension requests
Is it my fault if one idea leads to another? :D

Honestly, just brainstorming with thoughts not worthy an own extension request thread ;)

Ters

The entire code, also before the patch, seems kind of backwards compared to the name of the function and the description in the first post. I guess that's because there might not be any public stop to join with. Or maybe that whatever code calls make_public_and_join isn't prepared that the stop it called that method on is killed during the call?

Dwachs

Quote from: prissi on February 24, 2013, 10:38:03 PM
However, wasn't there somewhere an assumption that the basis pos is number 0 in the tile array?
That is, why the patch is more complicated than just changing one position variable ;) Tile 0 of the public halt is moved to position 0 of the new halt..

@Ters: Maybe method needs proper documentation / better name. Will see, what I can do.
Parsley, sage, rosemary, and maggikraut.