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

Remove the space at the end of J9NLS_VM_STACK_TRACE_EXCEPTION_IN, update all NLS translations #18446

Merged
merged 2 commits into from
Nov 13, 2023

Conversation

pshipton
Copy link
Member

@pshipton pshipton commented Nov 13, 2023

Avoid having a message with a space at the end to keep the line endings check happy. Explicitly print the space in the code.

Also update all the NLS translations.

@pshipton
Copy link
Member Author

I think I could just modify the existing message in place. Thoughts?

@pshipton
Copy link
Member Author

I've pushed the change. If you agree I will squash the two commits.

runtime/vm/exceptiondescribe.c Outdated Show resolved Hide resolved
runtime/nls/j9vm/j9vm.nls Outdated Show resolved Hide resolved
runtime/nls/j9vm/j9vm.nls Show resolved Hide resolved
@pshipton
Copy link
Member Author

Updated it.

@pshipton pshipton changed the title Remove the space at the end of J9NLS_VM_STACK_TRACE_EXCEPTION_IN Remove the space at the end of J9NLS_VM_STACK_TRACE_EXCEPTION_IN, update all NLS translations Nov 13, 2023
Avoid having a message with a space at the end to keep the line endings
check happy. Explicitly print the space in the code.

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
@keithc-ca
Copy link
Contributor

Jenkins compile alinux jdk17

@keithc-ca keithc-ca merged commit f9dffe6 into eclipse-openj9:master Nov 13, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants