summaryrefslogtreecommitdiffstats
path: root/gtk
diff options
context:
space:
mode:
Diffstat (limited to 'gtk')
-rw-r--r--gtk/src/callbacks.c4
-rw-r--r--gtk/src/ghb.ui21
-rw-r--r--gtk/src/hb-backend.c150
-rw-r--r--gtk/src/hb-backend.h1
-rw-r--r--gtk/src/internal_defaults.xml2
-rw-r--r--gtk/src/main.c34
-rw-r--r--gtk/src/presets.c7
-rw-r--r--gtk/src/settings.h1
-rw-r--r--gtk/src/subtitlehandler.c433
-rw-r--r--gtk/src/subtitlehandler.h4
10 files changed, 218 insertions, 439 deletions
diff --git a/gtk/src/callbacks.c b/gtk/src/callbacks.c
index 657d9578f..9e449e34f 100644
--- a/gtk/src/callbacks.c
+++ b/gtk/src/callbacks.c
@@ -1120,7 +1120,7 @@ container_changed_cb(GtkWidget *widget, signal_user_data_t *ud)
}
g_free(container);
}
- ghb_subtitle_adjust_burn(ud);
+ ghb_subtitle_prune(ud);
}
static gchar*
@@ -1258,7 +1258,7 @@ title_changed_cb(GtkWidget *widget, signal_user_data_t *ud)
titleindex = ghb_settings_combo_int(ud->settings, "title");
ghb_update_ui_combo_box (ud, "AudioTrack", titleindex, FALSE);
- ghb_subtitle_track_model(ud, titleindex);
+ ghb_update_ui_combo_box (ud, "SubtitleTrack", titleindex, FALSE);
ghb_set_pref_subtitle(titleindex, ud);
if (ghb_get_title_info (&tinfo, titleindex))
diff --git a/gtk/src/ghb.ui b/gtk/src/ghb.ui
index e52dac0d5..e182d0a0a 100644
--- a/gtk/src/ghb.ui
+++ b/gtk/src/ghb.ui
@@ -2253,6 +2253,27 @@
<property name="position">1</property>
</packing>
</child>
+ <child>
+ <object class="GtkAlignment" id="alignment48">
+ <property name="visible">True</property>
+ <property name="events">GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK | GDK_BUTTON_PRESS_MASK | GDK_BUTTON_RELEASE_MASK</property>
+ <property name="xscale">0</property>
+ <property name="yscale">0</property>
+ <property name="xalign">1</property>
+ <child>
+ <object class="GtkComboBox" id="SubtitleTrack">
+ <property name="width_request">215</property>
+ <property name="visible">True</property>
+ <property name="events">GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK | GDK_BUTTON_PRESS_MASK | GDK_BUTTON_RELEASE_MASK</property>
+ <signal handler="subtitle_track_changed_cb" name="changed"/>
+ </object>
+ </child>
+ </object>
+ <packing>
+ <property name="expand">True</property>
+ <property name="position">2</property>
+ </packing>
+ </child>
</object>
<packing>
<property name="expand">False</property>
diff --git a/gtk/src/hb-backend.c b/gtk/src/hb-backend.c
index fa38a20ce..8593b02e7 100644
--- a/gtk/src/hb-backend.c
+++ b/gtk/src/hb-backend.c
@@ -1681,7 +1681,7 @@ audio_track_opts_set(GtkBuilder *builder, const gchar *name, gint titleindex)
}
void
-ghb_subtitle_track_model(signal_user_data_t *ud, gint titleindex)
+subtitle_track_opts_set(GtkBuilder *builder, const gchar *name, gint titleindex)
{
GtkTreeIter iter;
GtkListStore *store;
@@ -1689,8 +1689,11 @@ ghb_subtitle_track_model(signal_user_data_t *ud, gint titleindex)
hb_title_t * title = NULL;
hb_subtitle_t * subtitle;
gint ii, count = 0;
+ static char ** options = NULL;
- g_debug("ghb_subtitle_track_model ()\n");
+ g_debug("subtitle_track_opts_set ()\n");
+ store = get_combo_box_store(builder, name);
+ gtk_list_store_clear(store);
if (h_scan != NULL)
{
list = hb_get_titles( h_scan );
@@ -1712,56 +1715,44 @@ ghb_subtitle_track_model(signal_user_data_t *ud, gint titleindex)
subtitle_opts.count = LANG_TABLE_SIZE+1;
subtitle_opts.map = g_malloc((LANG_TABLE_SIZE+1)*sizeof(options_map_t));
}
- if (ud->subtitle_track_model == NULL)
- {
- // Store contains:
- // 1 - String to display
- // 2 - bool indicating whether the entry is selectable (grey or not)
- // 3 - String that is used for presets
- // 4 - Int value determined by backend
- // 5 - String value determined by backend
- store = gtk_list_store_new(5, G_TYPE_STRING, G_TYPE_BOOLEAN,
- G_TYPE_STRING, G_TYPE_DOUBLE, G_TYPE_STRING);
- ud->subtitle_track_model = store;
- }
- else
- {
- store = ud->subtitle_track_model;
- gtk_list_store_clear(store);
- }
gtk_list_store_append(store, &iter);
gtk_list_store_set(store, &iter,
- 0, "Autoselect",
+ 0, "Foreign Audio Search",
1, TRUE,
2, "-1",
3, -1.0,
4, "auto",
-1);
- subtitle_opts.map[0].option = "Same as audio";
+ subtitle_opts.map[0].option = "Foreign Audio Search";
subtitle_opts.map[0].shortOpt = "-1";
subtitle_opts.map[0].ivalue = -1;
subtitle_opts.map[0].svalue = "auto";
if (count > 0)
{
+ if (options != NULL)
+ g_strfreev(options);
+ options = g_malloc((count+1)*sizeof(gchar*));
index_str_init(count-1);
for (ii = 0; ii < count; ii++)
{
subtitle = (hb_subtitle_t *)hb_list_item(title->list_subtitle, ii);
// Skip subtitles that must be burned if there is already
// a burned subtitle in the list
- subtitle_opts.map[ii+1].option = subtitle->lang;
+ options[ii] = g_strdup_printf("%d - %s", ii+1, subtitle->lang);
+ subtitle_opts.map[ii+1].option = options[ii];
subtitle_opts.map[ii+1].shortOpt = index_str[ii];
subtitle_opts.map[ii+1].ivalue = ii;
subtitle_opts.map[ii+1].svalue = subtitle->iso639_2;
gtk_list_store_append(store, &iter);
gtk_list_store_set(store, &iter,
- 0, subtitle->lang,
+ 0, options[ii],
1, TRUE,
2, index_str[ii],
3, (gdouble)ii,
4, subtitle->iso639_2,
-1);
}
+ options[count] = NULL;
}
else
{
@@ -1968,50 +1959,6 @@ ghb_find_audio_track(
return track;
}
-void
-ghb_add_all_subtitles(signal_user_data_t *ud, gint titleindex)
-{
- hb_list_t * list;
- hb_title_t * title;
- hb_subtitle_t * subtitle;
- gint ii;
- gint count = 0;
- GValue *subdict;
-
- g_debug("ghb_add_all_subtitles ()\n");
- if (h_scan == NULL)
- return;
- list = hb_get_titles( h_scan );
- title = (hb_title_t*)hb_list_item( list, titleindex );
- if (title == NULL)
- return;
-
- // Add special auto selection track
- subdict = ghb_dict_value_new();
- ghb_settings_set_boolean(subdict, "SubtitleEnabled", FALSE);
- ghb_settings_set_int(subdict, "SubtitleTrack", -1);
- ghb_settings_set_boolean(subdict, "SubtitleForced", FALSE);
- ghb_settings_set_boolean(subdict, "SubtitleBurned", FALSE);
- ghb_settings_set_boolean(subdict, "SubtitleDefaultTrack", FALSE);
- ghb_settings_set_string(subdict, "SubtitleLanguage", "auto");
- ghb_add_subtitle(ud, subdict, FALSE);
-
- count = hb_list_count( title->list_subtitle );
- for (ii = 0; ii < count; ii++)
- {
- subtitle = (hb_subtitle_t*)hb_list_item( title->list_subtitle, ii );
- subdict = ghb_dict_value_new();
- ghb_settings_set_boolean(subdict, "SubtitleEnabled", FALSE);
- ghb_settings_set_int(subdict, "SubtitleTrack", ii);
- ghb_settings_set_boolean(subdict, "SubtitleForced", FALSE);
- ghb_settings_set_boolean(subdict, "SubtitleBurned", FALSE);
- ghb_settings_set_boolean(subdict, "SubtitleDefaultTrack", FALSE);
- ghb_settings_set_string(subdict, "SubtitleLanguage",
- subtitle->iso639_2);
- ghb_add_subtitle(ud, subdict, FALSE);
- }
-}
-
gint
ghb_find_pref_subtitle_track(const gchar *lang)
{
@@ -2072,38 +2019,6 @@ ghb_find_subtitle_track(
return -2;
}
-gint
-ghb_pick_subtitle_track(signal_user_data_t *ud)
-{
- gint ii, count, track, candidate, first;
- GValue *settings, *subtitle_list;
-
- first = candidate = ghb_settings_combo_int(ud->settings, "SubtitleTrack");
- 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);
- track = ghb_settings_combo_int(settings, "SubtitleTrack");
- if (candidate == track)
- {
- // Already in use, pick another
- candidate++;
- if (candidate >= subtitle_opts.count-1)
- {
- candidate = 0;
- }
- if (candidate == first)
- {
- candidate = -1;
- break;
- }
- ii = -1;
- }
- }
- return candidate;
-}
-
static void
generic_opts_set(GtkBuilder *builder, const gchar *name, combo_opts_t *opts)
{
@@ -2275,6 +2190,7 @@ ghb_update_ui_combo_box(
language_opts_set(ud->builder, "SourceAudioLang");
title_opts_set(ud->builder, "title");
audio_track_opts_set(ud->builder, "AudioTrack", user_data);
+ subtitle_track_opts_set(ud->builder, "SubtitleTrack", user_data);
generic_opts_set(ud->builder, "VideoQualityGranularity", &vqual_granularity_opts);
generic_opts_set(ud->builder, "PicturePAR", &par_opts);
generic_opts_set(ud->builder, "PictureModulus", &alignment_opts);
@@ -2307,6 +2223,8 @@ ghb_update_ui_combo_box(
language_opts_set(ud->builder, "SourceAudioLang");
else if (strcmp(name, "title") == 0)
title_opts_set(ud->builder, "title");
+ else if (strcmp(name, "SubtitleTrack") == 0)
+ subtitle_track_opts_set(ud->builder, "SubtitleTrack", user_data);
else if (strcmp(name, "AudioTrack") == 0)
audio_track_opts_set(ud->builder, "AudioTrack", user_data);
else
@@ -3405,7 +3323,7 @@ ghb_validate_subtitles(signal_user_data_t *ud)
const GValue *slist, *settings;
gint count, ii, track, source;
- gboolean burned, enabled;
+ gboolean burned;
slist = ghb_settings_get_value(ud->settings, "subtitle_list");
count = ghb_array_len(slist);
@@ -3413,10 +3331,9 @@ ghb_validate_subtitles(signal_user_data_t *ud)
{
settings = ghb_array_get_nth(slist, ii);
track = ghb_settings_combo_int(settings, "SubtitleTrack");
- enabled = ghb_settings_get_boolean(settings, "SubtitleEnabled");
burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
source = ghb_subtitle_track_source(ud, track);
- if (enabled && !burned && mux == HB_MUX_MP4 && source == VOBSUB)
+ if (!burned && mux == HB_MUX_MP4 && source == VOBSUB)
{
// MP4 can only handle burned vobsubs. make sure there isn't
// already something burned in the list
@@ -4041,15 +3958,10 @@ add_job(hb_handle_t *h, GValue *js, gint unique_id, gint titleindex)
for (ii = 0; ii < count; ii++)
{
GValue *ssettings;
- gboolean enabled, force, burned, def, one_burned = FALSE;
+ gboolean force, burned, def, one_burned = FALSE;
ssettings = ghb_array_get_nth(subtitle_list, ii);
- enabled = ghb_settings_get_boolean(ssettings, "SubtitleEnabled");
-
- if (!enabled)
- continue;
-
subtitle = ghb_settings_get_int(ssettings, "SubtitleTrack");
force = ghb_settings_get_boolean(ssettings, "SubtitleForced");
burned = ghb_settings_get_boolean(ssettings, "SubtitleBurned");
@@ -4082,14 +3994,16 @@ add_job(hb_handle_t *h, GValue *js, gint unique_id, gint titleindex)
else if (subtitle >= 0)
{
hb_subtitle_t * subt;
+ hb_subtitle_config_t sub_config;
subt = (hb_subtitle_t *)hb_list_item(title->list_subtitle, subtitle);
+ sub_config = subt->config;
if (subt != NULL)
{
if (!burned && job->mux == HB_MUX_MKV &&
subt->format == PICTURESUB)
{
- subt->config.dest = PASSTHRUSUB;
+ sub_config.dest = PASSTHRUSUB;
}
else if (!burned && job->mux == HB_MUX_MP4 &&
subt->format == PICTURESUB)
@@ -4104,9 +4018,9 @@ add_job(hb_handle_t *h, GValue *js, gint unique_id, gint titleindex)
continue;
one_burned = TRUE;
}
- subt->config.force = force;
- subt->config.default_track = def;
- hb_list_add(job->list_subtitle, subt);
+ sub_config.force = force;
+ sub_config.default_track = def;
+ hb_subtitle_add( job, &sub_config, subtitle );
}
}
}
@@ -4210,8 +4124,7 @@ add_job(hb_handle_t *h, GValue *js, gint unique_id, gint titleindex)
// g_free(job->x264opts);
}
- // First remove any audios that are already in the list
- // This happens if you are encoding the same title a second time.
+ // clean up audio list
gint num_audio_tracks = hb_list_count(job->list_audio);
for(ii = 0; ii < num_audio_tracks; ii++)
{
@@ -4220,6 +4133,15 @@ add_job(hb_handle_t *h, GValue *js, gint unique_id, gint titleindex)
free(audio);
}
+ // clean up subtitle list
+ gint num_subtitle_tracks = hb_list_count(job->list_subtitle);
+ for(ii = 0; ii < num_subtitle_tracks; ii++)
+ {
+ hb_subtitle_t *subtitle = hb_list_item(job->list_subtitle, 0);
+ hb_list_rem(job->list_subtitle, subtitle);
+ free(subtitle);
+ }
+
if (detel_str) g_free(detel_str);
if (decomb_str) g_free(decomb_str);
if (deint_str) g_free(deint_str);
diff --git a/gtk/src/hb-backend.h b/gtk/src/hb-backend.h
index 4b5589734..23d09639a 100644
--- a/gtk/src/hb-backend.h
+++ b/gtk/src/hb-backend.h
@@ -154,7 +154,6 @@ gchar* ghb_dvd_volname(const gchar *device);
gint ghb_get_title_number(gint titleindex);
gint ghb_subtitle_track_source(signal_user_data_t *ud, gint track);
const char* ghb_subtitle_track_source_name(signal_user_data_t *ud, gint track);
-void ghb_subtitle_track_model(signal_user_data_t *ud, gint titleindex);
gboolean ghb_validate_vquality(GValue *settings);
gboolean ghb_validate_audio(signal_user_data_t *ud);
diff --git a/gtk/src/internal_defaults.xml b/gtk/src/internal_defaults.xml
index a216f73aa..64de850b8 100644
--- a/gtk/src/internal_defaults.xml
+++ b/gtk/src/internal_defaults.xml
@@ -269,8 +269,6 @@
<key>SubtitleList</key>
<array>
<dict>
- <key>SubtitleEnabled</key>
- <false />
<key>SubtitleLanguage</key>
<string>und</string>
<key>SubtitleForced</key>
diff --git a/gtk/src/main.c b/gtk/src/main.c
index 7331c0b7e..4c7750968 100644
--- a/gtk/src/main.c
+++ b/gtk/src/main.c
@@ -381,11 +381,9 @@ bind_audio_tree_model (signal_user_data_t *ud)
}
extern G_MODULE_EXPORT void subtitle_list_selection_changed_cb(void);
-extern G_MODULE_EXPORT void subtitle_enable_toggled_cb(void);
extern G_MODULE_EXPORT void subtitle_forced_toggled_cb(void);
extern G_MODULE_EXPORT void subtitle_burned_toggled_cb(void);
extern G_MODULE_EXPORT void subtitle_default_toggled_cb(void);
-extern G_MODULE_EXPORT void subtitle_track_changed_cb(void);
// Create and bind the tree model to the tree view for the subtitle track list
// Also, connect up the signal that lets us know the selection has changed
@@ -402,54 +400,44 @@ bind_subtitle_tree_model (signal_user_data_t *ud)
g_debug("bind_subtitle_tree_model ()\n");
treeview = GTK_TREE_VIEW(GHB_WIDGET (ud->builder, "subtitle_list"));
selection = gtk_tree_view_get_selection (treeview);
- // 6 columns in model. 4 are visible, the other 2 is for storing
+ // 6 columns in model. 5 are visible, the other 1 is for storing
// values that I need
- // Enable, Track, force, burn, default, type, track short, can delete
- treestore = gtk_list_store_new(8,
- G_TYPE_BOOLEAN, G_TYPE_STRING,
+ // Track, force, burn, default, type, track short
+ treestore = gtk_list_store_new(6,
+ G_TYPE_STRING,
G_TYPE_BOOLEAN, G_TYPE_BOOLEAN,
G_TYPE_BOOLEAN, G_TYPE_STRING,
- G_TYPE_STRING, G_TYPE_BOOLEAN);
+ G_TYPE_STRING);
gtk_tree_view_set_model(treeview, GTK_TREE_MODEL(treestore));
- cell = gtk_cell_renderer_toggle_new();
+ cell = gtk_cell_renderer_text_new();
column = gtk_tree_view_column_new_with_attributes(
- _("Enable"), cell, "active", 0, NULL);
- gtk_tree_view_append_column(treeview, GTK_TREE_VIEW_COLUMN(column));
- g_signal_connect(cell, "toggled", subtitle_enable_toggled_cb, ud);
-
- cell = gtk_cell_renderer_combo_new();
- ghb_subtitle_track_model(ud, -1);
- g_object_set(G_OBJECT(cell), "model", ud->subtitle_track_model,
- "text-column", 0, "editable", TRUE, "width", 200, "has-entry", FALSE, NULL);
- column = gtk_tree_view_column_new_with_attributes( _("Track"), cell,
- "text", 1, NULL);
+ _("Track"), cell, "text", 0, NULL);
gtk_tree_view_append_column(treeview, GTK_TREE_VIEW_COLUMN(column));
- g_signal_connect(cell, "changed", subtitle_track_changed_cb, ud);
cell = gtk_cell_renderer_toggle_new();
column = gtk_tree_view_column_new_with_attributes(
- _("Forced Only"), cell, "active", 2, NULL);
+ _("Forced Only"), cell, "active", 1, NULL);
gtk_tree_view_append_column(treeview, GTK_TREE_VIEW_COLUMN(column));
g_signal_connect(cell, "toggled", subtitle_forced_toggled_cb, ud);
cell = gtk_cell_renderer_toggle_new();
gtk_cell_renderer_toggle_set_radio(GTK_CELL_RENDERER_TOGGLE(cell), TRUE);
column = gtk_tree_view_column_new_with_attributes(
- _("Burned In"), cell, "active", 3, NULL);
+ _("Burned In"), cell, "active", 2, NULL);
gtk_tree_view_append_column(treeview, GTK_TREE_VIEW_COLUMN(column));
g_signal_connect(cell, "toggled", subtitle_burned_toggled_cb, ud);
cell = gtk_cell_renderer_toggle_new();
gtk_cell_renderer_toggle_set_radio(GTK_CELL_RENDERER_TOGGLE(cell), TRUE);
column = gtk_tree_view_column_new_with_attributes(
- _("Default"), cell, "active", 4, NULL);
+ _("Default"), cell, "active", 3, NULL);
gtk_tree_view_append_column(treeview, GTK_TREE_VIEW_COLUMN(column));
g_signal_connect(cell, "toggled", subtitle_default_toggled_cb, ud);
cell = gtk_cell_renderer_text_new();
column = gtk_tree_view_column_new_with_attributes(
- _("Type"), cell, "text", 5, NULL);
+ _("Type"), cell, "text", 4, NULL);
gtk_tree_view_append_column(treeview, GTK_TREE_VIEW_COLUMN(column));
diff --git a/gtk/src/presets.c b/gtk/src/presets.c
index 505ce023e..74a101b79 100644
--- a/gtk/src/presets.c
+++ b/gtk/src/presets.c
@@ -3056,11 +3056,8 @@ update_subtitle_presets(signal_user_data_t *ud)
for (ii = 0; ii < count; ii++)
{
subtitle = ghb_array_get_nth(subtitle_list, ii);
- if (ghb_settings_get_boolean(subtitle, "SubtitleEnabled"))
- {
- dict = ghb_value_dup(subtitle);
- ghb_array_append(slist, dict);
- }
+ dict = ghb_value_dup(subtitle);
+ ghb_array_append(slist, dict);
}
ghb_settings_set_value(ud->settings, "SubtitleList", slist);
}
diff --git a/gtk/src/settings.h b/gtk/src/settings.h
index 0616426ff..072baf07a 100644
--- a/gtk/src/settings.h
+++ b/gtk/src/settings.h
@@ -53,7 +53,6 @@ typedef struct
preview_t *preview;
gchar *appcast;
gint appcast_len;
- GtkListStore *subtitle_track_model;
} signal_user_data_t;
enum
diff --git a/gtk/src/subtitlehandler.c b/gtk/src/subtitlehandler.c
index c40e880e7..980e7171c 100644
--- a/gtk/src/subtitlehandler.c
+++ b/gtk/src/subtitlehandler.c
@@ -20,8 +20,7 @@
#include "preview.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);
void
free_subtitle_index_list(gpointer data)
@@ -37,7 +36,7 @@ free_subtitle_key(gpointer data)
}
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;
@@ -52,7 +51,7 @@ ghb_add_subtitle(signal_user_data_t *ud, GValue *settings, gboolean can_delete)
ghb_settings_set_value(ud->settings, "subtitle_list", subtitle_list);
}
ghb_array_append(subtitle_list, settings);
- add_to_subtitle_list(ud, settings, can_delete);
+ add_to_subtitle_list(ud, settings);
}
static void
@@ -78,7 +77,7 @@ add_all_pref_subtitles(signal_user_data_t *ud)
{
// Add to subtitle list
ghb_settings_set_int(subtitle, "SubtitleTrack", track);
- ghb_add_subtitle(ud, subtitle, TRUE);
+ ghb_add_subtitle(ud, subtitle);
}
}
}
@@ -97,6 +96,9 @@ ghb_set_pref_subtitle(gint titleindex, signal_user_data_t *ud)
g_debug("ghb_set_pref_subtitle %d", titleindex);
track_indices = g_hash_table_new_full(g_str_hash, g_str_equal,
NULL, 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,7 +106,6 @@ 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");
@@ -121,7 +122,9 @@ ghb_set_pref_subtitle(gint titleindex, signal_user_data_t *ud)
g_free(lang);
if (track >= -1)
{
- ghb_set_subtitle(ud, track, subtitle);
+ GValue *dup = ghb_value_dup(subtitle);
+ ghb_settings_set_int(dup, "SubtitleTrack", track);
+ ghb_add_subtitle(ud, dup);
}
}
g_hash_table_destroy(track_indices);
@@ -235,12 +238,7 @@ ghb_subtitle_exclusive_burn(signal_user_data_t *ud, gint 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);
+ gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 2, FALSE, -1);
}
}
}
@@ -274,61 +272,12 @@ ghb_subtitle_exclusive_default(signal_user_data_t *ud, gint track)
ghb_settings_set_boolean(settings, "SubtitleDefaultTrack", FALSE);
def = FALSE;
- gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 4, FALSE, -1);
+ gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 3, FALSE, -1);
}
}
}
G_MODULE_EXPORT void
-subtitle_enable_toggled_cb(
- GtkCellRendererToggle *cell,
- gchar *path,
- signal_user_data_t *ud)
-{
- GtkTreeView *tv;
- GtkTreeModel *tm;
- GtkTreeIter ti;
- gboolean active;
- gint row;
- GtkTreePath *tp;
- gint *indices;
- GValue *subtitle_list, *settings;
- gint track;
-
- g_debug("forced 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);
- 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);
- 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);
- }
-
- ghb_settings_set_boolean(settings, "SubtitleEnabled", active);
- gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 0, active, -1);
-}
-
-G_MODULE_EXPORT void
subtitle_forced_toggled_cb(
GtkCellRendererToggle *cell,
gchar *path,
@@ -351,7 +300,7 @@ subtitle_forced_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, 2, &active, -1);
+ gtk_tree_model_get(tm, &ti, 1, &active, -1);
active ^= 1;
// Get the row number
@@ -372,7 +321,7 @@ subtitle_forced_toggled_cb(
return;
ghb_settings_set_boolean(settings, "SubtitleForced", active);
- gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 2, active, -1);
+ gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 1, active, -1);
}
G_MODULE_EXPORT void
@@ -399,7 +348,7 @@ subtitle_burned_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, 3, &active, -1);
+ gtk_tree_model_get(tm, &ti, 2, &active, -1);
active ^= 1;
// Get the row number
@@ -424,7 +373,7 @@ subtitle_burned_toggled_cb(
ghb_settings_set_boolean(settings, "SubtitleBurned", active);
- gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 3, active, -1);
+ gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 2, active, -1);
// Unburn the rest
ghb_subtitle_exclusive_burn(ud, track);
}
@@ -453,7 +402,7 @@ subtitle_default_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, 4, &active, -1);
+ gtk_tree_model_get(tm, &ti, 3, &active, -1);
active ^= 1;
// Get the row number
@@ -471,168 +420,96 @@ subtitle_default_toggled_cb(
ghb_settings_set_boolean(settings, "SubtitleDefaultTrack", active);
- gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 4, active, -1);
+ gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 3, active, -1);
// allow only one default
ghb_subtitle_exclusive_default(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++)
- {
- settings = ghb_array_get_nth(subtitle_list, ii);
- tt = ghb_settings_combo_int(settings, "SubtitleTrack");
- if (tt == track)
- {
- return TRUE;
- }
- }
- return FALSE;
-}
-
-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;
-
- 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;
-
- 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);
-
- // 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 >= ghb_array_len(subtitle_list))
- return;
- settings = ghb_array_get_nth(subtitle_list, row);
-
- 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
+ gint row;
+ GValue *settings = NULL;
+ const GValue *subtitle_list;
+
+ 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))
{
- ghb_settings_set_boolean(settings, "SubtitleBurned", FALSE);
- }
- ghb_settings_set_string(settings, "SubtitleLanguage", lang);
+ const gchar *track, *source;
+ gboolean forced, burned, def;
+ gchar *s_track;
+ gint i_track;
+
+ // 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);
+ // find audio settings
+ 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);
- const gchar *track, *source;
- gboolean burned;
- gchar *s_track;
- gint i_track;
+ track = ghb_settings_combo_option(settings, "SubtitleTrack");
+ forced = ghb_settings_get_boolean(settings, "SubtitleForced");
+ burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
+ def = ghb_settings_get_boolean(settings, "SubtitleDefaultTrack");
- track = ghb_settings_combo_option(settings, "SubtitleTrack");
- burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
+ 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);
- 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);
+ gint i_source;
+ i_source = ghb_subtitle_track_source(ud, i_track);
+ if (i_source != VOBSUB)
+ {
+ // Force and burn only apply to VOBSUBS
+ forced = FALSE;
+ burned = FALSE;
+ ghb_settings_set_boolean(settings, "SubtitleForced", forced);
+ ghb_settings_set_boolean(settings, "SubtitleBurned", burned);
+ }
- 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,
- 5, source,
- // These are used to set combo values when a list item is selected
- 6, s_track,
+ 0, track,
+ 1, forced,
+ 2, burned,
+ 3, def,
+ 4, source,
+ // These are used to set combo box values when a list item is selected
+ 5, s_track,
-1);
- g_free(s_track);
- ghb_live_reset(ud);
-
+ g_free(s_track);
+ if (burned)
+ ghb_subtitle_exclusive_burn(ud, i_track);
+ }
}
-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++)
- {
- 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;
- }
- }
- for (ii = 0; ii < count; ii++)
+ g_debug("subtitle_track_changed_cb ()");
+ ghb_check_dependency(ud, 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");
-
- if (!mustBurn(ud, track))
- continue;
-
- enabled = ghb_settings_get_boolean(settings, "SubtitleEnabled");
- 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 (!found)
- {
- if (enabled && !burned)
- {
- ghb_settings_set_boolean(settings, "SubtitleBurned", TRUE);
- gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 3, TRUE, -1);
- found = TRUE;
- }
- }
- else
- {
- if (enabled && !burned)
- {
- ghb_settings_set_boolean(settings, "SubtitleEnabled", FALSE);
- gtk_list_store_set(GTK_LIST_STORE(tm), &ti, 0, FALSE, -1);
- }
- }
+ ghb_widget_to_setting(settings, widget);
+ subtitle_list_refresh_selected(ud);
}
+ ghb_live_reset(ud);
}
void
@@ -659,15 +536,14 @@ 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, def;
+ gboolean forced, burned, def;
gchar *s_track;
gint i_track;
@@ -677,7 +553,6 @@ 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");
@@ -689,15 +564,13 @@ add_to_subtitle_list(
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, def,
- 5, source,
+ 0, track,
+ 1, forced,
+ 2, burned,
+ 3, def,
+ 4, source,
// These are used to set combo box values when a list item is selected
- 6, s_track,
- 7, can_delete,
+ 5, s_track,
-1);
gtk_tree_selection_select_iter(selection, &iter);
g_free(s_track);
@@ -716,21 +589,12 @@ subtitle_list_selection_changed_cb(GtkTreeSelection *selection, signal_user_data
if (gtk_tree_selection_get_selected(selection, &store, &iter))
{
const gchar *track;
- gboolean can_delete;
- gtk_tree_model_get(store, &iter, 6, &track, 7, &can_delete, -1);
- ghb_settings_set_string(ud->settings, "SubtitleTrack", track);
+ gtk_tree_model_get(store, &iter, 5, &track, -1);
+ ghb_ui_update(ud, "SubtitleTrack", ghb_string_value(track));
- if (can_delete)
- {
- widget = GHB_WIDGET (ud->builder, "subtitle_remove");
- gtk_widget_set_sensitive(widget, TRUE);
- }
- else
- {
- widget = GHB_WIDGET (ud->builder, "subtitle_remove");
- gtk_widget_set_sensitive(widget, FALSE);
- }
+ widget = GHB_WIDGET (ud->builder, "subtitle_remove");
+ gtk_widget_set_sensitive(widget, TRUE);
}
}
@@ -745,32 +609,42 @@ subtitle_add_clicked_cb(GtkWidget *xwidget, signal_user_data_t *ud)
gint track;
g_debug("subtitle_add_clicked_cb ()");
- track = ghb_pick_subtitle_track(ud);
- if (track < 0)
+ 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)
return;
+ track = ghb_settings_get_int(ud->settings, "SubtitleTrack");
+
if (mustBurn(ud, track))
{
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)
+ ghb_array_append(subtitle_list, settings);
+ add_to_subtitle_list(ud, settings);
+ ghb_subtitle_exclusive_burn(ud, track);
+ if (count == 98)
{
- subtitle_list = ghb_array_value_new(8);
- ghb_settings_set_value(ud->settings, "subtitle_list", subtitle_list);
+ GtkWidget *widget;
+ widget = GHB_WIDGET (ud->builder, "subtitle_add");
+ gtk_widget_set_sensitive(widget, FALSE);
}
- ghb_array_append(subtitle_list, settings);
- add_to_subtitle_list(ud, settings, TRUE);
- count = ghb_array_len(subtitle_list);
}
G_MODULE_EXPORT void
@@ -824,49 +698,34 @@ subtitle_remove_clicked_cb(GtkWidget *widget, signal_user_data_t *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 enabled, forced, burned, def;
-
- g_debug("ghb_set_subtitle");
- slist = ghb_settings_get_value(ud->settings, "subtitle_list");
+ GValue *subtitle_list, *settings;
+ gint count, ii, track;
+ gboolean burned;
- count = ghb_array_len(slist);
- for (ii = 0; ii < count; ii++)
+ 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);
+ for (ii = count-1; ii >= 0; 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");
+ settings = ghb_array_get_nth(subtitle_list, ii);
burned = ghb_settings_get_boolean(settings, "SubtitleBurned");
- 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);
-
- ghb_settings_set_boolean(subtitle, "SubtitleEnabled", enabled);
- ghb_settings_set_boolean(subtitle, "SubtitleForced", forced);
- ghb_settings_set_boolean(subtitle, "SubtitleBurned", burned);
- ghb_settings_set_boolean(subtitle, "SubtitleDefaultTrack", def);
- gtk_list_store_set(GTK_LIST_STORE(tm), &ti,
- 0, enabled,
- 2, forced,
- 3, burned,
- 4, def,
- 7, FALSE,
- -1);
- break;
+ track = ghb_settings_combo_int(settings, "SubtitleTrack");
+ if (!burned && mustBurn(ud, track))
+ {
+ 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);
+ }
}
}
@@ -875,7 +734,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");
@@ -884,16 +743,12 @@ 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");
-
- ghb_set_subtitle(ud, track, subtitle);
+ subtitle = ghb_value_dup(ghb_array_get_nth(slist, ii));
+ ghb_add_subtitle(ud, subtitle);
}
}
diff --git a/gtk/src/subtitlehandler.h b/gtk/src/subtitlehandler.h
index e1e9fc912..a2b920f34 100644
--- a/gtk/src/subtitlehandler.h
+++ b/gtk/src/subtitlehandler.h
@@ -30,11 +30,11 @@
void ghb_set_pref_subtitle(gint titleindex, signal_user_data_t *ud);
void ghb_clear_subtitle_list(signal_user_data_t *ud);
void ghb_set_subtitle(signal_user_data_t *ud, gint track, GValue *settings);
-void ghb_subtitle_adjust_burn(signal_user_data_t *ud);
GValue* ghb_selected_subtitle_settings(signal_user_data_t *ud);
gint ghb_selected_subtitle_row(signal_user_data_t *ud);
void ghb_add_subtitle(
- signal_user_data_t *ud, GValue *settings, gboolean can_delete);
+ signal_user_data_t *ud, GValue *settings);
void ghb_reset_subtitles(signal_user_data_t *ud, GValue *settings);
+void ghb_subtitle_prune(signal_user_data_t *ud);
#endif // _SUBTITLEHANDLER_H_