[vlc-devel] Re: [patch] Fixed source aspect ratio calculation

[Nico Sabbi]

Derk-Jan Hartman wrote:

> On 17 okt 2005, at 10:49, Marian Durkovic wrote:
>
>> Hi all,
>>
>>   attached is the patch which fixes source aspect ratio calculation.
>> SAR needs to be calculated from visible image size, not from  encoded 
>> size.
>> With this patch, HDTV streams have proper SAR of 1:1 (square  pixels) as
>> per HDTV definition.
>
>
>
> I have one issue here:
>
> +        if( p_dec->fmt_out.video.i_visible_height == 1088 )
> +        {
> +            p_dec->fmt_out.video.i_visible_height = 1080;
> +            msg_Warn( p_dec, "Incorrect HDTV stream - reducing  
> height to 1080");
> +        }
>
>
> Now... what if i actually want to send a 1088 stream?
>
> DJ
>

isn't it better to resort to the x and y settings defined in the 
sequence_display_extension
(I don't remember the names atm), rather than hardcoding those values?
 
 
 --
 Email.it, the professional e-mail, gratis per te: http://www.email.it/f
 
 Sponsor:
 Sono aperte le iscrizioni per il corso Nazionale di Web Marketing
* Imparerai come promuoverti sui motori di ricerca - clicca qui
 Clicca qui: http://adv.email.it/cgi-bin/foclick.cgi?mid=3215&d=17-10

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html