Don't fill in vertical_position.lines with an arbitrary value.
authorCarl Hetherington <cth@carlh.net>
Thu, 9 Jun 2016 13:06:41 +0000 (14:06 +0100)
committerCarl Hetherington <cth@carlh.net>
Thu, 9 Jun 2016 13:06:41 +0000 (14:06 +0100)
src/stl_text_reader.cc

index c4c10907dca3bee52b05d33eee8ab79207288b07..c7b1fcfd91198d01c04d183afb2e55e40eb5d11b 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2014-2015 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2014-2016 Carl Hetherington <cth@carlh.net>
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -39,10 +39,11 @@ using namespace sub;
 
 STLTextReader::STLTextReader (istream& in)
 {
+       /* This reader extracts no information about where the subtitle
+          should be on screen, so its reference is TOP_OF_SUBTITLE.
+       */
        _subtitle.vertical_position.line = 0;
-       /* XXX: no idea what this should be */
-       _subtitle.vertical_position.lines = 32;
-       _subtitle.vertical_position.reference = TOP_OF_SCREEN;
+       _subtitle.vertical_position.reference = TOP_OF_SUBTITLE;
 
        while (in.good ()) {
                string line;