Skip to content

Commit

Permalink
Fix -XepDisableAllWarnings flag when passed on its own
Browse files Browse the repository at this point in the history
Without this change, the flag would not work when passed as the single Error Prone argument.  Not sure how to add a regression test for this, but the fix works locally.

Fixes #3462

COPYBARA_INTEGRATE_REVIEW=#3462 from msridhar:fix-disable-all-warnings 89f2c1d
PiperOrigin-RevId: 487205720
  • Loading branch information
msridhar authored and Error Prone Team committed Nov 9, 2022
1 parent 7d5048c commit cbaf3bb
Showing 1 changed file with 2 additions and 1 deletion.
Expand Up @@ -143,7 +143,8 @@ public ScannerSupplier applyOverrides(ErrorProneOptions errorProneOptions) {
&& errorProneOptions.getFlags().isEmpty()
&& !errorProneOptions.isEnableAllChecksAsWarnings()
&& !errorProneOptions.isDropErrorsToWarnings()
&& !errorProneOptions.isDisableAllChecks()) {
&& !errorProneOptions.isDisableAllChecks()
&& !errorProneOptions.isDisableAllWarnings()) {
return this;
}

Expand Down

0 comments on commit cbaf3bb

Please sign in to comment.