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

Show ignored options in java dumps #13579

Merged
merged 2 commits into from
Sep 29, 2021
Merged

Conversation

keithc-ca
Copy link
Contributor

The first commit cleans up:

  • remove the implicit dependency on order of flag bits in prepState and requestMask
  • add missing parentheses
  • explicitly compare values with NULL or 0
  • initialize local variables
  • use J9_ARE_ALL_BITS_SET, J9_ARE_ANY_BITS_SET and J9_ARE_NO_BITS_SET where appropriate

The second commit fixes #13471.

* remove implicit dependency on order of flag bits
  in prepState and requestMask
* add missing parentheses
* explicitly compare values with NULL or 0
* initialize local variables
* use J9_ARE_ALL_BITS_SET, J9_ARE_ANY_BITS_SET and J9_ARE_NO_BITS_SET
  where appropriate

Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
@pshipton
Copy link
Member

jenkins test sanity,extended win jdk8

2 similar comments
@pshipton
Copy link
Member

jenkins test sanity,extended win jdk8

@pshipton
Copy link
Member

jenkins test sanity,extended win jdk8

@pshipton
Copy link
Member

The testSoftMxNotDisclaimMemory_3 test failure is a known problem unrelated to this PR.

@pshipton
Copy link
Member

@keithc-ca pls update the documentation to mention this new feature.

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

Successfully merging this pull request may close these issues.

Enhance javacore output to indicate which options where ignored
2 participants