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

Add: Houses can accept up to 16 different cargo types via NewGRF #6872

Merged
merged 1 commit into from Jan 21, 2019

Conversation

nielsmh
Copy link
Contributor

@nielsmh nielsmh commented Jul 30, 2018

New Action0 property 23 for feature 07, variable length, format B n*(B B). Initial byte is number of structures following. First byte in structure is cargo id, second is acceptance level in 1/8 units.

This doesn't affect savegame data, and shouldn't affect backwards compatibility with NewGRFs. Any NewGRF using the new property would not load on older versions without the property, but chance is they also depend on more than 32 cargo types then.

@nielsmh nielsmh requested a review from frosch123 July 30, 2018 15:57
@nielsmh nielsmh added component: NewGRF This issue is related to NewGRFs needs review labels Aug 5, 2018
src/house.h Show resolved Hide resolved
src/newgrf.cpp Show resolved Hide resolved
src/table/town_land.h Show resolved Hide resolved
src/house.h Outdated Show resolved Hide resolved
src/newgrf.cpp Outdated Show resolved Hide resolved
src/newgrf.cpp Show resolved Hide resolved
@TrueBrain
Copy link
Member

We recently switched from Jenkins as CI to Azure Pipelines as CI. This means you need to rebase before this Pull Request will pass its checks. Sorry for the troubles!

Copy link
Member

@LordAro LordAro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code reads fine for me. I agree with @nielsmh wrt uint/CargoID - the numbers are used as array sizes/indices, CargoID doesn't seem appropriate

src/house.h Show resolved Hide resolved
src/house.h Show resolved Hide resolved
src/table/town_land.h Show resolved Hide resolved
src/house.h Outdated Show resolved Hide resolved
New Action0 property 23 for feature 07, variable length, format B n*(B B). Initial byte is number of structures following. First byte in structure is cargo id, second is acceptance level in 1/8 units.
@nielsmh nielsmh merged commit 48b334c into OpenTTD:master Jan 21, 2019
@nielsmh nielsmh deleted the housemoreaccepts branch March 8, 2019 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: NewGRF This issue is related to NewGRFs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants