diff options
author | jstebbins <[email protected]> | 2014-08-15 21:47:15 +0000 |
---|---|---|
committer | jstebbins <[email protected]> | 2014-08-15 21:47:15 +0000 |
commit | f0913ae0e27b0aa302666b629a2a3f566f6e79b3 (patch) | |
tree | b0edbc9ee498d40c9c16b3f165252466920c6cbe | |
parent | 4f016b6cc934df9ac86dc7633acacbb7cbe4cf0c (diff) |
LinGui: remove video qualite range warning
It just causes confusion
git-svn-id: svn://svn.handbrake.fr/HandBrake/trunk@6297 b64f7644-9d1e-0410-96f1-a4d463321fa5
-rw-r--r-- | gtk/src/hb-backend.c | 77 | ||||
-rw-r--r-- | gtk/src/queuehandler.c | 5 |
2 files changed, 1 insertions, 81 deletions
diff --git a/gtk/src/hb-backend.c b/gtk/src/hb-backend.c index 95266122a..55d9abc3b 100644 --- a/gtk/src/hb-backend.c +++ b/gtk/src/hb-backend.c @@ -694,6 +694,7 @@ ghb_vquality_default(signal_user_data_t *ud) switch (vcodec) { + case HB_VCODEC_X265: case HB_VCODEC_X264: return 20; case HB_VCODEC_THEORA: @@ -4164,82 +4165,6 @@ ghb_validate_audio(GValue *settings) return TRUE; } -gboolean -ghb_validate_vquality(GValue *settings) -{ - gint vcodec; - gchar *message; - gint min, max; - - vcodec = ghb_settings_video_encoder_codec(settings, "VideoEncoder"); - gdouble vquality; - vquality = ghb_settings_get_double(settings, "VideoQualitySlider"); - if (ghb_settings_get_boolean(settings, "vquality_type_constant")) - { - switch (vcodec) - { - case HB_VCODEC_X264: - { - min = 16; - max = 30; - } break; - - case HB_VCODEC_FFMPEG_MPEG2: - case HB_VCODEC_FFMPEG_MPEG4: - { - min = 1; - max = 8; - } break; - - case HB_VCODEC_THEORA: - case HB_VCODEC_FFMPEG_VP8: - { - min = 0; - max = 63; - } break; - - default: - { - min = 48; - max = 62; - } break; - } - if (vcodec == HB_VCODEC_X264 && vquality == 0.0) - { - message = g_strdup_printf( - _("Warning: lossless h.264 selected\n\n" - "Lossless h.264 is not well supported by\n" - "many players and editors.\n\n" - "It will produce enormous output files.\n\n" - "Are you sure you wish to use this setting?")); - if (!ghb_message_dialog(GTK_MESSAGE_QUESTION, message, - _("Cancel"), _("Continue"))) - { - g_free(message); - return FALSE; - } - g_free(message); - ghb_settings_set_string(settings, "VideoProfile", "auto"); - } - else if (vquality < min || vquality > max) - { - message = g_strdup_printf( - _("Interesting video quality choice: %d\n\n" - "Typical values range from %d to %d.\n\n" - "Are you sure you wish to use this setting?"), - (gint)vquality, min, max); - if (!ghb_message_dialog(GTK_MESSAGE_QUESTION, message, - _("Cancel"), _("Continue"))) - { - g_free(message); - return FALSE; - } - g_free(message); - } - } - return TRUE; -} - static void add_job(hb_handle_t *h, GValue *js, gint unique_id, int titleindex) { diff --git a/gtk/src/queuehandler.c b/gtk/src/queuehandler.c index f2d40d3eb..b7e68ce2d 100644 --- a/gtk/src/queuehandler.c +++ b/gtk/src/queuehandler.c @@ -898,11 +898,6 @@ validate_settings(signal_user_data_t *ud, GValue *settings, gint batch) g_unlink(dest); } g_free(dest); - // Validate video quality is in a reasonable range - if (!ghb_validate_vquality(settings)) - { - return FALSE; - } // Validate audio settings if (!ghb_validate_audio(settings)) { |