summaryrefslogtreecommitdiffstats
path: root/gtk
diff options
context:
space:
mode:
authorjstebbins <[email protected]>2014-08-15 21:47:15 +0000
committerjstebbins <[email protected]>2014-08-15 21:47:15 +0000
commitf0913ae0e27b0aa302666b629a2a3f566f6e79b3 (patch)
treeb0edbc9ee498d40c9c16b3f165252466920c6cbe /gtk
parent4f016b6cc934df9ac86dc7633acacbb7cbe4cf0c (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
Diffstat (limited to 'gtk')
-rw-r--r--gtk/src/hb-backend.c77
-rw-r--r--gtk/src/queuehandler.c5
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))
{