8257872: UL: -Xlog does not check number of options #1685
There was a problem hiding this comment.
Should this be a warning or an error? The behavior seems inconsistent:
In this PR I chose a warning level to ignore the invalid option and run a application. It isn't serious thing and some people might run with invalid option already.
The fourth option is for output-option and can be described relatively free. So I suspect that it is ignored even when it is invalid.
@jyukutyo This change now passes all automated pre-integration checks.
ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details.
After integration, the commit message for the final commit will be:
At the time when this comment was updated there had been 89 new commits pushed to the
As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details.
As you do not have Committer status in this project an existing Committer must agree to sponsor your change. Possible candidates are the reviewers of this PR (@dholmes-ora, @iklam, @YaSuenag) but any other Committer may sponsor as well.
➡️ To flag this PR as ready for integration with the above commit message, type
Your commit was automatically rebased without conflicts.
Pushed as commit 34650f5.
💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.