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

Build roads tool: build road up to objects instead of discarding entire build #3318

Closed
DorpsGek opened this issue Nov 19, 2009 · 5 comments
Closed
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

Tve4 opened the ticket and wrote:

When building a road (either with using "build section" or "use autoroad tool") and the segment of the road to be built meets and obstruction (tunnel, industry, etc.) simply build the road from start point up to the obstruction (instead of refusing to build anything). Railroad build tool always builds rails up to the obstruction.

Reported version: 0.7.3
Operating system: Windows


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

DorpsGek commented Dec 5, 2009

Terkhen wrote:

I have coded a patch that implements this feature. I think it is a bit hackish, so any ideas on how to implement it more correctly will be welcome.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/3318#comment7024

@DorpsGek
Copy link
Member Author

Terkhen wrote:

New version with a cleaner code.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/3318#comment7067

@DorpsGek
Copy link
Member Author

DorpsGek commented Jan 9, 2010

Terkhen wrote:

New version which uses DiagDir instead of a boolean value to check road direction (made by Hirundo) and adds some comments and code cleanup.

Edit: Download the second version (CORRECTED): A comment needed to be rewritten.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/3318#comment7315

@DorpsGek
Copy link
Member Author

Terkhen wrote:

Update to current trunk, after some parts of the code where commited to trunk. I also cleaned up a hackish if / else.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/3318#comment7353

@DorpsGek
Copy link
Member Author

Rubidium closed the ticket.

Reason for closing: Implemented

In r18803


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

@DorpsGek DorpsGek added flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) Build system 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