Strip Unicode U+202B (right-to-left-embedding) code; it looks like DoM does RTL ...

[libsub.git] / src / stl_binary_reader.cc

diff --git a/src/stl_binary_reader.cc b/src/stl_binary_reader.cc
index d47768419996409129648bcb73ee7bd3cad30585..55e0188fdcc8011f25400cb13ae45f212ff64f97 100644 (file)
--- a/src/stl_binary_reader.cc
+++ b/src/stl_binary_reader.cc
@@ -94,6 +94,12 @@ STLBinaryReader::STLBinaryReader (istream& in)
                vector<string> lines;
                split (lines, whole, is_any_of ("\x8a"));
 
+               /* Italic / underline specifications can span lines, so we need to track them
+                  outside the lines loop.
+               */
+               bool italic = false;
+               bool underline = false;
+
                for (size_t i = 0; i < lines.size(); ++i) {
                        RawSubtitle sub;
                        sub.from = get_timecode (5);
@@ -101,7 +107,25 @@ STLBinaryReader::STLBinaryReader (istream& in)
                        sub.vertical_position.line = get_int (13, 1) + i;
                        sub.vertical_position.lines = maximum_rows;
                        sub.vertical_position.reference = TOP_OF_SCREEN;
+                       sub.italic = italic;
+                       sub.underline = underline;
+
+                       /* XXX: not sure what to do with JC = 0, "unchanged presentation" */
+                       int const h = get_int (14, 1);
+                       switch (h) {
+                       case 0:
+                       case 2:
+                               sub.horizontal_position.reference = HORIZONTAL_CENTRE_OF_SCREEN;
+                               break;
+                       case 1:
+                               sub.horizontal_position.reference = LEFT_OF_SCREEN;
+                               break;
+                       case 3:
+                               sub.horizontal_position.reference = RIGHT_OF_SCREEN;
+                               break;
+                       }
 
+                       /* Loop over characters */
                        string text;
                        for (size_t j = 0; j < lines[i].size(); ++j) {
 
@@ -121,21 +145,24 @@ STLBinaryReader::STLBinaryReader (istream& in)
 
                                switch (c) {
                                case 0x80:
-                                       sub.italic = true;
+                                       italic = true;
                                        break;
                                case 0x81:
-                                       sub.italic = false;
+                                       italic = false;
                                        break;
                                case 0x82:
-                                       sub.underline = true;
+                                       underline = true;
                                        break;
                                case 0x83:
-                                       sub.underline = false;
+                                       underline = false;
                                        break;
                                default:
                                        text += lines[i][j];
                                        break;
                                }
+
+                               sub.italic = italic;
+                               sub.underline = underline;
                        }
 
                        if (!text.empty ()) {