X-Git-Url: http://git.osdn.jp/view?a=blobdiff_plain;f=gtk%2Fsrc%2Fcallbacks.c;h=8fd2b7a94ed02a5ea4ae324d2928089996350fec;hb=5f753d54ac498a7d363f82ee1203b182ec4bb310;hp=42f7b6a096b7c2ca8bde2f2e385fef08a334cfed;hpb=546645566708444b0da1942315c42f77d07d6d7f;p=handbrake-jp%2Fhandbrake-jp-git.git diff --git a/gtk/src/callbacks.c b/gtk/src/callbacks.c index 42f7b6a0..8fd2b7a9 100644 --- a/gtk/src/callbacks.c +++ b/gtk/src/callbacks.c @@ -18,6 +18,7 @@ #include #include #include +#include #if !defined(_WIN32) #include @@ -27,7 +28,7 @@ #include #include -#include +#include #include #else #define WINVER 0x0500 @@ -44,6 +45,7 @@ #include "callbacks.h" #include "queuehandler.h" #include "audiohandler.h" +#include "subtitlehandler.h" #include "resources.h" #include "settings.h" #include "presets.h" @@ -59,6 +61,8 @@ static void update_chapter_list(signal_user_data_t *ud); static GList* dvd_device_list(); static void prune_logs(signal_user_data_t *ud); void ghb_notify_done(signal_user_data_t *ud); +gpointer ghb_check_update(signal_user_data_t *ud); +static gboolean appcast_busy = FALSE; // This is a dependency map used for greying widgets // that are dependent on the state of another widget. @@ -275,7 +279,7 @@ on_quit1_activate(GtkMenuItem *quit, signal_user_data_t *ud) gint state = ghb_get_queue_state(); g_debug("on_quit1_activate ()"); if (state & GHB_STATE_WORKING) - { + { if (ghb_cancel_encode("Closing HandBrake will terminate encoding.\n")) { ghb_hb_cleanup(FALSE); @@ -290,63 +294,11 @@ on_quit1_activate(GtkMenuItem *quit, signal_user_data_t *ud) gtk_main_quit(); } -static void -set_destination(signal_user_data_t *ud) -{ - g_debug("set_destination"); - if (ghb_settings_get_boolean(ud->settings, "use_source_name")) - { - GString *str = g_string_new(""); - gchar *vol_name, *filename, *extension; - gchar *new_name; - gint title; - - filename = ghb_settings_get_string(ud->settings, "dest_file"); - extension = ghb_settings_get_string(ud->settings, "FileFormat"); - vol_name = ghb_settings_get_string(ud->settings, "volume_label"); - g_string_append_printf(str, "%s", vol_name); - title = ghb_settings_combo_int(ud->settings, "title"); - if (title >= 0) - { - if (ghb_settings_get_boolean( - ud->settings, "title_no_in_destination")) - { - - title = ghb_settings_combo_int(ud->settings, "title"); - g_string_append_printf(str, " - %d", title+1); - } - if (ghb_settings_get_boolean( - ud->settings, "chapters_in_destination")) - { - gint start, end; - - if (!ghb_settings_get_boolean( - ud->settings, "title_no_in_destination")) - { - g_string_append_printf(str, " -"); - } - start = ghb_settings_get_int(ud->settings, "start_chapter"); - end = ghb_settings_get_int(ud->settings, "end_chapter"); - if (start == end) - g_string_append_printf(str, " Ch %d", start); - else - g_string_append_printf(str, " Ch %d-%d", start, end); - } - } - g_string_append_printf(str, ".%s", extension); - new_name = g_string_free(str, FALSE); - ghb_ui_update(ud, "dest_file", ghb_string_value(new_name)); - g_free(filename); - g_free(extension); - g_free(vol_name); - g_free(new_name); - } -} - gboolean -uppers_and_unders(const gchar *str) +uppers_and_unders(gchar *str) { if (str == NULL) return FALSE; + str = g_strchomp(g_strchug(str)); while (*str) { if (*str == ' ') @@ -399,6 +351,215 @@ camel_convert(gchar *str) } } +#if defined(_WIN32) +static gchar* +get_dvd_device_name(gchar *device) +{ + return g_strdup(device); +} +#else +static gchar* +get_dvd_device_name(GDrive *gd) +{ + return g_drive_get_identifier(gd, G_VOLUME_IDENTIFIER_KIND_UNIX_DEVICE); +} +#endif + +static GHashTable *volname_hash = NULL; +static GMutex *volname_mutex = NULL; + +static void +free_volname_key(gpointer data) +{ + if (data != NULL) + g_free(data); +} + +static void +free_volname_value(gpointer data) +{ + if (data != NULL) + g_free(data); +} + +#if defined(_WIN32) +static gchar* +get_direct_dvd_volume_name(const gchar *drive) +{ + gchar *result = NULL; + gchar vname[51], fsname[51]; + + if (GetVolumeInformation(drive, vname, 50, NULL, NULL, NULL, fsname, 51)) + { + result = g_strdup_printf("%s", vname); + } + return result; +} +#else +static gchar* +get_direct_dvd_volume_name(const gchar *drive) +{ + gchar *result; + + result = ghb_dvd_volname (drive); + return result; +} +#endif + +static gchar* +get_dvd_volume_name(gpointer gd) +{ + gchar *label = NULL; + gchar *result; + gchar *drive; + + drive = get_dvd_device_name(gd); + g_mutex_lock(volname_mutex); + label = g_strdup(g_hash_table_lookup(volname_hash, drive)); + g_mutex_unlock(volname_mutex); + if (label != NULL) + { + if (uppers_and_unders(label)) + { + camel_convert(label); + } +#if defined(_WIN32) + result = g_strdup_printf("%s (%s)", label, drive); +#else + result = g_strdup_printf("%s - %s", drive, label); +#endif + g_free(label); + } + else + { + result = g_strdup_printf("%s", drive); + } + g_free(drive); + return result; +} + +void +ghb_volname_cache_init(void) +{ + volname_mutex = g_mutex_new(); + volname_hash = g_hash_table_new_full(g_str_hash, g_str_equal, + free_volname_key, free_volname_value); +} + +static void +free_drive(gpointer drive) +{ +#if defined(_WIN32) + g_free(drive); +#else + g_object_unref(drive); +#endif +} + +gpointer +ghb_cache_volnames(signal_user_data_t *ud) +{ + GList *link, *drives; + + g_debug("ghb_cache_volnames()"); + link = drives = dvd_device_list(); + if (drives == NULL) + return NULL; + + g_mutex_lock(volname_mutex); + g_hash_table_remove_all(volname_hash); + while (link != NULL) + { + gchar *name, *drive; + +#if !defined(_WIN32) + if (!g_drive_has_media (link->data)) + { + g_object_unref(link->data); + link = link->next; + continue; + } +#endif + drive = get_dvd_device_name(link->data); + name = get_direct_dvd_volume_name(drive); + + if (drive != NULL && name != NULL) + { + g_hash_table_insert(volname_hash, drive, name); + } + else + { + if (drive != NULL) + g_free(drive); + if (name != NULL) + g_free(name); + } + + free_drive(link->data); + link = link->next; + } + g_mutex_unlock(volname_mutex); + + g_list_free(drives); + + g_idle_add((GSourceFunc)ghb_file_menu_add_dvd, ud); + + return NULL; +} + +static void +set_destination(signal_user_data_t *ud) +{ + g_debug("set_destination"); + if (ghb_settings_get_boolean(ud->settings, "use_source_name")) + { + GString *str = g_string_new(""); + gchar *vol_name, *filename, *extension; + gchar *new_name; + gint title; + + filename = ghb_settings_get_string(ud->settings, "dest_file"); + extension = ghb_settings_get_string(ud->settings, "FileFormat"); + vol_name = ghb_settings_get_string(ud->settings, "volume_label"); + g_string_append_printf(str, "%s", vol_name); + title = ghb_settings_combo_int(ud->settings, "title"); + if (title >= 0) + { + if (ghb_settings_get_boolean( + ud->settings, "title_no_in_destination")) + { + + title = ghb_settings_combo_int(ud->settings, "title"); + g_string_append_printf(str, " - %d", title+1); + } + if (ghb_settings_get_boolean( + ud->settings, "chapters_in_destination")) + { + gint start, end; + + if (!ghb_settings_get_boolean( + ud->settings, "title_no_in_destination")) + { + g_string_append_printf(str, " -"); + } + start = ghb_settings_get_int(ud->settings, "start_chapter"); + end = ghb_settings_get_int(ud->settings, "end_chapter"); + if (start == end) + g_string_append_printf(str, " Ch %d", start); + else + g_string_append_printf(str, " Ch %d-%d", start, end); + } + } + g_string_append_printf(str, ".%s", extension); + new_name = g_string_free(str, FALSE); + ghb_ui_update(ud, "dest_file", ghb_string_value(new_name)); + g_free(filename); + g_free(extension); + g_free(vol_name); + g_free(new_name); + } +} + static gchar* get_file_label(const gchar *filename) { @@ -422,26 +583,6 @@ get_file_label(const gchar *filename) } static gchar* -get_drive_name(const gchar *drive) -{ - gchar *result; -#if defined(_WIN32) - gchar vname[51], fsname[51]; - if (GetVolumeInformation(drive, vname, 50, NULL, NULL, NULL, fsname, 51)) - { - result = g_strdup_printf("%s (%s)", vname, drive); - } - else - { - result = g_strdup_printf("%s", drive); - } -#else - result = g_strdup_printf("%s", drive); -#endif - return result; -} - -static gchar* resolve_drive_name(gchar *filename) { #if defined(_WIN32) @@ -663,9 +804,10 @@ source_dialog_extra_widgets( gtk_combo_box_append_text (combo, "Not Selected"); while (link != NULL) { - gchar *name = (gchar*)link->data; + gchar *name = get_dvd_device_name(link->data); gtk_combo_box_append_text(combo, name); g_free(name); + free_drive(link->data); link = link->next; } g_list_free(drives); @@ -691,6 +833,7 @@ ghb_do_scan( jstatus = ghb_settings_get_int(ghb_queue_edit_settings, "job_status"); ghb_settings_to_ui(ud, ghb_queue_edit_settings); ghb_set_audio(ud, ghb_queue_edit_settings); + ghb_reset_subtitles(ud, ghb_queue_edit_settings); if (jstatus == GHB_QUEUE_PENDING) { ghb_value_free(ghb_queue_edit_settings); @@ -1036,6 +1179,8 @@ G_MODULE_EXPORT void container_changed_cb(GtkWidget *widget, signal_user_data_t *ud) { const GValue *audio_list; + gboolean markers; + g_debug("container_changed_cb ()"); ghb_widget_to_setting(ud->settings, widget); update_destination_extension(ud); @@ -1056,6 +1201,19 @@ container_changed_cb(GtkWidget *widget, signal_user_data_t *ud) } g_free(container); } + markers = ghb_settings_get_boolean(ud->settings, "ChapterMarkers"); + if (markers) + { + gchar *container; + + container = ghb_settings_get_string(ud->settings, "FileFormat"); + if (strcmp(container, "mp4") == 0) + { + ghb_ui_update(ud, "FileFormat", ghb_string_value("m4v")); + } + g_free(container); + } + ghb_subtitle_prune(ud); } static gchar* @@ -1172,7 +1330,11 @@ show_title_info(signal_user_data_t *ud, ghb_title_info_t *tinfo) widget = GHB_WIDGET (ud->builder, "start_chapter"); gtk_spin_button_set_value (GTK_SPIN_BUTTON(widget), 1); gtk_spin_button_set_range (GTK_SPIN_BUTTON(widget), 1, tinfo->num_chapters); - ud->dont_clear_presets = TRUE; + + widget = GHB_WIDGET (ud->builder, "angle"); + gtk_spin_button_set_value (GTK_SPIN_BUTTON(widget), 1); + gtk_spin_button_set_range (GTK_SPIN_BUTTON(widget), 1, tinfo->angle_count); + ud->dont_clear_presets = FALSE; } static gboolean update_preview = FALSE; @@ -1188,10 +1350,9 @@ title_changed_cb(GtkWidget *widget, signal_user_data_t *ud) ghb_check_dependency(ud, widget); titleindex = ghb_settings_combo_int(ud->settings, "title"); - ghb_update_ui_combo_box (ud->builder, "AudioTrack", titleindex, FALSE); - ghb_update_ui_combo_box (ud->builder, "Subtitles", titleindex, FALSE); + ghb_update_ui_combo_box (ud, "AudioTrack", titleindex, FALSE); + ghb_update_ui_combo_box (ud, "SubtitleTrack", titleindex, FALSE); - ghb_update_from_preset(ud, "Subtitles"); if (ghb_get_title_info (&tinfo, titleindex)) { show_title_info(ud, &tinfo); @@ -1199,6 +1360,7 @@ title_changed_cb(GtkWidget *widget, signal_user_data_t *ud) update_chapter_list (ud); ghb_adjust_audio_rate_combos(ud); ghb_set_pref_audio(titleindex, ud); + ghb_set_pref_subtitle(titleindex, ud); if (ghb_settings_get_boolean(ud->settings, "vquality_type_target")) { gint bitrate = ghb_calculate_target_bitrate (ud->settings, titleindex); @@ -1233,6 +1395,29 @@ setting_widget_changed_cb(GtkWidget *widget, signal_user_data_t *ud) } G_MODULE_EXPORT void +chapter_markers_changed_cb(GtkWidget *widget, signal_user_data_t *ud) +{ + gboolean markers; + + ghb_widget_to_setting(ud->settings, widget); + ghb_check_dependency(ud, widget); + ghb_clear_presets_selection(ud); + ghb_live_reset(ud); + markers = ghb_settings_get_boolean(ud->settings, "ChapterMarkers"); + if (markers) + { + gchar *container; + + container = ghb_settings_get_string(ud->settings, "FileFormat"); + if (strcmp(container, "mp4") == 0) + { + ghb_ui_update(ud, "FileFormat", ghb_string_value("m4v")); + } + g_free(container); + } +} + +G_MODULE_EXPORT void vquality_changed_cb(GtkWidget *widget, signal_user_data_t *ud) { ghb_widget_to_setting(ud->settings, widget); @@ -1321,6 +1506,17 @@ start_chapter_changed_cb(GtkWidget *widget, signal_user_data_t *ud) { set_destination(ud); } + widget = GHB_WIDGET (ud->builder, "chapters_tab"); + // End may have been changed above, get it again + end = ghb_settings_get_int(ud->settings, "end_chapter"); + if (start == end) + { + gtk_widget_hide(widget); + } + else + { + gtk_widget_show(widget); + } } G_MODULE_EXPORT void @@ -1340,6 +1536,17 @@ end_chapter_changed_cb(GtkWidget *widget, signal_user_data_t *ud) { set_destination(ud); } + widget = GHB_WIDGET (ud->builder, "chapters_tab"); + // Start may have been changed above, get it again + start = ghb_settings_get_int(ud->settings, "start_chapter"); + if (start == end) + { + gtk_widget_hide(widget); + } + else + { + gtk_widget_show(widget); + } } G_MODULE_EXPORT void @@ -1410,6 +1617,8 @@ crop_changed_cb(GtkWidget *widget, signal_user_data_t *ud) widget = GHB_WIDGET (ud->builder, "crop_dimensions"); text = g_strdup_printf ("%d x %d", width, height); gtk_label_set_text (GTK_LABEL(widget), text); + widget = GHB_WIDGET (ud->builder, "crop_dimensions2"); + gtk_label_set_text (GTK_LABEL(widget), text); g_free(text); } gchar *text; @@ -1424,7 +1633,7 @@ crop_changed_cb(GtkWidget *widget, signal_user_data_t *ud) G_MODULE_EXPORT void display_width_changed_cb(GtkWidget *widget, signal_user_data_t *ud) { - g_debug("scale_changed_cb ()"); + g_debug("display_width_changed_cb ()"); ghb_widget_to_setting(ud->settings, widget); ghb_check_dependency(ud, widget); ghb_clear_presets_selection(ud); @@ -1432,25 +1641,13 @@ display_width_changed_cb(GtkWidget *widget, signal_user_data_t *ud) if (GTK_WIDGET_SENSITIVE(widget)) ghb_set_scale (ud, GHB_PIC_KEEP_DISPLAY_WIDTH); - gint pic_par; - - pic_par = ghb_settings_combo_int(ud->settings, "PicturePAR"); - if (pic_par == 3) - { - gint par_width, par_height; - - par_width = ghb_settings_get_int(ud->settings, "par_width"); - par_height = ghb_settings_get_int(ud->settings, "par_height"); - ghb_settings_set_int(ud->settings, "PicturePARWidth", par_width); - ghb_settings_set_int(ud->settings, "PicturePARHeight", par_height); - } update_preview = TRUE; } -void +G_MODULE_EXPORT void display_height_changed_cb(GtkWidget *widget, signal_user_data_t *ud) { - g_debug("scale_changed_cb ()"); + g_debug("display_height_changed_cb ()"); ghb_widget_to_setting(ud->settings, widget); ghb_check_dependency(ud, widget); ghb_clear_presets_selection(ud); @@ -1458,22 +1655,24 @@ display_height_changed_cb(GtkWidget *widget, signal_user_data_t *ud) if (GTK_WIDGET_SENSITIVE(widget)) ghb_set_scale (ud, GHB_PIC_KEEP_DISPLAY_HEIGHT); - gint pic_par; + update_preview = TRUE; +} - pic_par = ghb_settings_combo_int(ud->settings, "PicturePAR"); - if (pic_par == 3) - { - gint par_width, par_height; +G_MODULE_EXPORT void +par_changed_cb(GtkWidget *widget, signal_user_data_t *ud) +{ + g_debug("par_changed_cb ()"); + ghb_widget_to_setting(ud->settings, widget); + ghb_check_dependency(ud, widget); + ghb_clear_presets_selection(ud); + ghb_live_reset(ud); + if (GTK_WIDGET_SENSITIVE(widget)) + ghb_set_scale (ud, GHB_PIC_KEEP_PAR); - par_width = ghb_settings_get_int(ud->settings, "par_width"); - par_height = ghb_settings_get_int(ud->settings, "par_height"); - ghb_settings_set_int(ud->settings, "PicturePARWidth", par_width); - ghb_settings_set_int(ud->settings, "PicturePARHeight", par_height); - } update_preview = TRUE; } -void +G_MODULE_EXPORT void scale_changed_cb(GtkWidget *widget, signal_user_data_t *ud) { g_debug("scale_changed_cb ()"); @@ -1516,6 +1715,18 @@ scale_changed_cb(GtkWidget *widget, signal_user_data_t *ud) } G_MODULE_EXPORT void +show_crop_changed_cb(GtkWidget *widget, signal_user_data_t *ud) +{ + g_debug("show_crop_changed_cb ()"); + ghb_widget_to_setting(ud->settings, widget); + ghb_check_dependency(ud, widget); + ghb_live_reset(ud); + if (GTK_WIDGET_SENSITIVE(widget)) + ghb_set_scale (ud, 0); + update_preview = TRUE; +} + +G_MODULE_EXPORT void generic_entry_changed_cb(GtkEntry *entry, signal_user_data_t *ud) { // Normally (due to user input) I only want to process the entry @@ -1865,6 +2076,17 @@ ghb_backend_events(signal_user_data_t *ud) ghb_track_status(); ghb_get_status(&status); progress = GTK_PROGRESS_BAR(GHB_WIDGET (ud->builder, "progressbar")); + if (status.scan.state == GHB_STATE_IDLE && + status.queue.state == GHB_STATE_IDLE) + { + static gboolean prev_dvdnav; + gboolean dvdnav = ghb_settings_get_boolean(ud->settings, "use_dvdnav"); + if (dvdnav != prev_dvdnav) + { + hb_dvd_set_dvdnav(dvdnav); + prev_dvdnav = dvdnav; + } + } // First handle the status of title scans // Then handle the status of the queue if (status.scan.state & GHB_STATE_SCANNING) @@ -1895,7 +2117,7 @@ ghb_backend_events(signal_user_data_t *ud) ghb_title_info_t tinfo; - ghb_update_ui_combo_box(ud->builder, "title", 0, FALSE); + ghb_update_ui_combo_box(ud, "title", 0, FALSE); titleindex = ghb_longest_title(); ghb_ui_update(ud, "title", ghb_int64_value(titleindex)); @@ -1916,6 +2138,7 @@ ghb_backend_events(signal_user_data_t *ud) jstatus = ghb_settings_get_int(ghb_queue_edit_settings, "job_status"); ghb_settings_to_ui(ud, ghb_queue_edit_settings); ghb_set_audio(ud, ghb_queue_edit_settings); + ghb_reset_subtitles(ud, ghb_queue_edit_settings); if (jstatus == GHB_QUEUE_PENDING) { ghb_value_free(ghb_queue_edit_settings); @@ -2022,6 +2245,12 @@ ghb_backend_events(signal_user_data_t *ud) ghb_settings_set_int(js, "job_status", qstatus); ghb_save_queue(ud->queue); ud->cancel_encode = FALSE; +#if !GTK_CHECK_VERSION(2, 16, 0) + GtkStatusIcon *si; + + si = GTK_STATUS_ICON(GHB_OBJECT(ud->builder, "hb_status")); + gtk_status_icon_set_tooltip(si, "HandBrake"); +#endif } else if (status.queue.state & GHB_STATE_MUXING) { @@ -2068,6 +2297,12 @@ ghb_backend_events(signal_user_data_t *ud) status_str = working_status_string(ud, &status.queue); label = GTK_LABEL(GHB_WIDGET(ud->builder, "queue_status")); gtk_label_set_text (label, status_str); +#if !GTK_CHECK_VERSION(2, 16, 0) + GtkStatusIcon *si; + + si = GTK_STATUS_ICON(GHB_OBJECT(ud->builder, "hb_status")); + gtk_status_icon_set_tooltip(si, status_str); +#endif g_free(status_str); } if (status.scan.state & GHB_STATE_WORKING) @@ -2097,6 +2332,34 @@ ghb_backend_events(signal_user_data_t *ud) } } +#if GTK_CHECK_VERSION(2, 16, 0) +G_MODULE_EXPORT gboolean +status_icon_query_tooltip_cb( + GtkStatusIcon *si, + gint x, + gint y, + gboolean kbd_mode, + GtkTooltip *tt, + signal_user_data_t *ud) +{ + ghb_status_t status; + gchar *status_str; + + ghb_get_status(&status); + if (status.queue.state & GHB_STATE_WORKING) + status_str = working_status_string(ud, &status.queue); + else if (status.queue.state & GHB_STATE_WORKDONE) + status_str = g_strdup("Encode Complete"); + else + status_str = g_strdup("HandBrake"); + + gtk_tooltip_set_text(tt, status_str); + gtk_tooltip_set_icon_from_icon_name(tt, "hb-icon", GTK_ICON_SIZE_BUTTON); + g_free(status_str); + return TRUE; +} +#endif + G_MODULE_EXPORT gboolean ghb_timer_cb(gpointer data) { @@ -2122,9 +2385,41 @@ ghb_timer_cb(gpointer data) } if (update_preview) { + g_debug("Updating preview\n"); ghb_set_preview_image (ud); update_preview = FALSE; } + + if (!appcast_busy) + { + gchar *updates; + updates = ghb_settings_get_string(ud->settings, "check_updates"); + gint64 duration = 0; + if (strcmp(updates, "daily") == 0) + duration = 60 * 60 * 24; + else if (strcmp(updates, "weekly") == 0) + duration = 60 * 60 * 24 * 7; + else if (strcmp(updates, "monthly") == 0) + duration = 60 * 60 * 24 * 7; + + g_free(updates); + if (duration != 0) + { + gint64 last; + time_t tt; + + last = ghb_settings_get_int64(ud->settings, "last_update_check"); + time(&tt); + if (last + duration < tt) + { + ghb_settings_set_int64(ud->settings, + "last_update_check", tt); + ghb_pref_save(ud->settings, "last_update_check"); + g_thread_create((GThreadFunc)ghb_check_update, ud, + FALSE, NULL); + } + } + } return TRUE; } @@ -2271,7 +2566,7 @@ ghb_log(gchar *log, ...) _now = time(NULL); now = localtime( &_now ); - snprintf(fmt, 362, "[%02d:%02d:%02d] lingui: %s\n", + snprintf(fmt, 362, "[%02d:%02d:%02d] gtkgui: %s\n", now->tm_hour, now->tm_min, now->tm_sec, log); va_start(args, log); vfprintf(stderr, fmt, args); @@ -2560,13 +2855,6 @@ chapter_edited_cb( gtk_tree_path_free (treepath); } -G_MODULE_EXPORT void -chapter_list_selection_changed_cb(GtkTreeSelection *selection, signal_user_data_t *ud) -{ - g_debug("chapter_list_selection_changed_cb ()"); - //chapter_selection_changed = TRUE; -} - void debug_log_handler(const gchar *domain, GLogLevelFlags flags, const gchar *msg, gpointer data) { @@ -2634,6 +2922,21 @@ pref_changed_cb(GtkWidget *widget, signal_user_data_t *ud) } G_MODULE_EXPORT void +skip_taskbar_cb(GtkWidget *widget, signal_user_data_t *ud) +{ + g_debug("pref_changed_cb"); + ghb_widget_to_setting (ud->settings, widget); + ghb_check_dependency(ud, widget); + const gchar *name = gtk_widget_get_name(widget); + ghb_pref_save(ud->settings, name); + + GtkWindow *window; + window = GTK_WINDOW(GHB_WIDGET (ud->builder, "hb_window")); + gtk_window_set_skip_taskbar_hint(window, + ghb_settings_get_boolean(ud->settings, "skip_taskbar")); +} + +G_MODULE_EXPORT void vqual_granularity_changed_cb(GtkWidget *widget, signal_user_data_t *ud) { g_debug("vqual_granularity_changed_cb"); @@ -2681,52 +2984,66 @@ hbfd_feature_changed_cb(GtkWidget *widget, signal_user_data_t *ud) gtk_action_set_visible(action, hbfd); } -void +gboolean ghb_file_menu_add_dvd(signal_user_data_t *ud) { GList *link, *drives; + static GtkActionGroup *agroup = NULL; + static gint merge_id; - GtkActionGroup *agroup = GTK_ACTION_GROUP( - gtk_builder_get_object(ud->builder, "actiongroup1")); - GtkUIManager *ui = GTK_UI_MANAGER( - gtk_builder_get_object(ud->builder, "uimanager1")); - guint merge_id = gtk_ui_manager_new_merge_id(ui); - + g_debug("ghb_file_menu_add_dvd()"); link = drives = dvd_device_list(); - while (link != NULL) + if (drives != NULL) { - GtkAction *action; - gchar *drive = (gchar*)link->data; - gchar *name = get_drive_name(drive); - - action = gtk_action_group_get_action(agroup, drive); - if (action != NULL) - { - gtk_action_group_remove_action(agroup, action); - g_object_unref(G_OBJECT(action)); - } - // Create action for this drive - action = gtk_action_new(drive, name, - "Scan this DVD source", "gtk-cdrom"); - // Add action to action group - gtk_action_group_add_action_with_accel(agroup, action, NULL); - // Add to ui manager + GtkUIManager *ui = GTK_UI_MANAGER( + gtk_builder_get_object(ud->builder, "uimanager1")); + + if (agroup == NULL) + { + agroup = gtk_action_group_new("dvdgroup"); + gtk_ui_manager_insert_action_group(ui, agroup, 0); + } + else + gtk_ui_manager_remove_ui(ui, merge_id); + + merge_id = gtk_ui_manager_new_merge_id(ui); + // Add separator gtk_ui_manager_add_ui(ui, merge_id, - "ui/menubar1/menuitem1/quit1", drive, drive, - GTK_UI_MANAGER_AUTO, TRUE); - // Connect signal to action (menu item) - g_signal_connect(action, "activate", - (GCallback)dvd_source_activate_cb, ud); - g_free(name); - g_free(drive); - link = link->next; - } - g_list_free(drives); + "ui/menubar1/menuitem1/quit1", "dvdsep", NULL, + GTK_UI_MANAGER_SEPARATOR, TRUE); - // Add separator - gtk_ui_manager_add_ui(ui, merge_id, - "ui/menubar1/menuitem1/quit1", "", NULL, - GTK_UI_MANAGER_AUTO, TRUE); + while (link != NULL) + { + GtkAction *action; + gchar *drive = get_dvd_device_name(link->data); + gchar *name = get_dvd_volume_name(link->data); + + action = gtk_action_group_get_action(agroup, drive); + if (action != NULL) + { + gtk_action_group_remove_action(agroup, action); + g_object_unref(G_OBJECT(action)); + } + // Create action for this drive + action = gtk_action_new(drive, name, + "Scan this DVD source", "gtk-cdrom"); + // Add action to action group + gtk_action_group_add_action_with_accel(agroup, action, NULL); + // Add to ui manager + gtk_ui_manager_add_ui(ui, merge_id, + "ui/menubar1/menuitem1/dvdsep", drive, drive, + GTK_UI_MANAGER_AUTO, TRUE); + // Connect signal to action (menu item) + g_signal_connect(action, "activate", + (GCallback)dvd_source_activate_cb, ud); + g_free(name); + g_free(drive); + free_drive(link->data); + link = link->next; + } + g_list_free(drives); + } + return FALSE; } gboolean ghb_is_cd(GDrive *gd); @@ -2772,11 +3089,10 @@ dvd_device_list() gd = (GDrive*)link->data; if (ghb_is_cd(gd)) { - gchar *device; - device = g_drive_get_identifier(gd, G_VOLUME_IDENTIFIER_KIND_UNIX_DEVICE); - dvd_devices = g_list_append(dvd_devices, (gpointer)device); + dvd_devices = g_list_append(dvd_devices, gd); } - g_object_unref (gd); + else + g_object_unref (gd); link = link->next; } g_list_free(drives); @@ -2816,6 +3132,8 @@ handle_media_change(const gchar *device, gboolean insert, signal_user_data_t *ud static gint ins_count = 0; static gint rem_count = 0; + // The media change event in windows bounces around a bit + // so I debounce it here // DVD insertion detected. Scan it. dtype = GetDriveType(device); if (dtype != DRIVE_CDROM) @@ -2826,7 +3144,7 @@ handle_media_change(const gchar *device, gboolean insert, signal_user_data_t *ud ins_count++; if (ins_count == 2) { - ghb_file_menu_add_dvd(ud); + g_thread_create((GThreadFunc)ghb_cache_volnames, ud, FALSE, NULL); if (ud->current_dvd_device != NULL && strcmp(device, ud->current_dvd_device) == 0) { @@ -2834,9 +3152,7 @@ handle_media_change(const gchar *device, gboolean insert, signal_user_data_t *ud progress = GTK_PROGRESS_BAR(GHB_WIDGET (ud->builder, "progressbar")); gtk_progress_bar_set_text (progress, "Scanning ..."); gtk_progress_bar_set_fraction (progress, 0); - update_source_label(ud, device); - ghb_hb_cleanup(TRUE); - prune_logs(ud); + update_source_label(ud, device); gint preview_count; preview_count = ghb_settings_get_int(ud->settings, "preview_count"); ghb_backend_scan(device, 0, preview_count); @@ -2849,7 +3165,7 @@ handle_media_change(const gchar *device, gboolean insert, signal_user_data_t *ud rem_count++; if (rem_count == 2) { - ghb_file_menu_add_dvd(ud); + g_thread_create((GThreadFunc)ghb_cache_volnames, ud, FALSE, NULL); if (ud->current_dvd_device != NULL && strcmp(device, ud->current_dvd_device) == 0) { @@ -2920,42 +3236,36 @@ G_MODULE_EXPORT void drive_changed_cb(GVolumeMonitor *gvm, GDrive *gd, signal_user_data_t *ud) { gchar *device; - gint state = ghb_get_scan_state(); - static gboolean first_time = TRUE; + gint state; + + g_debug("drive_changed_cb()"); + g_thread_create((GThreadFunc)ghb_cache_volnames, ud, FALSE, NULL); - if (state != GHB_STATE_IDLE) return; - if (ud->current_dvd_device == NULL) return; - // A drive change event happens when the program initially starts - // and I don't want to automatically scan at that time. - if (first_time) + state = ghb_get_scan_state(); + device = g_drive_get_identifier(gd, G_VOLUME_IDENTIFIER_KIND_UNIX_DEVICE); + if (ud->current_dvd_device == NULL || + strcmp(device, ud->current_dvd_device) != 0 || + state != GHB_STATE_IDLE ) { - first_time = FALSE; return; } - device = g_drive_get_identifier(gd, G_VOLUME_IDENTIFIER_KIND_UNIX_DEVICE); - - // DVD insertion detected. Scan it. - if (strcmp(device, ud->current_dvd_device) == 0) + if (g_drive_has_media(gd)) { - if (g_drive_has_media (gd)) - { - GtkProgressBar *progress; - progress = GTK_PROGRESS_BAR(GHB_WIDGET (ud->builder, "progressbar")); - gtk_progress_bar_set_text (progress, "Scanning ..."); - gtk_progress_bar_set_fraction (progress, 0); - update_source_label(ud, device); - prune_logs(ud); - gint preview_count; - preview_count = ghb_settings_get_int(ud->settings, "preview_count"); - ghb_backend_scan(device, 0, preview_count); - } - else - { - prune_logs(ud); - ghb_backend_scan("/dev/null", 0, 1); - } + GtkProgressBar *progress; + progress = GTK_PROGRESS_BAR(GHB_WIDGET (ud->builder, "progressbar")); + gtk_progress_bar_set_text (progress, "Scanning ..."); + gtk_progress_bar_set_fraction (progress, 0); + update_source_label(ud, device); + gint preview_count; + preview_count = ghb_settings_get_int(ud->settings, "preview_count"); + ghb_backend_scan(device, 0, preview_count); + } + else + { + ghb_hb_cleanup(TRUE); + prune_logs(ud); + ghb_backend_scan("/dev/null", 0, 1); } - g_free(device); } #endif @@ -2992,7 +3302,7 @@ ghb_inhibit_gpm() conn = dbus_g_bus_get(DBUS_BUS_SESSION, &error); if (error != NULL) { - g_debug("DBUS cannot connect: %s", error->message); + g_warning("DBUS cannot connect: %s", error->message); g_error_free(error); return; } @@ -3000,7 +3310,7 @@ ghb_inhibit_gpm() GPM_DBUS_INHIBIT_PATH, GPM_DBUS_INHIBIT_INTERFACE); if (proxy == NULL) { - g_debug("Could not get DBUS proxy: %s", GPM_DBUS_SERVICE); + g_warning("Could not get DBUS proxy: %s", GPM_DBUS_SERVICE); dbus_g_connection_unref(conn); return; } @@ -3010,18 +3320,20 @@ ghb_inhibit_gpm() G_TYPE_INVALID, G_TYPE_UINT, &gpm_cookie, G_TYPE_INVALID); + gpm_inhibited = TRUE; if (!res) { - g_warning("Inhibit method failed"); - gpm_cookie = -1; - } - if (error != NULL) - { - g_warning("Inhibit problem: %s", error->message); - g_error_free(error); + if (error != NULL) + { + g_warning("Inhibit failed: %s", error->message); + g_error_free(error); + gpm_cookie = -1; + } + else + g_warning("Inhibit failed"); gpm_cookie = -1; + gpm_inhibited = FALSE; } - gpm_inhibited = TRUE; g_object_unref(G_OBJECT(proxy)); dbus_g_connection_unref(conn); #endif @@ -3046,7 +3358,7 @@ ghb_uninhibit_gpm() conn = dbus_g_bus_get(DBUS_BUS_SESSION, &error); if (error != NULL) { - g_debug("DBUS cannot connect: %s", error->message); + g_warning("DBUS cannot connect: %s", error->message); g_error_free(error); return; } @@ -3054,7 +3366,7 @@ ghb_uninhibit_gpm() GPM_DBUS_INHIBIT_PATH, GPM_DBUS_INHIBIT_INTERFACE); if (proxy == NULL) { - g_debug("Could not get DBUS proxy: %s", GPM_DBUS_SERVICE); + g_warning("Could not get DBUS proxy: %s", GPM_DBUS_SERVICE); dbus_g_connection_unref(conn); return; } @@ -3064,12 +3376,13 @@ ghb_uninhibit_gpm() G_TYPE_INVALID); if (!res) { - g_warning("UnInhibit method failed"); - } - if (error != NULL) - { - g_warning("UnInhibit problem: %s", error->message); - g_error_free(error); + if (error != NULL) + { + g_warning("UnInhibit failed: %s", error->message); + g_error_free(error); + } + else + g_warning("UnInhibit failed"); } gpm_inhibited = FALSE; dbus_g_connection_unref(conn); @@ -3267,7 +3580,6 @@ format_vquality_cb(GtkScale *scale, gdouble val, signal_user_data_t *ud) return g_strdup_printf("QP: %.4g (%.0f%%)", val, percent); } break; - case HB_VCODEC_XVID: case HB_VCODEC_FFMPEG: { percent = 100. * (30 - (val - 1)) / 30.; @@ -3288,23 +3600,13 @@ format_vquality_cb(GtkScale *scale, gdouble val, signal_user_data_t *ud) return g_strdup_printf("QP: %.1f / %.1f%%", val, percent); } -#if !defined(_WIN32) -G_MODULE_EXPORT void -html_link_cb(GtkHTML *html, const gchar *url, signal_user_data_t *ud) -{ - browse_url(url); -} -#endif - -static gpointer check_stable_update(signal_user_data_t *ud); -static gboolean stable_update_lock = FALSE; - static void process_appcast(signal_user_data_t *ud) { gchar *description = NULL, *build = NULL, *version = NULL, *msg; #if !defined(_WIN32) - GtkWidget *html, *window; + GtkWidget *window; + static GtkWidget *html = NULL; #endif GtkWidget *dialog, *label; gint response, ibuild = 0, skip; @@ -3312,8 +3614,6 @@ process_appcast(signal_user_data_t *ud) if (ud->appcast == NULL || ud->appcast_len < 15 || strncmp(&(ud->appcast[9]), "200 OK", 6)) { - if (!stable_update_lock && hb_get_build(NULL) % 100) - g_idle_add((GSourceFunc)check_stable_update, ud); goto done; } ghb_appcast_parse(ud->appcast, &description, &build, &version); @@ -3323,29 +3623,28 @@ process_appcast(signal_user_data_t *ud) if (description == NULL || build == NULL || version == NULL || ibuild <= hb_get_build(NULL) || skip == ibuild) { - if (!stable_update_lock && hb_get_build(NULL) % 100) - g_thread_create((GThreadFunc)check_stable_update, ud, FALSE, NULL); goto done; } msg = g_strdup_printf("HandBrake %s/%s is now available (you have %s/%d).", version, build, hb_get_version(NULL), hb_get_build(NULL)); label = GHB_WIDGET(ud->builder, "update_message"); gtk_label_set_text(GTK_LABEL(label), msg); + #if !defined(_WIN32) - html = gtk_html_new_from_string(description, -1); - g_signal_connect(html, "link_clicked", G_CALLBACK(html_link_cb), ud); - window = GHB_WIDGET(ud->builder, "update_scroll"); - gtk_container_add(GTK_CONTAINER(window), html); - // Show it - gtk_widget_set_size_request(html, 420, 240); - gtk_widget_show(html); + if (html == NULL) + { + html = webkit_web_view_new(); + window = GHB_WIDGET(ud->builder, "update_scroll"); + gtk_container_add(GTK_CONTAINER(window), html); + // Show it + gtk_widget_set_size_request(html, 420, 240); + gtk_widget_show(html); + } + webkit_web_view_open(WEBKIT_WEB_VIEW(html), description); #endif dialog = GHB_WIDGET(ud->builder, "update_dialog"); response = gtk_dialog_run(GTK_DIALOG(dialog)); gtk_widget_hide(dialog); -#if !defined(_WIN32) - gtk_widget_destroy(html); -#endif if (response == GTK_RESPONSE_OK) { // Skip @@ -3361,6 +3660,7 @@ done: g_free(ud->appcast); ud->appcast_len = 0; ud->appcast = NULL; + appcast_busy = FALSE; } void @@ -3418,6 +3718,7 @@ ghb_net_open(signal_user_data_t *ud, gchar *address, gint port) if( !( host = gethostbyname( address ) ) ) { g_warning( "gethostbyname failed (%s)", address ); + appcast_busy = FALSE; return NULL; } @@ -3430,12 +3731,14 @@ ghb_net_open(signal_user_data_t *ud, gchar *address, gint port) if( fd < 0 ) { g_debug( "socket failed" ); + appcast_busy = FALSE; return NULL; } if(connect(fd, (struct sockaddr*)&sock, sizeof(struct sockaddr_in )) < 0 ) { g_debug( "connect failed" ); + appcast_busy = FALSE; return NULL; } ioc = g_io_channel_unix_new(fd); @@ -3453,48 +3756,38 @@ ghb_check_update(signal_user_data_t *ud) gsize len; GIOChannel *ioc; GError *gerror = NULL; + GRegex *regex; + GMatchInfo *mi; + gchar *host, *appcast; g_debug("ghb_check_update"); - if (hb_get_build(NULL) % 100) - { - query = - "GET /appcast_unstable.xml HTTP/1.0\r\nHost: handbrake.fr\r\n\r\n"; - } - else + appcast_busy = TRUE; + regex = g_regex_new("^http://(.+)/(.+)$", 0, 0, NULL); + if (!g_regex_match(regex, HB_PROJECT_URL_APPCAST, 0, &mi)) { - stable_update_lock = TRUE; - query = "GET /appcast.xml HTTP/1.0\r\nHost: handbrake.fr\r\n\r\n"; - } - ioc = ghb_net_open(ud, "handbrake.fr", 80); - if (ioc == NULL) return NULL; + } - g_io_channel_write_chars(ioc, query, strlen(query), &len, &gerror); - g_io_channel_flush(ioc, &gerror); - // This function is initiated by g_idle_add. Must return false - // so that it is not called again - return NULL; -} + host = g_match_info_fetch(mi, 1); + appcast = g_match_info_fetch(mi, 2); -static gpointer -check_stable_update(signal_user_data_t *ud) -{ - gchar *query; - gsize len; - GIOChannel *ioc; - GError *gerror = NULL; + if (host == NULL || appcast == NULL) + return NULL; + + query = g_strdup_printf( "GET /%s HTTP/1.0\r\nHost: %s\r\n\r\n", + appcast, host); - g_debug("check_stable_update"); - stable_update_lock = TRUE; - query = "GET /appcast.xml HTTP/1.0\r\nHost: handbrake.fr\r\n\r\n"; - ioc = ghb_net_open(ud, "handbrake.fr", 80); + ioc = ghb_net_open(ud, host, 80); if (ioc == NULL) return NULL; g_io_channel_write_chars(ioc, query, strlen(query), &len, &gerror); g_io_channel_flush(ioc, &gerror); - // This function is initiated by g_idle_add. Must return false - // so that it is not called again + g_free(query); + g_free(host); + g_free(appcast); + g_match_info_free(mi); + g_regex_unref(regex); return NULL; } @@ -3521,7 +3814,6 @@ ghb_notify_done(signal_user_data_t *ud) GtkStatusIcon *si; si = GTK_STATUS_ICON(GHB_OBJECT(ud->builder, "hb_status")); - gtk_status_icon_set_from_icon_name(si, "hb-status-empty"); #if !defined(_WIN32) NotifyNotification *notification;