Commit 3ab344c2 by olly Committed by Andrew Lewis

Fix 1 ErrorProneStyle finding:

* The Google Java Style Guide requires that each switch statement includes a default statement group, even if it contains no code. (This requirement is lifted for any switch statement that covers all values of an enum.)
  (see http://go/bugpattern/MissingDefault)

This CL looks good? Just LGTM and Approve it!
This CL doesn’t look good? This is what you can do:
* Suggest a fix on the CL (go/how-to-suggest-fix).
* Revert this CL, by replying "REVERT: <provide reason>"
* File a bug under go/error-prone-bug for category ErrorProneStyle if the change looks generally problematic.
* Revert this CL and not get a CL that cleans up these paths in the future by
replying "BLOCKLIST: <provide reason>". This is not reversible! We recommend to
opt out the respective paths in your CL Robot configuration instead:
go/clrobot-opt-out.

This CL was generated by CL Robot - a tool that cleans up code findings
(go/clrobot). The affected code paths have been enabled for CL Robot in //depot/google3/java/com/google/android/libraries/exoplayer/METADATA which is reachable following include_presubmits from //depot/google3/third_party/java_src/android_libs/exoplayer/METADATA.
Anything wrong with the signup? File a bug at go/clrobot-bug.

#codehealth

PiperOrigin-RevId: 368586454
parent 14e085d6
...@@ -140,6 +140,7 @@ import javax.microedition.khronos.opengles.GL10; ...@@ -140,6 +140,7 @@ import javax.microedition.khronos.opengles.GL10;
case "scaleY": case "scaleY":
uniform.setFloat(bitmapScaleY); uniform.setFloat(bitmapScaleY);
break; break;
default: // fall out
} }
} }
for (GlUtil.Attribute copyExternalAttribute : attributes) { for (GlUtil.Attribute copyExternalAttribute : attributes) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment