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

Change: Heading for 1.11 now #7986

Merged
merged 1 commit into from Feb 8, 2020
Merged

Change: Heading for 1.11 now #7986

merged 1 commit into from Feb 8, 2020

Conversation

@glx22
Copy link
Contributor

glx22 commented Feb 8, 2020

No description provided.

bin/ai/compat_1.10.nut Outdated Show resolved Hide resolved
@glx22 glx22 force-pushed the glx22:1.11 branch from 79691bf to 7cdd152 Feb 8, 2020
@glx22 glx22 merged commit 2f264f2 into OpenTTD:master Feb 8, 2020
8 checks passed
8 checks passed
Commit checker
Details
OpenTTD CI Build #20200208.10 succeeded
Details
OpenTTD CI (Linux linux-amd64-clang-3.9) Linux linux-amd64-clang-3.9 succeeded
Details
OpenTTD CI (Linux linux-amd64-gcc-6) Linux linux-amd64-gcc-6 succeeded
Details
OpenTTD CI (Linux linux-i386-gcc-6) Linux linux-i386-gcc-6 succeeded
Details
OpenTTD CI (MacOS) MacOS succeeded
Details
OpenTTD CI (Windows Win32) Windows Win32 succeeded
Details
OpenTTD CI (Windows Win64) Windows Win64 succeeded
Details
@glx22 glx22 deleted the glx22:1.11 branch Feb 8, 2020
@James103
Copy link
Contributor

James103 commented Feb 10, 2020

Looking at the data format for _openttd_newgrf_version, it looks like only 4 bits are allocated to the OpenTTD minor version, which means that there's only 16 possible values that could be stored. Does that mean that, if the format of _openttd_newgrf_version is not changed in the meantime, OpenTTD 1.15 must be followed by OpenTTD 2.0?

@LordAro
Copy link
Member

LordAro commented Feb 10, 2020

Yup!

@LordAro
Copy link
Member

LordAro commented Feb 10, 2020

Alternatively, we just overflow into the major version, so that 2.0 == 1.16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.