Add default to switch in TtmlDecoder

Also de-dupe a couple of case statements PiperOrigin-RevId: 290610993

Add default to switch in TtmlDecoder
Also de-dupe a couple of case statements PiperOrigin-RevId: 290610993
04d32533 · ibaker · Ian Baker · 9cfc75ff · 04d32533
Commit 04d32533 authored Jan 20, 2020 by ibaker Committed by Ian Baker Jan 24, 2020
Showing with 3 additions and 4 deletions
library/core/src/main/java/com/google/android/exoplayer2/text/ttml/TtmlDecoder.java
--- a/library/core/src/main/java/com/google/android/exoplayer2/text/ttml/TtmlDecoder.java
+++ b/library/core/src/main/java/com/google/android/exoplayer2/text/ttml/TtmlDecoder.java
@@ -488,20 +488,19 @@ public final class TtmlDecoder extends SimpleSubtitleDecoder {
        case TtmlNode.ATTR_TTS_TEXT_ALIGN:
          switch (Util.toLowerInvariant(attributeValue)) {
            case TtmlNode.LEFT:
-              style = createIfNull(style).setTextAlign(Layout.Alignment.ALIGN_NORMAL);
-              break;
            case TtmlNode.START:
              style = createIfNull(style).setTextAlign(Layout.Alignment.ALIGN_NORMAL);
              break;
            case TtmlNode.RIGHT:
-              style = createIfNull(style).setTextAlign(Layout.Alignment.ALIGN_OPPOSITE);
-              break;
            case TtmlNode.END:
              style = createIfNull(style).setTextAlign(Layout.Alignment.ALIGN_OPPOSITE);
              break;
            case TtmlNode.CENTER:
              style = createIfNull(style).setTextAlign(Layout.Alignment.ALIGN_CENTER);
              break;
+            default:
+              // ignore
+              break;
          }
          break;
        case TtmlNode.ATTR_TTS_TEXT_DECORATION: