-
Notifications
You must be signed in to change notification settings - Fork 396
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 string literal conversion warnings in env #7186
Conversation
d62c488
to
82ff59a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Please squash the two commits.
a4b6106
to
1377be5
Compare
Jenkins build all |
Build failures due to a link error:
I'm not sure why it would be a link error instead of a regular compiler error, but maybe it's because I updated the parameter types of |
Sorry I missed that! I guess you'll need to update any downstream implementation in OpenJ9 pull request eclipse-openj9/openj9#18467 as well. |
1377be5
to
e019b8f
Compare
Fix string literal conversion warnings in compiler/env Signed-off-by: Dylan Tuttle <jdylantuttle@gmail.com>
e019b8f
to
d9016be
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thanks!
Jenkins build all |
There's an odd failure in eclipse.omr (x86-64 Linux):
I don't think it's related to this pull request, and I found at least one previous reference in pull request 6907 to that failure occurring. Re-running. |
The x86-64 macOS failures appear to be due to issue #6516:
|
Performing a coordinated merge with eclipse-openj9/openj9#18467 |
Work towards fixing AIX warnings about assigning string literals to non-const char pointers by adding 'const' qualifiers to some string variables and parameters (or worst case scenario, casting to (char *)) in env.
This PR contributes to (but does not close) eclipse-openj9/openj9#14859
This PR depends on:
and must be merged in coordination with eclipse-openj9/openj9#18467