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

Fix translation buffers not terminating at last cell #1640

Merged
merged 1 commit into from
Nov 16, 2021
Merged

Fix translation buffers not terminating at last cell #1640

merged 1 commit into from
Nov 16, 2021

Conversation

Adrianilloo
Copy link
Contributor

@Adrianilloo Adrianilloo commented Nov 15, 2021

Fixes #1594 per @dvander instructions and self tests

@asherkin
Copy link
Member

Please could you rebase and retarget this PR against the master branch.

@Adrianilloo Adrianilloo changed the base branch from 1.10-dev to master November 15, 2021 22:44
@Adrianilloo Adrianilloo deleted the 1.10-dev branch November 15, 2021 22:50
@Adrianilloo Adrianilloo restored the 1.10-dev branch November 15, 2021 22:51
@Adrianilloo Adrianilloo reopened this Nov 15, 2021
@Adrianilloo
Copy link
Contributor Author

Adrianilloo commented Nov 15, 2021

Ok, followed your steps now. Failed at trying to switch the source branch as well, but that isn't possible.

Copy link
Member

@asherkin asherkin left a comment

Choose a reason for hiding this comment

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

Thanks - LGTM

@asherkin asherkin merged commit 18f9d65 into alliedmodders:master Nov 16, 2021
@Adrianilloo
Copy link
Contributor Author

Adrianilloo commented Nov 16, 2021

Thanks! Will this be made into 1.10 as well?

@asherkin
Copy link
Member

Will this be made into 1.10 as well?

We're not currently backporting non-critical things to 1.10 as we're in the final push for 1.11 - please do upgrade to 1.11 and test it out!

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

Successfully merging this pull request may close these issues.

Translated buffers are early null terminated
2 participants