OSDN Git Service

MacGui: Remove Target Size as a rate control option as it doesn't really work correct...
[handbrake-jp/handbrake-jp-git.git] / gtk / src / subtitlehandler.c
index 255da47..0746b26 100644 (file)
@@ -1,7 +1,7 @@
 /* -*- Mode: C; indent-tabs-mode: t; c-basic-offset: 4; tab-width: 4 -*- */
 /*
  * subtitlehandler.c
- * Copyright (C) John Stebbins 2008 <stebbins@stebbins>
+ * Copyright (C) John Stebbins 2008-2011 <stebbins@stebbins>
  * 
  * subtitlehandler.c is free software.
  * 
 #include "values.h"
 #include "callbacks.h"
 #include "preview.h"
+#include "presets.h"
+#include "audiohandler.h"
 #include "subtitlehandler.h"
 
-static void add_to_subtitle_list(
-       signal_user_data_t *ud, GValue *settings, gboolean can_delete);
+static void add_to_subtitle_list(signal_user_data_t *ud, GValue *settings);
+static void add_to_srt_list(signal_user_data_t *ud, GValue *settings);
 
-void
+static void
 free_subtitle_index_list(gpointer data)
 {
        g_free(data);
 }
 
-void
+static void
 free_subtitle_key(gpointer data)
 {
        if (data != NULL)
                g_free(data);
 }
 
+static gboolean
+mustBurn(signal_user_data_t *ud, GValue *settings)
+{
+       return FALSE;
+}
+
+static gboolean
+canBurn(int source)
+{
+       return (source == VOBSUB || source == SSASUB);
+}
+
+static gboolean
+canForce(int source)
+{
+       return (source == VOBSUB);
+}
+
+gboolean
+ghb_soft_in_subtitle_list(GValue *subtitle_list)
+{
+       gint count, ii;
+       GValue *settings;
+
+       count = ghb_array_len(subtitle_list);
+       for (ii = 0; ii < count; ii++)
+       {
+               settings = ghb_array_get_nth(subtitle_list, ii);
+               if (!ghb_settings_get_boolean(settings, "SubtitleBurned"))
+               {
+                       return TRUE;
+               }
+       }
+       return FALSE;
+}
+
+void
+ghb_subtitle_exclusive_burn(signal_user_data_t *ud, gint index)
+{
+       GValue *subtitle_list;
+       GValue *settings;
+       gint ii, count;
+       GtkTreeView  *tv;
+       GtkTreeModel *tm;
+       GtkTreeIter   ti;
+       gboolean burned;
+
+       g_debug("ghb_subtitle_exclusive_burn");
+       subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
+       count = ghb_array_len(subtitle_list);
+       for (ii = 0; ii < count; ii++)
+       {
+               settings = ghb_array_get_nth(subtitle_list, ii);
+               burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
+
+               tv = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "subtitle_list"));
+               g_return_if_fail(tv != NULL);
+               tm = gtk_tree_view_get_model(tv);
+               gtk_tree_model_iter_nth_child(tm, &ti, NULL, ii);
+               if (burned && ii != index && !mustBurn(ud, settings))
+               {
+                       ghb_settings_set_boolean(settings, "SubtitleBurned", FALSE);
+                       gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 2, FALSE, -1);
+               }
+       }
+}
+
+void
+ghb_subtitle_exclusive_default(signal_user_data_t *ud, gint index)
+{
+       GValue *subtitle_list;
+       GValue *settings;
+       gint ii, count;
+       GtkTreeView  *tv;
+       GtkTreeModel *tm;
+       GtkTreeIter   ti;
+       gboolean def;
+
+       g_debug("ghb_subtitle_exclusive_default");
+       subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
+       count = ghb_array_len(subtitle_list);
+       for (ii = 0; ii < count; ii++)
+       {
+               settings = ghb_array_get_nth(subtitle_list, ii);
+               def = ghb_settings_get_boolean(settings, "SubtitleDefaultTrack");
+
+               tv = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "subtitle_list"));
+               g_return_if_fail(tv != NULL);
+               tm = gtk_tree_view_get_model(tv);
+               gtk_tree_model_iter_nth_child(tm, &ti, NULL, ii);
+               if (def && ii != index)
+               {
+
+                       ghb_settings_set_boolean(settings, "SubtitleDefaultTrack", FALSE);
+                       gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 3, FALSE, -1);
+               }
+       }
+}
+
+void
+ghb_add_srt(signal_user_data_t *ud, GValue *settings)
+{
+       // Add the current subtitle settings to the list.
+       GValue *subtitle_list;
+       gint count;
+       const gchar *lang;
+       
+       g_debug("ghb_add_srt ()");
+
+       ghb_settings_set_boolean(settings, "SubtitleBurned", FALSE);
+       // Add the long track description so the queue can access it
+       // when a different title is selected.
+       lang = ghb_settings_combo_option(settings, "SrtLanguage");
+       ghb_settings_set_string(settings, "SubtitleTrackDescription", lang);
+
+       subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
+       if (subtitle_list == NULL)
+       {
+               subtitle_list = ghb_array_value_new(8);
+               ghb_settings_set_value(ud->settings, "subtitle_list", subtitle_list);
+       }
+       count = ghb_array_len(subtitle_list);
+
+       // Don't allow more than 99
+       // This is a had limit imposed by libhb/sync.c:GetFifoForId()
+       if (count >= 99)
+       {
+               ghb_value_free(settings);
+               return;
+       }
+
+       ghb_array_append(subtitle_list, settings);
+       add_to_srt_list(ud, settings);
+
+       if (count == 98)
+       {
+               GtkWidget *widget;
+               widget = GHB_WIDGET (ud->builder, "subtitle_add");
+               gtk_widget_set_sensitive(widget, FALSE);
+               widget = GHB_WIDGET (ud->builder, "srt_add");
+               gtk_widget_set_sensitive(widget, FALSE);
+       }
+       ghb_update_destination_extension(ud);
+       ghb_live_reset(ud);
+}
+
 void
-ghb_add_subtitle(signal_user_data_t *ud, GValue *settings, gboolean can_delete)
+ghb_add_subtitle(signal_user_data_t *ud, GValue *settings)
 {
        // Add the current subtitle settings to the list.
        GValue *subtitle_list;
+       gint count;
+       gboolean burned;
+       const gchar *track;
+       const gchar *lang;
        
        g_debug("ghb_add_subtitle ()");
 
-       // Only allow up to 8 subtitle entries
+       // Add the long track description so the queue can access it
+       // when a different title is selected.
+       track = ghb_settings_combo_option(settings, "SubtitleTrack");
+       ghb_settings_set_string(settings, "SubtitleTrackDescription", track);
+
+       lang = ghb_settings_combo_string(settings, "SubtitleTrack");
+       ghb_settings_set_string(settings, "SubtitleLanguage", lang);
+
        subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
        if (subtitle_list == NULL)
        {
                subtitle_list = ghb_array_value_new(8);
                ghb_settings_set_value(ud->settings, "subtitle_list", subtitle_list);
        }
+       count = ghb_array_len(subtitle_list);
+
+       // Don't allow more than 99
+       // This is a had limit imposed by libhb/sync.c:GetFifoForId()
+       if (count >= 99)
+       {
+               ghb_value_free(settings);
+               return;
+       }
+
        ghb_array_append(subtitle_list, settings);
-       add_to_subtitle_list(ud, settings, can_delete);
+       add_to_subtitle_list(ud, settings);
+
+       burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
+       if (burned)
+               ghb_subtitle_exclusive_burn(ud, count);
+       if (count == 98)
+       {
+               GtkWidget *widget;
+               widget = GHB_WIDGET (ud->builder, "subtitle_add");
+               gtk_widget_set_sensitive(widget, FALSE);
+               widget = GHB_WIDGET (ud->builder, "srt_add");
+               gtk_widget_set_sensitive(widget, FALSE);
+       }
+       ghb_update_destination_extension(ud);
+       ghb_live_reset(ud);
 }
 
 static void
@@ -76,9 +259,13 @@ add_all_pref_subtitles(signal_user_data_t *ud)
                g_free(lang);
                if (track >= -1)
                {
+                       int source;
+
                        // Add to subtitle list
                        ghb_settings_set_int(subtitle, "SubtitleTrack", track);
-                       ghb_add_subtitle(ud, subtitle, TRUE);
+                       source = ghb_subtitle_track_source(ud, track);
+                       ghb_settings_set_int(subtitle, "SubtitleSource", source);
+                       ghb_add_subtitle(ud, subtitle);
                }
        }
 }
@@ -88,15 +275,38 @@ ghb_set_pref_subtitle(gint titleindex, signal_user_data_t *ud)
 {
        gint track;
        GHashTable *track_indices;
-       char *lang;
+       gchar *lang, *pref_lang = NULL;
+       gchar *audio_lang;
+       gint foreign_lang_index = -1;
+       gboolean found_cc = FALSE;
 
        const GValue *pref_subtitle;
        GValue *subtitle;
-       gint count, ii;
+       gint count, ii, jj;
        
        g_debug("ghb_set_pref_subtitle %d", titleindex);
+
+       // Check to see if we need to add a subtitle track for foreign audio
+       // language films. A subtitle track will be added if:
+       //
+       // The first (default) audio track language does NOT match the users
+       // chosen Preferred Language AND the Preferred Language is NOT Any (und).
+       //
+       audio_lang = ghb_get_user_audio_lang(ud, titleindex, 0);
+       pref_lang = ghb_settings_get_string(ud->settings, "PreferredLanguage");
+
+       if (audio_lang != NULL && pref_lang != NULL &&
+               (strcmp(audio_lang, pref_lang) == 0 || strcmp("und", pref_lang) == 0))
+       {
+               g_free(pref_lang);
+               pref_lang = NULL;
+       }
+
        track_indices = g_hash_table_new_full(g_str_hash, g_str_equal, 
-                                                                                       NULL, free_subtitle_index_list);
+                                                                                       free_subtitle_key, free_subtitle_index_list);
+
+       ghb_ui_update(ud, "SubtitleTrack", ghb_int_value(0));
+
        // Clear the subtitle list
        ghb_clear_subtitle_list(ud);
        if (titleindex < 0)
@@ -104,26 +314,123 @@ ghb_set_pref_subtitle(gint titleindex, signal_user_data_t *ud)
                add_all_pref_subtitles(ud);
                return;
        }
-       ghb_add_all_subtitles(ud, titleindex);
 
        // Find "best" subtitle based on subtitle preferences
        pref_subtitle = ghb_settings_get_value(ud->settings, "SubtitleList");
 
        count = ghb_array_len(pref_subtitle);
+       jj = 0;
        for (ii = 0; ii < count; ii++)
        {
+               gint source;
+               gboolean force, burn;
+
                subtitle = ghb_array_get_nth(pref_subtitle, ii);
                lang = ghb_settings_get_string(subtitle, "SubtitleLanguage");
+               source = ghb_settings_get_int(subtitle, "SubtitleSource");
+               burn = ghb_settings_get_boolean(subtitle, "SubtitleBurned");
+               force = ghb_settings_get_boolean(subtitle, "SubtitleForced");
                // If there are multiple subtitles using the same language, then
                // select sequential tracks for each.  The hash keeps track 
                // of the tracks used for each language.
-               track = ghb_find_subtitle_track(titleindex, lang, track_indices);
+               track = ghb_find_subtitle_track(titleindex, lang, burn, 
+                                                                               force, source, track_indices);
                g_free(lang);
                if (track >= -1)
                {
-                       ghb_set_subtitle(ud, track, subtitle);
+                       GValue *dup = ghb_value_dup(subtitle);
+                       lang = ghb_subtitle_track_lang(ud, track);
+                       ghb_settings_set_int(dup, "SubtitleTrack", track);
+                       if (foreign_lang_index < 0 && pref_lang != NULL &&
+                               strcmp(lang, pref_lang) == 0)
+                       {
+                               foreign_lang_index = jj;
+                               ghb_settings_take_value(dup, "SubtitleForced", 
+                                                               ghb_boolean_value_new(FALSE));
+                               ghb_settings_take_value(dup, "SubtitleDefaultTrack", 
+                                                               ghb_boolean_value_new(TRUE));
+                       }
+                       source = ghb_subtitle_track_source(ud, track);
+                       ghb_settings_set_int(dup, "SubtitleSource", source);
+                       if (source == CC608SUB || source == CC708SUB)
+                               found_cc = TRUE;
+                       ghb_add_subtitle(ud, dup);
+                       jj++;
+                       g_free(lang);
+               }
+       }
+       if (foreign_lang_index < 0 && pref_lang != NULL)
+       {
+               // Subtitle for foreign language audio not added yet
+               GValue *settings;
+               gboolean burn;
+
+               track = ghb_find_subtitle_track(titleindex, pref_lang, FALSE, FALSE, VOBSUB, track_indices);
+               if (track >= -1)
+               {
+                       int source;
+
+                       settings = ghb_dict_value_new();
+                       ghb_settings_set_int(settings, "SubtitleTrack", track);
+                       source = ghb_subtitle_track_source(ud, track);
+                       ghb_settings_set_int(settings, "SubtitleSource", source);
+                       burn = mustBurn(ud, settings);
+                       ghb_settings_take_value(settings, "SubtitleForced", 
+                                                       ghb_boolean_value_new(FALSE));
+                       ghb_settings_take_value(settings, "SubtitleBurned", 
+                                                       ghb_boolean_value_new(burn));
+                       ghb_settings_take_value(settings, "SubtitleDefaultTrack", 
+                                                       ghb_boolean_value_new(TRUE));
+
+                       ghb_add_subtitle(ud, settings);
+                       foreign_lang_index = jj;
+               }
+       }
+       if (foreign_lang_index >= 0)
+       {
+               GValue *subtitle_list;
+               gboolean burn, def;
+
+               subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
+               subtitle = ghb_array_get_nth(subtitle_list, foreign_lang_index);
+
+               burn = ghb_settings_get_boolean(subtitle, "SubtitleBurned");
+               def = ghb_settings_get_boolean(subtitle, "SubtitleDefaultTrack");
+               if (burn)
+                       ghb_subtitle_exclusive_burn(ud, foreign_lang_index);
+               if (def)
+                       ghb_subtitle_exclusive_default(ud, foreign_lang_index);
+               ghb_log("adding subtitle for foreign language audio: %s", audio_lang);
+       }
+       if (ghb_settings_get_boolean(ud->settings, "AddCC") && !found_cc)
+       {
+               // Subtitle for foreign language audio not added yet
+               GValue *settings;
+
+               track = ghb_find_cc_track(titleindex);
+               if (track >= 0)
+               {
+                       int source;
+
+                       settings = ghb_dict_value_new();
+                       ghb_settings_set_int(settings, "SubtitleTrack", track);
+                       source = ghb_subtitle_track_source(ud, track);
+                       ghb_settings_set_int(settings, "SubtitleSource", source);
+                       ghb_settings_take_value(settings, "SubtitleForced", 
+                                                       ghb_boolean_value_new(FALSE));
+                       ghb_settings_take_value(settings, "SubtitleBurned", 
+                                                       ghb_boolean_value_new(FALSE));
+                       ghb_settings_take_value(settings, "SubtitleDefaultTrack", 
+                                                       ghb_boolean_value_new(FALSE));
+
+                       ghb_add_subtitle(ud, settings);
+                       ghb_log("adding Closed Captions: %s", audio_lang);
                }
        }
+       if (pref_lang != NULL)
+               g_free(pref_lang);
+       if (audio_lang != NULL)
+               g_free(audio_lang);
        g_hash_table_destroy(track_indices);
 }
 
@@ -152,27 +459,6 @@ ghb_selected_subtitle_row(signal_user_data_t *ud)
        return row;
 }
 
-static gboolean
-mustBurn(signal_user_data_t *ud, gint track)
-{
-       gint mux;
-
-       mux = ghb_settings_combo_int(ud->settings, "FileFormat");
-       if (mux == HB_MUX_MP4)
-       {
-               gint source;
-
-               // MP4 can only handle burned vobsubs.  make sure there isn't
-               // already something burned in the list
-               source = ghb_subtitle_track_source(ud, track);
-               if (source == VOBSUB)
-               {
-                       return TRUE;
-               }
-       }
-       return FALSE;
-}
-
 GValue*
 ghb_selected_subtitle_settings(signal_user_data_t *ud)
 {
@@ -206,47 +492,8 @@ ghb_selected_subtitle_settings(signal_user_data_t *ud)
        return settings;
 }
 
-void
-ghb_subtitle_exclusive_burn(signal_user_data_t *ud, gint track)
-{
-       GValue *subtitle_list;
-       GValue *settings;
-       gint ii, count, tt;
-       GtkTreeView  *tv;
-       GtkTreeModel *tm;
-       GtkTreeIter   ti;
-       gboolean burned;
-
-       g_debug("ghb_subtitle_exclusive_burn");
-       subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
-       count = ghb_array_len(subtitle_list);
-       for (ii = 0; ii < count; ii++)
-       {
-               settings = ghb_array_get_nth(subtitle_list, ii);
-               tt = ghb_settings_combo_int(settings, "SubtitleTrack");
-               burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
-
-               tv = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "subtitle_list"));
-               g_return_if_fail(tv != NULL);
-               tm = gtk_tree_view_get_model(tv);
-               gtk_tree_model_iter_nth_child(tm, &ti, NULL, ii);
-               if (burned && tt != track)
-               {
-
-                       ghb_settings_set_boolean(settings, "SubtitleBurned", FALSE);
-                       burned = FALSE;
-                       gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 3, FALSE, -1);
-               }
-               if (!burned && mustBurn(ud, tt))
-               {
-                       ghb_settings_set_boolean(settings, "SubtitleEnabled", FALSE);
-                       gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 0, FALSE, -1);
-               }
-       }
-}
-
 G_MODULE_EXPORT void
-subtitle_enable_toggled_cb(
+subtitle_forced_toggled_cb(
        GtkCellRendererToggle *cell, 
        gchar                 *path,
        signal_user_data_t    *ud)
@@ -259,7 +506,7 @@ subtitle_enable_toggled_cb(
        GtkTreePath  *tp;
        gint *indices;
        GValue *subtitle_list, *settings;
-       gint track;
+       gint source;
 
        g_debug("forced toggled");
        tp = gtk_tree_path_new_from_string (path);
@@ -268,34 +515,32 @@ subtitle_enable_toggled_cb(
        tm = gtk_tree_view_get_model(tv);
        g_return_if_fail(tm != NULL);
        gtk_tree_model_get_iter(tm, &ti, tp);
-       gtk_tree_model_get(tm, &ti, 0, &active, -1);
+       gtk_tree_model_get(tm, &ti, 1, &active, -1);
        active ^= 1;
 
        // Get the row number
        indices = gtk_tree_path_get_indices (tp);
        row = indices[0];
        gtk_tree_path_free(tp);
+
        subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
 
        if (row < 0 || row >= ghb_array_len(subtitle_list))
                return;
 
        settings = ghb_array_get_nth(subtitle_list, row);
-       track = ghb_settings_combo_int(settings, "SubtitleTrack");
 
-       if (active && mustBurn(ud, track))
-       {
-               ghb_settings_set_boolean(settings, "SubtitleBurned", TRUE);
-               gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 3, TRUE, -1);
-               ghb_subtitle_exclusive_burn(ud, track);
-       }
+       source = ghb_settings_get_int(settings, "SubtitleSource");
+       if (!canForce(source))
+               return;
 
-       ghb_settings_set_boolean(settings, "SubtitleEnabled", active);
-       gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 0, active, -1);
+       ghb_settings_set_boolean(settings, "SubtitleForced", active);
+       gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 1, active, -1);
+       ghb_live_reset(ud);
 }
 
 G_MODULE_EXPORT void
-subtitle_forced_toggled_cb(
+subtitle_burned_toggled_cb(
        GtkCellRendererToggle *cell, 
        gchar                 *path,
        signal_user_data_t    *ud)
@@ -303,14 +548,16 @@ subtitle_forced_toggled_cb(
        GtkTreeView  *tv;
        GtkTreeModel *tm;
        GtkTreeIter   ti;
+       GtkTreePath  *tp;
        gboolean      active;
        gint          row;
-       GtkTreePath  *tp;
        gint *indices;
-       GValue *subtitle_list, *settings;
-       gint source, track;
+       GValue *subtitle_list;
+       gint count;
+       GValue *settings;
+       gint source;
 
-       g_debug("forced toggled");
+       g_debug("burned toggled");
        tp = gtk_tree_path_new_from_string (path);
        tv = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "subtitle_list"));
        g_return_if_fail(tv != NULL);
@@ -326,23 +573,35 @@ subtitle_forced_toggled_cb(
        gtk_tree_path_free(tp);
 
        subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
-
-       if (row < 0 || row >= ghb_array_len(subtitle_list))
+       count = ghb_array_len(subtitle_list);
+       if (row < 0 || row >= count)
                return;
 
        settings = ghb_array_get_nth(subtitle_list, row);
-       track = ghb_settings_combo_int(settings, "SubtitleTrack");
-
-       source = ghb_subtitle_track_source(ud, track);
-       if (source != VOBSUB)
+       source = ghb_settings_get_int(settings, "SubtitleSource");
+       if (!canBurn(source))
+               return;
+       if (!active && mustBurn(ud, settings))
                return;
 
-       ghb_settings_set_boolean(settings, "SubtitleForced", active);
+       ghb_settings_set_boolean(settings, "SubtitleBurned", active);
        gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 2, active, -1);
+
+       if (active)
+       {
+               ghb_settings_set_boolean(settings, "SubtitleDefaultTrack", !active);
+               gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 3, !active, -1);
+       }
+
+       // Unburn the rest
+       if (active)
+               ghb_subtitle_exclusive_burn(ud, row);
+       ghb_update_destination_extension(ud);
+       ghb_live_reset(ud);
 }
 
 G_MODULE_EXPORT void
-subtitle_burned_toggled_cb(
+subtitle_default_toggled_cb(
        GtkCellRendererToggle *cell, 
        gchar                 *path,
        signal_user_data_t    *ud)
@@ -355,10 +614,10 @@ subtitle_burned_toggled_cb(
        gint          row;
        gint *indices;
        GValue *subtitle_list;
-       gint count, track, source;
+       gint count;
        GValue *settings;
 
-       g_debug("burned toggled");
+       g_debug("default toggled");
        tp = gtk_tree_path_new_from_string (path);
        tv = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "subtitle_list"));
        g_return_if_fail(tv != NULL);
@@ -379,178 +638,240 @@ subtitle_burned_toggled_cb(
                return;
 
        settings = ghb_array_get_nth(subtitle_list, row);
-       track = ghb_settings_combo_int(settings, "SubtitleTrack");
-
-       source = ghb_subtitle_track_source(ud, track);
-       if (source != VOBSUB)
-               return;
-
-       if (!active && mustBurn(ud, track))
+       if (active && mustBurn(ud, settings))
                return;
 
-       ghb_settings_set_boolean(settings, "SubtitleBurned", active);
-
+       ghb_settings_set_boolean(settings, "SubtitleDefaultTrack", active);
        gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 3, active, -1);
-       // Unburn the rest
-       ghb_subtitle_exclusive_burn(ud, track);
-}
-
-static gboolean
-trackUsed(signal_user_data_t *ud, gint track)
-{
-       gint ii, count, tt;
-       GValue *settings, *subtitle_list;
 
-       subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
-       count = ghb_array_len(subtitle_list);
-       for (ii = 0; ii < count; ii++)
+       if (active)
        {
-               settings = ghb_array_get_nth(subtitle_list, ii);
-               tt = ghb_settings_combo_int(settings, "SubtitleTrack");
-               if (tt == track)
-               {
-                       return TRUE;
-               }
+               ghb_settings_set_boolean(settings, "SubtitleBurned", !active);
+               gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 2, !active, -1);
        }
-       return FALSE;
+       // allow only one default
+       ghb_subtitle_exclusive_default(ud, row);
+       ghb_live_reset(ud);
 }
 
-G_MODULE_EXPORT void
-subtitle_track_changed_cb(
-       GtkCellRendererCombo *combo,
-       gchar *path,
-       GtkTreeIter *iter,
-       signal_user_data_t *ud)
+static void
+subtitle_list_refresh_selected(signal_user_data_t *ud)
 {
-       GtkTreeView  *tv;
-       GtkTreeModel *tm;
-       GtkTreeIter   ti;
-       GtkTreePath  *tp;
-       gdouble dtrack;
+       GtkTreeView *treeview;
+       GtkTreePath *treepath;
+       GtkTreeSelection *selection;
+       GtkTreeModel *store;
+       GtkTreeIter iter;
        gint *indices;
-       int tt, row;
-       GValue *subtitle_list, *settings;
-       const char *lang;
+       gint row;
+       GValue *settings = NULL;
+       const GValue *subtitle_list;
+       gboolean allow_force = FALSE;
+       gboolean allow_burn = FALSE;
+       
+       g_debug("subtitle_list_refresh_selected ()");
+       treeview = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "subtitle_list"));
+       selection = gtk_tree_view_get_selection (treeview);
+       if (gtk_tree_selection_get_selected(selection, &store, &iter))
+       {
+               gchar *track, *source;
+               gboolean forced, burned, def;
+               gchar *s_track;
+               gint offset = 0;
+       
+               // Get the row number
+               treepath = gtk_tree_model_get_path (store, &iter);
+               indices = gtk_tree_path_get_indices (treepath);
+               row = indices[0];
+               gtk_tree_path_free(treepath);
+               if (row < 0) return;
+               subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
+               if (row >= ghb_array_len(subtitle_list))
+                       return;
+               settings = ghb_array_get_nth(subtitle_list, row);
 
-       g_debug("subtitle_track_changed_cb ()");
-       gtk_tree_model_get(GTK_TREE_MODEL(ud->subtitle_track_model), iter, 
-                                               3, &dtrack, -1);
-       tt = (gint)dtrack;
-       if (trackUsed(ud, tt))
-               return;
+               def = ghb_settings_get_boolean(settings, "SubtitleDefaultTrack");
 
-       tp = gtk_tree_path_new_from_string (path);
-       tv = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "subtitle_list"));
-       g_return_if_fail(tv != NULL);
-       tm = gtk_tree_view_get_model(tv);
-       g_return_if_fail(tm != NULL);
-       gtk_tree_model_get_iter(tm, &ti, tp);
+               gint i_source;
+               i_source = ghb_settings_get_int(settings, "SubtitleSource");
+               if (!canBurn(i_source))
+               {
+                       burned = FALSE;
+                       ghb_settings_set_boolean(settings, "SubtitleBurned", burned);
+               }
+               if (!canForce(i_source))
+               {
+                       // Force only apply to VOBSUBS
+                       forced = FALSE;
+                       ghb_settings_set_boolean(settings, "SubtitleForced", forced);
+               }
 
-       // Get the row number
-       indices = gtk_tree_path_get_indices (tp);
-       row = indices[0];
-       gtk_tree_path_free(tp);
+               if (i_source == SRTSUB)
+               {
+                       const gchar *lang;
+                       gchar *code;
 
-       subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
-       if (row >= ghb_array_len(subtitle_list))
-               return;
-       settings = ghb_array_get_nth(subtitle_list, row);
+                       lang = ghb_settings_combo_option(settings, "SrtLanguage");
+                       code = ghb_settings_get_string(settings, "SrtCodeset");
 
-       ghb_settings_set_int(settings, "SubtitleTrack", tt);
-       lang = ghb_settings_combo_string(settings, "SubtitleTrack");
-       if (mustBurn(ud, tt))
-       {
-               ghb_settings_set_boolean(settings, "SubtitleBurned", TRUE);
-       }
-       else
-       {
-               ghb_settings_set_boolean(settings, "SubtitleBurned", FALSE);
-       }
-       ghb_settings_set_string(settings, "SubtitleLanguage", lang);
+                       s_track = ghb_settings_get_string(settings, "SrtFile");
+                       if (g_file_test(s_track, G_FILE_TEST_IS_REGULAR))
+                       {
+                               gchar *basename;
 
-       const gchar *track, *source;
-       gboolean burned;
-       gchar *s_track;
-       gint i_track;
+                               basename = g_path_get_basename(s_track);
+                               track = g_strdup_printf("%s (%s)(SRT)(%s)", lang, code, basename);
+                               source = g_strdup_printf("SRT (%s)", basename);
+                               g_free(basename);
+                       }
+                       else
+                       {
+                               track = g_strdup_printf("%s (%s)(SRT)", lang, code);
+                               source = g_strdup_printf("SRT (none)");
+                       }
+                       g_free(code);
+                       offset = ghb_settings_get_int(settings, "SrtOffset");
 
-       track = ghb_settings_combo_option(settings, "SubtitleTrack");
-       burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
+                       forced = FALSE;
+                       burned = FALSE;
+               }
+               else
+               {
+                       track = g_strdup(
+                               ghb_settings_combo_option(settings, "SubtitleTrack"));
+                       source = g_strdup(ghb_subtitle_source_name(i_source));
+                       s_track = ghb_settings_get_string(settings, "SubtitleTrack");
 
-       s_track = ghb_settings_get_string(settings, "SubtitleTrack");
-       i_track = ghb_settings_get_int(settings, "SubtitleTrack");
-       source = ghb_subtitle_track_source_name(ud, i_track);
+                       forced = ghb_settings_get_boolean(settings, "SubtitleForced");
+                       burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
+               }
+
+               if (canBurn(i_source))
+                       allow_burn = TRUE;
+               if (canForce(i_source))
+                       allow_force = TRUE;
 
-       gtk_list_store_set(GTK_LIST_STORE(tm), &ti
+               gtk_list_store_set(GTK_LIST_STORE(store), &iter
                        // These are displayed in list
-                       1, track,
-                       3, burned,
-                       4, source,
-                       // These are used to set combo values when a list item is selected
+                       0, track,
+                       1, forced,
+                       2, burned,
+                       3, def,
+                       4, offset,
+                       // These are used to set combo box values when a list item is selected
                        5, s_track,
+                       6, i_source,
+                       7, allow_force,
+                       8, allow_burn,
                        -1);
-       g_free(s_track);
-       ghb_live_reset(ud);
-
+               g_free(track);
+               g_free(source);
+               g_free(s_track);
+               if (burned)
+                       ghb_subtitle_exclusive_burn(ud, row);
+       }
 }
 
-void
-ghb_subtitle_adjust_burn(signal_user_data_t *ud)
+G_MODULE_EXPORT void
+subtitle_track_changed_cb(GtkWidget *widget, signal_user_data_t *ud)
 {
-       GValue *subtitle_list;
        GValue *settings;
-       gint ii, count, track;
-       GtkTreeView  *tv;
-       GtkTreeModel *tm;
-       GtkTreeIter   ti;
-       gboolean burned, enabled, found;
 
-       g_debug("ghb_subtitle_adjust_burn");
-       subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
-       count = ghb_array_len(subtitle_list);
-       for (ii = 0; ii < count; ii++)
+       g_debug("subtitle_track_changed_cb ()");
+       ghb_check_dependency(ud, widget, NULL);
+       ghb_widget_to_setting(ud->settings, widget);
+       settings = ghb_selected_subtitle_settings(ud);
+       if (settings != NULL)
        {
-               settings = ghb_array_get_nth(subtitle_list, ii);
-               enabled = ghb_settings_get_boolean(settings, "SubtitleEnabled");
-               burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
-               if (enabled && burned)
-               {
-                       found = TRUE;
-                       break;
-               }
+               const gchar *track, *lang;
+               gint tt, source;
+
+               ghb_widget_to_setting(settings, widget);
+               track = ghb_settings_combo_option(settings, "SubtitleTrack");
+               ghb_settings_set_string(settings, "SubtitleTrackDescription", track);
+               tt = ghb_settings_get_int(settings, "SubtitleTrack");
+               source = ghb_subtitle_track_source(ud, tt);
+               ghb_settings_set_int(settings, "SubtitleSource", source);
+               lang = ghb_settings_combo_string(settings, "SubtitleTrack");
+               ghb_settings_set_string(settings, "SubtitleLanguage", lang);
+               subtitle_list_refresh_selected(ud);
+               ghb_live_reset(ud);
        }
-       for (ii = 0; ii < count; ii++)
+       ghb_live_reset(ud);
+}
+
+G_MODULE_EXPORT void
+srt_changed_cb(GtkWidget *widget, signal_user_data_t *ud)
+{
+       GValue *settings;
+
+       g_debug("srt_changed_cb ()");
+       ghb_check_dependency(ud, widget, NULL);
+       ghb_widget_to_setting(ud->settings, widget);
+       settings = ghb_selected_subtitle_settings(ud);
+       if (settings != NULL)
        {
-               settings = ghb_array_get_nth(subtitle_list, ii);
-               track = ghb_settings_combo_int(settings, "SubtitleTrack");
+               ghb_widget_to_setting(settings, widget);
+               subtitle_list_refresh_selected(ud);
 
-               if (!mustBurn(ud, track))
-                       continue;
+               ghb_live_reset(ud);
+       }
+}
 
-               enabled = ghb_settings_get_boolean(settings, "SubtitleEnabled");
-               burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
+G_MODULE_EXPORT void
+srt_file_changed_cb(GtkWidget *widget, signal_user_data_t *ud)
+{
+       GValue *settings;
 
-               tv = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "subtitle_list"));
-               g_return_if_fail(tv != NULL);
-               tm = gtk_tree_view_get_model(tv);
-               gtk_tree_model_iter_nth_child(tm, &ti, NULL, ii);
+       g_debug("srt_file_changed_cb ()");
+       ghb_check_dependency(ud, widget, NULL);
+       ghb_widget_to_setting(ud->settings, widget);
+       settings = ghb_selected_subtitle_settings(ud);
+       if (settings != NULL)
+       {
+               gchar *filename, *dirname;
+
+               ghb_widget_to_setting(settings, widget);
+               subtitle_list_refresh_selected(ud);
 
-               if (!found)
+               ghb_live_reset(ud);
+
+               filename = ghb_settings_get_string(settings, "SrtFile");
+               if (g_file_test(filename, G_FILE_TEST_IS_DIR))
                {
-                       if (enabled && !burned)
-                       {
-                               ghb_settings_set_boolean(settings, "SubtitleBurned", TRUE);
-                               gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 3, TRUE, -1);
-                               found = TRUE;
-                       }
+                       ghb_settings_set_string(ud->settings, "SrtDir", filename);
                }
                else
                {
-                       if (enabled && !burned)
-                       {
-                               ghb_settings_set_boolean(settings, "SubtitleEnabled", FALSE);
-                               gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 0, FALSE, -1);
-                       }
+                       dirname = g_path_get_dirname(filename);
+                       ghb_settings_set_string(ud->settings, "SrtDir", dirname);
+                       g_free(dirname);
                }
+               ghb_pref_save(ud->settings, "SrtDir");
+               g_free(filename);
+       }
+}
+
+G_MODULE_EXPORT void
+srt_lang_changed_cb(GtkWidget *widget, signal_user_data_t *ud)
+{
+       GValue *settings;
+
+       g_debug("srt_lang_changed_cb ()");
+       ghb_check_dependency(ud, widget, NULL);
+       ghb_widget_to_setting(ud->settings, widget);
+       settings = ghb_selected_subtitle_settings(ud);
+       if (settings != NULL)
+       {
+               const gchar *lang;
+
+               ghb_widget_to_setting(settings, widget);
+               subtitle_list_refresh_selected(ud);
+
+               ghb_live_reset(ud);
+
+               lang = ghb_settings_combo_option(settings, "SrtLanguage");
+               ghb_settings_set_string(settings, "SubtitleTrackDescription", lang);
        }
 }
 
@@ -578,17 +899,18 @@ ghb_clear_subtitle_list(signal_user_data_t *ud)
 static void
 add_to_subtitle_list(
        signal_user_data_t *ud, 
-       GValue *settings, 
-       gboolean can_delete)
+       GValue *settings)
 {
        GtkTreeView *treeview;
        GtkTreeIter iter;
        GtkListStore *store;
        GtkTreeSelection *selection;
        const gchar *track, *source;
-       gboolean forced, burned, enabled;
+       gboolean forced, burned, def;
        gchar *s_track;
-       gint i_track;
+       gint i_source;
+       gboolean allow_force = FALSE;
+       gboolean allow_burn = FALSE;
        
        g_debug("add_to_subtitle_list ()");
        treeview = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "subtitle_list"));
@@ -596,30 +918,100 @@ add_to_subtitle_list(
        store = GTK_LIST_STORE(gtk_tree_view_get_model(treeview));
 
        track = ghb_settings_combo_option(settings, "SubtitleTrack");
-       enabled = ghb_settings_get_boolean(settings, "SubtitleEnabled");
        forced = ghb_settings_get_boolean(settings, "SubtitleForced");
        burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
+       def = ghb_settings_get_boolean(settings, "SubtitleDefaultTrack");
 
        s_track = ghb_settings_get_string(settings, "SubtitleTrack");
-       i_track = ghb_settings_get_int(settings, "SubtitleTrack");
-       source = ghb_subtitle_track_source_name(ud, i_track);
+       i_source = ghb_settings_get_int(settings, "SubtitleSource");
+       source = ghb_subtitle_source_name(i_source);
+
+       if (canBurn(i_source))
+               allow_burn = TRUE;
+       if (canForce(i_source))
+               allow_force = TRUE;
 
        gtk_list_store_append(store, &iter);
        gtk_list_store_set(store, &iter, 
                // These are displayed in list
-               0, enabled,
-               1, track,
-               2, forced,
-               3, burned,
-               4, source,
+               0, track,
+               1, forced,
+               2, burned,
+               3, def,
                // These are used to set combo box values when a list item is selected
                5, s_track,
-               6, can_delete,
+               6, i_source,
+               7, allow_force,
+               8, allow_burn,
+               9, FALSE,
                -1);
        gtk_tree_selection_select_iter(selection, &iter);
        g_free(s_track);
-       if (burned)
-               ghb_subtitle_exclusive_burn(ud, i_track);
+}
+
+static void
+add_to_srt_list(
+       signal_user_data_t *ud, 
+       GValue *settings)
+{
+       GtkTreeView *treeview;
+       GtkTreeIter iter;
+       GtkListStore *store;
+       GtkTreeSelection *selection;
+       const gchar *lang;
+       gboolean forced, burned, def;
+       gchar *filename, *code, *track, *source;
+       gint i_source, offset;
+       
+       g_debug("add_to_srt_list ()");
+       treeview = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "subtitle_list"));
+       selection = gtk_tree_view_get_selection (treeview);
+       store = GTK_LIST_STORE(gtk_tree_view_get_model(treeview));
+
+       lang = ghb_settings_combo_option(settings, "SrtLanguage");
+       code = ghb_settings_get_string(settings, "SrtCodeset");
+       forced = FALSE;
+       burned = FALSE;
+       def = ghb_settings_get_boolean(settings, "SubtitleDefaultTrack");
+
+       filename = ghb_settings_get_string(settings, "SrtFile");
+       if (g_file_test(filename, G_FILE_TEST_IS_REGULAR))
+       {
+               gchar *basename;
+
+               basename = g_path_get_basename(filename);
+               track = g_strdup_printf("%s (%s)(SRT)(%s)", lang, code, basename);
+               source = g_strdup_printf("SRT (%s)", basename);
+               g_free(basename);
+       }
+       else
+       {
+               track = g_strdup_printf("%s (%s)(SRT)", lang, code);
+               source = g_strdup_printf("SRT (none)");
+       }
+       i_source = SRTSUB;
+       offset = ghb_settings_get_int(settings, "SrtOffset");
+
+       gtk_list_store_append(store, &iter);
+       gtk_list_store_set(store, &iter, 
+               // These are displayed in list
+               0, track,
+               1, forced,
+               2, burned,
+               3, def,
+               4, offset,
+               // These are used to set combo box values when a list item is selected
+               5, filename,
+               6, i_source,
+               7, FALSE,
+               8, FALSE,
+               9, TRUE,
+               -1);
+       gtk_tree_selection_select_iter(selection, &iter);
+       g_free(code);
+       g_free(track);
+       g_free(filename);
+       g_free(source);
 }
 
 G_MODULE_EXPORT void
@@ -632,62 +1024,178 @@ subtitle_list_selection_changed_cb(GtkTreeSelection *selection, signal_user_data
        g_debug("subtitle_list_selection_changed_cb ()");
        if (gtk_tree_selection_get_selected(selection, &store, &iter))
        {
-               const gchar *track;
-               gboolean can_delete;
+               gint source;
+               GtkTreePath *treepath;
+               gint *indices, row;
+               GValue *subtitle_list, *settings;
+
+               treepath = gtk_tree_model_get_path (store, &iter);
+               indices = gtk_tree_path_get_indices (treepath);
+               row = indices[0];
+               gtk_tree_path_free(treepath);
+
+               subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
+               if (row >= ghb_array_len(subtitle_list))
+                       return;
 
-               gtk_tree_model_get(store, &iter, 5, &track, 6, &can_delete, -1);
-               ghb_settings_set_string(ud->settings, "SubtitleTrack", track);
+               settings = ghb_array_get_nth(subtitle_list, row);
 
-               if (can_delete)
+               source = ghb_settings_get_int(settings, "SubtitleSource");
+               if (source == SRTSUB)
                {
-                       widget = GHB_WIDGET (ud->builder, "subtitle_remove");
-                       gtk_widget_set_sensitive(widget, TRUE);
+                       gchar *str;
+                       gint offset;
+
+                       str = ghb_settings_get_string(settings, "SrtLanguage");
+                       ghb_ui_update(ud, "SrtLanguage", ghb_string_value(str));
+                       g_free(str);
+
+                       str = ghb_settings_get_string(settings, "SrtCodeset");
+                       ghb_ui_update(ud, "SrtCodeset", ghb_string_value(str));
+                       g_free(str);
+
+                       str = ghb_settings_get_string(settings, "SrtFile");
+                       ghb_ui_update(ud, "SrtFile", ghb_string_value(str));
+                       g_free(str);
+
+                       offset = ghb_settings_get_int(settings, "SrtOffset");
+                       ghb_ui_update(ud, "SrtOffset", ghb_int_value(offset));
+
+                       widget = GHB_WIDGET(ud->builder, "subtitle_track_label");
+                       gtk_widget_hide(widget);
+                       widget = GHB_WIDGET(ud->builder, "SubtitleTrack");
+                       gtk_widget_hide(widget);
+                       widget = GHB_WIDGET(ud->builder, "srt_lang_label");
+                       gtk_widget_show(widget);
+                       widget = GHB_WIDGET(ud->builder, "srt_code_label");
+                       gtk_widget_show(widget);
+                       widget = GHB_WIDGET(ud->builder, "srt_file_label");
+                       gtk_widget_show(widget);
+                       widget = GHB_WIDGET(ud->builder, "srt_offset_label");
+                       gtk_widget_show(widget);
+                       widget = GHB_WIDGET(ud->builder, "SrtLanguage");
+                       gtk_widget_show(widget);
+                       widget = GHB_WIDGET(ud->builder, "SrtCodeset");
+                       gtk_widget_show(widget);
+                       widget = GHB_WIDGET(ud->builder, "SrtFile");
+                       gtk_widget_show(widget);
+                       widget = GHB_WIDGET(ud->builder, "SrtOffset");
+                       gtk_widget_show(widget);
                }
                else
                {
-                       widget = GHB_WIDGET (ud->builder, "subtitle_remove");
-                       gtk_widget_set_sensitive(widget, FALSE);
+                       gchar *track;
+
+                       track = ghb_settings_get_string(settings, "SubtitleTrack");
+                       ghb_ui_update(ud, "SubtitleTrack", ghb_string_value(track));
+                       g_free(track);
+
+                       widget = GHB_WIDGET(ud->builder, "srt_lang_label");
+                       gtk_widget_hide(widget);
+                       widget = GHB_WIDGET(ud->builder, "srt_code_label");
+                       gtk_widget_hide(widget);
+                       widget = GHB_WIDGET(ud->builder, "srt_file_label");
+                       gtk_widget_hide(widget);
+                       widget = GHB_WIDGET(ud->builder, "srt_offset_label");
+                       gtk_widget_hide(widget);
+                       widget = GHB_WIDGET(ud->builder, "SrtLanguage");
+                       gtk_widget_hide(widget);
+                       widget = GHB_WIDGET(ud->builder, "SrtCodeset");
+                       gtk_widget_hide(widget);
+                       widget = GHB_WIDGET(ud->builder, "SrtFile");
+                       gtk_widget_hide(widget);
+                       widget = GHB_WIDGET(ud->builder, "SrtOffset");
+                       gtk_widget_hide(widget);
+                       widget = GHB_WIDGET(ud->builder, "subtitle_track_label");
+                       gtk_widget_show(widget);
+                       widget = GHB_WIDGET(ud->builder, "SubtitleTrack");
+                       gtk_widget_show(widget);
                }
+               widget = GHB_WIDGET (ud->builder, "subtitle_remove");
+               gtk_widget_set_sensitive(widget, TRUE);
+       }
+       else
+       {
+               widget = GHB_WIDGET(ud->builder, "srt_lang_label");
+               gtk_widget_hide(widget);
+               widget = GHB_WIDGET(ud->builder, "srt_code_label");
+               gtk_widget_hide(widget);
+               widget = GHB_WIDGET(ud->builder, "srt_file_label");
+               gtk_widget_hide(widget);
+               widget = GHB_WIDGET(ud->builder, "srt_offset_label");
+               gtk_widget_hide(widget);
+               widget = GHB_WIDGET(ud->builder, "SrtLanguage");
+               gtk_widget_hide(widget);
+               widget = GHB_WIDGET(ud->builder, "SrtCodeset");
+               gtk_widget_hide(widget);
+               widget = GHB_WIDGET(ud->builder, "SrtFile");
+               gtk_widget_hide(widget);
+               widget = GHB_WIDGET(ud->builder, "SrtOffset");
+               gtk_widget_hide(widget);
+               widget = GHB_WIDGET(ud->builder, "subtitle_track_label");
+               gtk_widget_show(widget);
+               widget = GHB_WIDGET(ud->builder, "SubtitleTrack");
+               gtk_widget_show(widget);
        }
 }
 
 G_MODULE_EXPORT void
+srt_add_clicked_cb(GtkWidget *xwidget, signal_user_data_t *ud)
+{
+       // Add the current subtitle settings to the list.
+       GValue *settings;
+       gchar *dir, *filename, *lang;
+       
+       g_debug("subtitle_add_clicked_cb ()");
+
+       settings = ghb_dict_value_new();
+       ghb_settings_set_int(settings, "SubtitleSource", SRTSUB);
+       lang = ghb_settings_get_string(ud->settings, "PreferredLanguage");
+       ghb_settings_set_string(settings, "SrtLanguage", lang);
+       g_free(lang);
+       ghb_settings_set_string(settings, "SrtCodeset", "UTF-8");
+
+       dir = ghb_settings_get_string(ud->settings, "SrtDir");
+       filename = g_strdup_printf("%s/none", dir);
+       ghb_settings_set_string(settings, "SrtFile", filename);
+       g_free(dir);
+       g_free(filename);
+
+       ghb_settings_set_int(settings, "SrtOffset", 0);
+       ghb_settings_take_value(settings, "SubtitleDefaultTrack", 
+                                                       ghb_boolean_value_new(FALSE));
+
+       ghb_add_srt(ud, settings);
+}
+
+G_MODULE_EXPORT void
 subtitle_add_clicked_cb(GtkWidget *xwidget, signal_user_data_t *ud)
 {
        // Add the current subtitle settings to the list.
        GValue *settings;
-       gint count;
-       GValue *subtitle_list;
        gboolean burned = FALSE;
-       gint track;
+       gint track, source;
        
        g_debug("subtitle_add_clicked_cb ()");
-       track = ghb_pick_subtitle_track(ud);
-       if (track < 0)
-               return;
 
-       if (mustBurn(ud, track))
+       track = ghb_settings_get_int(ud->settings, "SubtitleTrack");
+
+       settings = ghb_dict_value_new();
+       ghb_settings_set_int(settings, "SubtitleTrack", track);
+       source = ghb_subtitle_track_source(ud, track);
+       ghb_settings_set_int(settings, "SubtitleSource", source);
+       if (mustBurn(ud, settings))
        {
                burned = TRUE;
        }
-       settings = ghb_dict_value_new();
-       // Only allow up to 8 subtitle entries
-       ghb_settings_set_int(settings, "SubtitleTrack", track);
-       ghb_settings_set_boolean(settings, "SubtitleEnabled", TRUE);
        ghb_settings_take_value(settings, "SubtitleForced", 
                                                        ghb_boolean_value_new(FALSE));
        ghb_settings_take_value(settings, "SubtitleBurned", 
                                                        ghb_boolean_value_new(burned));
+       ghb_settings_take_value(settings, "SubtitleDefaultTrack", 
+                                                       ghb_boolean_value_new(FALSE));
 
-       subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
-       if (subtitle_list == NULL)
-       {
-               subtitle_list = ghb_array_value_new(8);
-               ghb_settings_set_value(ud->settings, "subtitle_list", subtitle_list);
-       }
-       ghb_array_append(subtitle_list, settings);
-       add_to_subtitle_list(ud, settings, TRUE);
-       count = ghb_array_len(subtitle_list);
+       ghb_add_subtitle(ud, settings);
 }
 
 G_MODULE_EXPORT void
@@ -731,56 +1239,58 @@ subtitle_remove_clicked_cb(GtkWidget *widget, signal_user_data_t *ud)
                if (row < 0) return;
                widget = GHB_WIDGET (ud->builder, "subtitle_add");
                gtk_widget_set_sensitive(widget, TRUE);
+               widget = GHB_WIDGET (ud->builder, "srt_add");
+               gtk_widget_set_sensitive(widget, TRUE);
                subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
                if (row >= ghb_array_len(subtitle_list))
                        return;
                GValue *old = ghb_array_get_nth(subtitle_list, row);
                ghb_value_free(old);
                ghb_array_remove(subtitle_list, row);
+               ghb_live_reset(ud);
        }
 }
 
 void
-ghb_set_subtitle(signal_user_data_t *ud, gint track, GValue *settings)
+ghb_subtitle_prune(signal_user_data_t *ud)
 {
        GtkTreeView  *tv;
        GtkTreeModel *tm;
        GtkTreeIter   ti;
-       GValue *slist;
-       GValue *subtitle;
-       gint count, ii, tt;
-       gboolean forced, burned, enabled;
-       
-       g_debug("ghb_set_subtitle");
-       slist = ghb_settings_get_value(ud->settings, "subtitle_list");
+       GValue *subtitle_list;
+       gint count, ii;
+       gint first_track = 0, one_burned = 0;
 
-       count = ghb_array_len(slist);
-       for (ii = 0; ii < count; ii++)
-       {
-               subtitle = ghb_array_get_nth(slist, ii);
-               tt = ghb_settings_get_int(subtitle, "SubtitleTrack");
-               if (track != tt)
-                       continue;
-               
-               enabled = ghb_settings_get_boolean(settings, "SubtitleEnabled");
-               forced = ghb_settings_get_boolean(settings, "SubtitleForced");
-               burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
+       subtitle_list = ghb_settings_get_value(ud->settings, "subtitle_list");
+       if (subtitle_list == NULL)
+               return;
+       count = ghb_array_len(subtitle_list);
 
-               tv = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "subtitle_list"));
-               g_return_if_fail(tv != NULL);
-               tm = gtk_tree_view_get_model(tv);
-               gtk_tree_model_iter_nth_child(tm, &ti, NULL, ii);
+       tv = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "subtitle_list"));
+       g_return_if_fail(tv != NULL);
+       tm = gtk_tree_view_get_model(tv);
+       for (ii = count-1; ii >= 0; ii--)
+       {
+               gboolean burned;
+               GValue *settings;
 
-               ghb_settings_set_boolean(subtitle, "SubtitleEnabled", enabled);
-               ghb_settings_set_boolean(subtitle, "SubtitleForced", forced);
-               ghb_settings_set_boolean(subtitle, "SubtitleBurned", burned);
-               gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 
-                       0, enabled, 
-                       2, forced, 
-                       3, burned, 
-                       6, FALSE, 
-                       -1);
-               break;
+               settings = ghb_array_get_nth(subtitle_list, ii);
+               burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
+               if (!burned && mustBurn(ud, settings))
+               {
+                       gtk_tree_model_iter_nth_child(tm, &ti, NULL, ii);
+                       gtk_list_store_remove (GTK_LIST_STORE(tm), &ti);
+                       ghb_array_remove(subtitle_list, ii);
+               }
+               if (burned)
+               {
+                       first_track = ii;
+                       one_burned++;
+               }
+       }
+       if (one_burned)
+       {
+               ghb_subtitle_exclusive_burn(ud, first_track);
        }
 }
 
@@ -789,7 +1299,7 @@ ghb_reset_subtitles(signal_user_data_t *ud, GValue *settings)
 {
        GValue *slist;
        GValue *subtitle;
-       gint count, ii, track;
+       gint count, ii;
        gint titleindex;
        
        g_debug("ghb_reset_subtitles");
@@ -798,16 +1308,18 @@ ghb_reset_subtitles(signal_user_data_t *ud, GValue *settings)
        if (titleindex < 0)
                return;
 
-       ghb_add_all_subtitles(ud, titleindex);
        slist = ghb_settings_get_value(settings, "subtitle_list");
-
        count = ghb_array_len(slist);
        for (ii = 0; ii < count; ii++)
        {
-               subtitle = ghb_array_get_nth(slist, ii);
-               track = ghb_settings_get_int(subtitle, "SubtitleTrack");
+               int source;
 
-               ghb_set_subtitle(ud, track, subtitle);
+               subtitle = ghb_value_dup(ghb_array_get_nth(slist, ii));
+               source = ghb_settings_get_int(subtitle, "SubtitleSource");
+               if (source == SRTSUB)
+                       ghb_add_srt(ud, subtitle);
+               else
+                       ghb_add_subtitle(ud, subtitle);
        }
 }