diff options
author | jstebbins <[email protected]> | 2012-05-05 18:22:37 +0000 |
---|---|---|
committer | jstebbins <[email protected]> | 2012-05-05 18:22:37 +0000 |
commit | 06254850dc2c1ec43d57f4d8e2026fa67967d5b9 (patch) | |
tree | c7ff6a64e1310d79f4f17b1a6bfe9aebaf88a958 /gtk | |
parent | 1c8309d77ea303954f8c9e3919c595de54f173f2 (diff) |
libhb: Fix problem with positions of burned-in subs
When cropping and/or scaling, the position of burned-in SSA subs was broken.
Also, when scaling, the position of all other burned-in subs was broken.
Part of this fix is to revert a bit of filter initialization changes that
were made in this commit https://trac.handbrake.fr/changeset/4605
The other part of the fix is to add cropping parameters to the initialization
of the subtitle rendering filter. This filter needs the *original* title
dimensions and the crop values in order to compute the positions properly.
The changes that I am reverting gave it the scaled job dimensions and the
crop values. This was wrong in so many ways it's embarassing :p
git-svn-id: svn://svn.handbrake.fr/HandBrake/trunk@4642 b64f7644-9d1e-0410-96f1-a4d463321fa5
Diffstat (limited to 'gtk')
-rw-r--r-- | gtk/src/hb-backend.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/gtk/src/hb-backend.c b/gtk/src/hb-backend.c index 1c9d3f924..f104850e7 100644 --- a/gtk/src/hb-backend.c +++ b/gtk/src/hb-backend.c @@ -5081,7 +5081,10 @@ add_job(hb_handle_t *h, GValue *js, gint unique_id, gint titleindex) { // Add filter that renders vobsubs filter = hb_filter_init(HB_FILTER_RENDER_SUB); - hb_add_filter( job, filter, NULL ); + filter_str = g_strdup_printf("%d:%d:%d:%d", + crop[0], crop[1], crop[2], crop[3]); + hb_add_filter( job, filter, filter_str ); + g_free(filter_str); } |