X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fsubtitle_string.cc;h=f5f8c8acb34e60d0bc9332681a9763c06213278a;hb=52c79bb908dcfbb8e4f075b728699fce466ecef1;hp=cf0ddf41623b4ac4277a6b019f8419ad96d998d7;hpb=1e558b3f3a2435ead56426e892591e85d2da6887;p=libdcp.git

diff --git a/src/subtitle_string.cc b/src/subtitle_string.cc
index cf0ddf41..f5f8c8ac 100644
--- a/src/subtitle_string.cc
+++ b/src/subtitle_string.cc
@@ -1,20 +1,34 @@
 /*
-    Copyright (C) 2012-2015 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2016 Carl Hetherington <cth@carlh.net>
 
-    This program is free software; you can redistribute it and/or modify
+    This file is part of libdcp.
+
+    libdcp is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
     the Free Software Foundation; either version 2 of the License, or
     (at your option) any later version.
 
-    This program is distributed in the hope that it will be useful,
+    libdcp is distributed in the hope that it will be useful,
     but WITHOUT ANY WARRANTY; without even the implied warranty of
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
     GNU General Public License for more details.
 
     You should have received a copy of the GNU General Public License
-    along with this program; if not, write to the Free Software
-    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+    along with libdcp.  If not, see <http://www.gnu.org/licenses/>.
+
+    In addition, as a special exception, the copyright holders give
+    permission to link the code of portions of this program with the
+    OpenSSL library under certain conditions as described in each
+    individual source file, and distribute linked combinations
+    including the two.
 
+    You must obey the GNU General Public License in all respects
+    for all of the code used other than OpenSSL.  If you modify
+    file(s) with this exception, you may extend this exception to your
+    version of the file(s), but you are not obligated to do so.  If you
+    do not wish to do so, delete this exception statement from your
+    version.  If you delete this exception statement from all source
+    files in the program, then also delete it here.
 */
 
 #include "subtitle_string.h"
@@ -30,6 +44,8 @@ using namespace dcp;
 SubtitleString::SubtitleString (
 	optional<string> font,
 	bool italic,
+	bool bold,
+	bool underline,
 	Colour colour,
 	int size,
 	float aspect_adjust,
@@ -39,28 +55,25 @@ SubtitleString::SubtitleString (
 	HAlign h_align,
 	float v_position,
 	VAlign v_align,
+	Direction direction,
 	string text,
 	Effect effect,
 	Colour effect_colour,
 	Time fade_up_time,
 	Time fade_down_time
 	)
-	: _font (font)
+	: Subtitle (in, out, h_position, h_align, v_position, v_align, fade_up_time, fade_down_time)
+	, _font (font)
 	, _italic (italic)
+	, _bold (bold)
+	, _underline (underline)
 	, _colour (colour)
 	, _size (size)
 	, _aspect_adjust (aspect_adjust)
-	, _in (in)
-	, _out (out)
-	, _h_position (h_position)
-	, _h_align (h_align)
-	, _v_position (v_position)
-	, _v_align (v_align)
+	, _direction (direction)
 	, _text (text)
 	, _effect (effect)
 	, _effect_colour (effect_colour)
-	, _fade_up_time (fade_up_time)
-	, _fade_down_time (fade_down_time)
 {
 
 }
@@ -72,7 +85,7 @@ SubtitleString::size_in_pixels (int screen_height) const
 	   height is 11 inches, so a 72pt font would be 1/11th of the screen
 	   height.
 	*/
-	
+
 	return _size * screen_height / (11 * 72);
 }
 
@@ -82,6 +95,8 @@ dcp::operator== (SubtitleString const & a, SubtitleString const & b)
 	return (
 		a.font() == b.font() &&
 		a.italic() == b.italic() &&
+		a.bold() == b.bold() &&
+		a.underline() == b.underline() &&
 		a.colour() == b.colour() &&
 		a.size() == b.size() &&
 		fabs (a.aspect_adjust() - b.aspect_adjust()) < ASPECT_ADJUST_EPSILON &&
@@ -91,6 +106,7 @@ dcp::operator== (SubtitleString const & a, SubtitleString const & b)
 		a.h_align() == b.h_align() &&
 		a.v_position() == b.v_position() &&
 		a.v_align() == b.v_align() &&
+		a.direction() == b.direction() &&
 		a.text() == b.text() &&
 		a.effect() == b.effect() &&
 		a.effect_colour() == b.effect_colour() &&
@@ -99,6 +115,12 @@ dcp::operator== (SubtitleString const & a, SubtitleString const & b)
 		);
 }
 
+bool
+dcp::operator!= (SubtitleString const & a, SubtitleString const & b)
+{
+	return !(a == b);
+}
+
 ostream&
 dcp::operator<< (ostream& s, SubtitleString const & sub)
 {
@@ -107,14 +129,25 @@ dcp::operator<< (ostream& s, SubtitleString const & sub)
 	  << "font " << sub.font().get_value_or ("[default]") << ", ";
 
 	if (sub.italic()) {
-		s << "italic";
+		s << "italic, ";
 	} else {
-		s << "non-italic";
+		s << "non-italic, ";
 	}
-	
-	s << ", size " << sub.size() << ", aspect " << sub.aspect_adjust() << ", colour " << sub.colour()
+
+	if (sub.bold()) {
+		s << "bold, ";
+	} else {
+		s << "normal, ";
+	}
+
+	if (sub.underline()) {
+		s << "underlined, ";
+	}
+
+	s << "size " << sub.size() << ", aspect " << sub.aspect_adjust() << ", colour " << sub.colour()
 	  << ", vpos " << sub.v_position() << ", valign " << ((int) sub.v_align())
 	  << ", hpos " << sub.h_position() << ", halign " << ((int) sub.h_align())
+	  << ", direction " << ((int) sub.direction())
 	  << ", effect " << ((int) sub.effect()) << ", effect colour " << sub.effect_colour();
 
 	return s;