CEA608 - Add space when handling mid-row codes

Issue: #3906 ------------- Created by MOE: https:/google/moe MOE_MIGRATED_REVID=200526335
google · Jun 18, 2018 · afc19bf · afc19bf
1 parent 4dc6c85
commit afc19bf
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/RELEASENOTES.md b/RELEASENOTES.md
@@ -45,6 +45,8 @@
  ([#4315](https:/google/ExoPlayer/issues/4315)).
 * Set `METADATA_KEY_TITLE` on media descriptions
  ((#4292)[https:/google/ExoPlayer/issues/4292]).
+* Fix missing whitespace in CEA-608
+ ([#3906](https:/google/ExoPlayer/issues/3906)).
 
 ### 2.8.1 ###
 

diff --git a/library/core/src/main/java/com/google/android/exoplayer2/text/cea/Cea608Decoder.java b/library/core/src/main/java/com/google/android/exoplayer2/text/cea/Cea608Decoder.java
@@ -374,6 +374,9 @@ private boolean handleCtrl(byte cc1, byte cc2) {
  private void handleMidrowCtrl(byte cc2) {
  // TODO: support the extended styles (i.e. backgrounds and transparencies)
 
+ // A midrow control code advances the cursor.
+ currentCueBuilder.append(' ');
+
  // cc2 - 0|0|1|0|ATRBT|U
  // ATRBT is the 3-byte encoded attribute, and U is the underline toggle
  boolean isUnderlined = (cc2 & 0x01) == 0x01;