diff options
author | dynaflash <[email protected]> | 2007-09-13 19:28:19 +0000 |
---|---|---|
committer | dynaflash <[email protected]> | 2007-09-13 19:28:19 +0000 |
commit | bfcd060f9b555f88b01086331fb79d0ec1b8e1e2 (patch) | |
tree | 7148452f5dc8f69316e84b4aaec98c48ad1abea4 /libhb | |
parent | 59da0d1afd24ff1f72664ed30023f07e01e5b07d (diff) |
Reverts an errant checking for from rev 960
git-svn-id: svn://svn.handbrake.fr/HandBrake/trunk@961 b64f7644-9d1e-0410-96f1-a4d463321fa5
Diffstat (limited to 'libhb')
-rw-r--r-- | libhb/hb.c | 119 | ||||
-rw-r--r-- | libhb/hb.h | 2 | ||||
-rw-r--r-- | libhb/work.c | 70 |
3 files changed, 53 insertions, 138 deletions
diff --git a/libhb/hb.c b/libhb/hb.c index b2d672872..a53bd1f4a 100644 --- a/libhb/hb.c +++ b/libhb/hb.c @@ -431,90 +431,75 @@ void hb_get_preview( hb_handle_t * h, hb_title_t * title, int picture, * @param job Handle to hb_job_t. * @param aspect Desired aspect ratio. Value of -1 uses title aspect. * @param pixels Maximum desired pixel count. - * @param par If true, skip the cropping and keep-aspect-ratio stuff. */ -void hb_set_size( hb_job_t * job, int aspect, int pixels, int par ) +void hb_set_size( hb_job_t * job, int aspect, int pixels ) { hb_title_t * title = job->title; int croppedWidth = title->width - title->crop[2] - title->crop[3]; int croppedHeight = title->height - title->crop[0] - title->crop[1]; - - int croppedAspect; - if (par) + int croppedAspect = title->aspect * title->height * croppedWidth / + croppedHeight / title->width; + int addCrop; + int i, w, h; + + if( aspect <= 0 ) { - /* That means we're setting size for video with non-square pixels */ - croppedAspect = croppedWidth * HB_ASPECT_BASE / croppedHeight; + /* Keep the best possible aspect ratio */ aspect = croppedAspect; } - else - { - /* We're free to change cropping and maintain display aspect ratio */ - croppedAspect = title->aspect * title->height * croppedWidth / - croppedHeight / title->width; - - int addCrop; - if( aspect <= 0 ) + /* Crop if necessary to obtain the desired ratio */ + memcpy( job->crop, title->crop, 4 * sizeof( int ) ); + if( aspect < croppedAspect ) + { + /* Need to crop on the left and right */ + addCrop = croppedWidth - aspect * croppedHeight * title->width / + title->aspect / title->height; + if( addCrop & 3 ) { - /* Keep the best possible aspect ratio */ - aspect = croppedAspect; + addCrop = ( addCrop + 1 ) / 2; + job->crop[2] += addCrop; + job->crop[3] += addCrop; } - - /* Crop if necessary to obtain the desired ratio */ - memcpy( job->crop, title->crop, 4 * sizeof( int ) ); - if( aspect < croppedAspect ) + else if( addCrop & 2 ) { - /* Need to crop on the left and right */ - addCrop = croppedWidth - aspect * croppedHeight * title->width / - title->aspect / title->height; - if( addCrop & 3 ) - { - addCrop = ( addCrop + 1 ) / 2; - job->crop[2] += addCrop; - job->crop[3] += addCrop; - } - else if( addCrop & 2 ) - { - addCrop /= 2; - job->crop[2] += addCrop - 1; - job->crop[3] += addCrop + 1; - } - else - { - addCrop /= 2; - job->crop[2] += addCrop; - job->crop[3] += addCrop; - } + addCrop /= 2; + job->crop[2] += addCrop - 1; + job->crop[3] += addCrop + 1; } - else if( aspect > croppedAspect ) + else { - /* Need to crop on the top and bottom */ - addCrop = croppedHeight - croppedWidth * title->aspect * - title->height / aspect / title->width; - if( addCrop & 3 ) - { - addCrop = ( addCrop + 1 ) / 2; - job->crop[0] += addCrop; - job->crop[1] += addCrop; - } - else if( addCrop & 2 ) - { - addCrop /= 2; - job->crop[0] += addCrop - 1; - job->crop[1] += addCrop + 1; - } - else - { - addCrop /= 2; - job->crop[0] += addCrop; - job->crop[1] += addCrop; - } + addCrop /= 2; + job->crop[2] += addCrop; + job->crop[3] += addCrop; } - } - - int i, w, h; + else if( aspect > croppedAspect ) + { + /* Need to crop on the top and bottom */ + addCrop = croppedHeight - croppedWidth * title->aspect * + title->height / aspect / title->width; + if( addCrop & 3 ) + { + addCrop = ( addCrop + 1 ) / 2; + job->crop[0] += addCrop; + job->crop[1] += addCrop; + } + else if( addCrop & 2 ) + { + addCrop /= 2; + job->crop[0] += addCrop - 1; + job->crop[1] += addCrop + 1; + } + else + { + addCrop /= 2; + job->crop[0] += addCrop; + job->crop[1] += addCrop; + } + } + /* Compute a resolution from the number of pixels and aspect */ for( i = 0;; i++ ) { diff --git a/libhb/hb.h b/libhb/hb.h index c33f70f62..6f8af2aee 100644 --- a/libhb/hb.h +++ b/libhb/hb.h @@ -77,7 +77,7 @@ hb_list_t * hb_get_titles( hb_handle_t * ); void hb_get_preview( hb_handle_t *, hb_title_t *, int, uint8_t * ); -void hb_set_size( hb_job_t *, int ratio, int pixels, int par ); +void hb_set_size( hb_job_t *, int ratio, int pixels ); /* Handling jobs */ int hb_count( hb_handle_t * ); diff --git a/libhb/work.c b/libhb/work.c index 70ca36a1e..5b3f914c2 100644 --- a/libhb/work.c +++ b/libhb/work.c @@ -128,76 +128,6 @@ static void do_job( hb_job_t * job, int cpu_count ) job->height=title->height-job->crop[0]-job->crop[1]; job->width=title->width-job->crop[2]-job->crop[3]; } - else if ( job->pixel_ratio == 2 ) - { - /* "Loose" anamorphic. - - Uses mod16-compliant dimensions, - - Allows users to set the width - - Handles ITU pixel aspects - */ - /* Use mod16 values for better compression */ - job->width = MULTIPLE_16( job->width ); - job->height = MULTIPLE_16( job->height ); - job->width = MAX( 16, job->width ); - job->height = MAX( 16, job->height ); - - /* Set up some variables to make the math easier to follow. */ - int cropped_width = title->width - job->crop[2] - job->crop[3] ; - int cropped_height = title->height - job->crop[0] - job->crop[1] ; - int storage_aspect = cropped_width * HB_ASPECT_BASE / cropped_height; - int pixels = job->width * (job->width / storage_aspect * HB_ASPECT_BASE); - - /* While keeping the DVD storage aspect, resize the job width and height - so they fit into the user's specified dimensions. */ - hb_set_size(job, -1, pixels, 1); - - if (cropped_width <= 706) - { - /* Handle ITU PARs */ - if (title->height == 480) - { - /* It's NTSC */ - if (title->aspect == 16) - { - /* It's widescreen */ - job->pixel_aspect_width = 40; - job->pixel_aspect_height = 33; - } - else - { - /*It's 4:3 */ - job->pixel_aspect_width = 10; - job->pixel_aspect_height = 11; - } - } - else if (title->height == 576) - { - /* It's PAL */ - if(title->aspect == 16) - { - /* It's widescreen */ - job->pixel_aspect_width = 16; - job->pixel_aspect_height = 11; - } - else - { - /* It's 4:3 */ - job->pixel_aspect_width = 12; - job->pixel_aspect_height = 11; - } - } - } - - /* Figure out what dimensions the source would display at. */ - int source_display_width = cropped_width * ((float)job->pixel_aspect_width / (float)job->pixel_aspect_height) ; - - /* The film AR is the source's display width / cropped source height. - The output display width is the output height * film AR. - The output PAR is the output display width / output storage width. */ - job->pixel_aspect_width = job->height * source_display_width / cropped_height; - job->pixel_aspect_height = job->width; - - } /* Keep width and height within these boundaries */ if (job->maxHeight && (job->height > job->maxHeight) ) |