Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Non-Mammoth train handling is incorrect #4461

Closed
DorpsGek opened this issue Feb 3, 2011 · 7 comments
Closed

Non-Mammoth train handling is incorrect #4461

DorpsGek opened this issue Feb 3, 2011 · 7 comments
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

DorpsGek commented Feb 3, 2011

Eddi opened the ticket and wrote:

tested in r21949

when disabling mammoth trains, the game only allows to attach 8 wagons to an engine, instead of 9 [total length 10 halftiles]

also, when attaching dualheaded or articulated vehicles, the length of 10 halftiles may be exceeded. [not sure if this is actually a feature]

Reported version: trunk
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/4461
@DorpsGek
Copy link
Member Author

DorpsGek commented Feb 3, 2011

SmatZ wrote:

Commit by smatz :: r12131 trunk/src/train_cmd.cpp (2008-02-13 16:49:25 UTC)
-Fix (r3374): with mammoth trains disabled, maximum train length was limited to 9

is was broken, then fixed, then it appeared again sometimes between 0.7 and 1.0...

The other problem is present at least since 0.4.

Is there any point in keeping this setting?


This comment was imported from FlySpray: https://bugs.openttd.org/task/4461#comment9577

@DorpsGek
Copy link
Member Author

DorpsGek commented Feb 3, 2011

dihedral wrote:

i would only vote for a setting limiting the max length of a train in order to forbid exsessivly long trains on a multiplayer game - as actually having a 99 tile long train is something some admins would like to rather not see ;-)


This comment was imported from FlySpray: https://bugs.openttd.org/task/4461#comment9579

@DorpsGek
Copy link
Member Author

DorpsGek commented Feb 3, 2011

Eddi wrote:

I agree with that. The limit should be the length in halftiles, not the number of vehicles/vehicle parts.


This comment was imported from FlySpray: https://bugs.openttd.org/task/4461#comment9584

@DorpsGek
Copy link
Member Author

DorpsGek commented Feb 3, 2011

Yexo wrote:

In other words: remove the current setting as it's not really useful due to articulated vehicles being (much) longer than a half-tile. Than think about possible implementing a new setting that sets the maximum length of trains in half-tiles (or tiles? more intuitive for users).


This comment was imported from FlySpray: https://bugs.openttd.org/task/4461#comment9586

@DorpsGek
Copy link
Member Author

DorpsGek commented Feb 3, 2011

frosch wrote:

Just a historical note: Counting dual-headed vehicles as one vehicle was added to please the Bjarni's old autoreplace.

But yes, the current setting makes no sense :)


This comment was imported from FlySpray: https://bugs.openttd.org/task/4461#comment9587

@DorpsGek
Copy link
Member Author

DorpsGek commented Feb 3, 2011

Eddi wrote:

Yes, the whole thing should be converted to tiles. Including the number shown in the depot. i never understood why this wasn't done right from the start...


This comment was imported from FlySpray: https://bugs.openttd.org/task/4461#comment9588

@DorpsGek
Copy link
Member Author

DorpsGek commented Feb 4, 2011

Rubidium closed the ticket.

Reason for closing: Fixed

In r21960


This comment was imported from FlySpray: https://bugs.openttd.org/task/4461

@DorpsGek DorpsGek closed this as completed Feb 4, 2011
@DorpsGek DorpsGek added flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) Vehicles labels Apr 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)
Projects
None yet
Development

No branches or pull requests

1 participant