[vlc-devel] [PATCH 1/2] include: use vlc_rational_t for display zoom

Steve Lhomme robux4 at ycbcr.xyz
Tue Feb 9 16:35:31 UTC 2021


yes

On 2021-02-09 17:20, Alexandre Janniaux wrote:
> Hi,
> 
> Patchset LGTM.
> 
> Regards,
> --
> Alexandre Janniaux
> Videolabs
> 
> On Tue, Feb 09, 2021 at 06:13:48PM +0200, remi at remlab.net wrote:
>> From: RĂ©mi Denis-Courmont <remi at remlab.net>
>>
>> ---
>>   include/vlc_vout_display.h | 5 +----
>>   1 file changed, 1 insertion(+), 4 deletions(-)
>>
>> diff --git a/include/vlc_vout_display.h b/include/vlc_vout_display.h
>> index 58a0c2e430..36018a9542 100644
>> --- a/include/vlc_vout_display.h
>> +++ b/include/vlc_vout_display.h
>> @@ -119,10 +119,7 @@ typedef struct vout_display_cfg {
>>       bool is_display_filled;
>>
>>       /** Zoom ratio */
>> -    struct {
>> -        unsigned num;
>> -        unsigned den;
>> -    } zoom;
>> +    vlc_rational_t zoom;
>>
>>       vlc_viewpoint_t viewpoint;
>>   } vout_display_cfg_t;
>> --
>> 2.30.0
>>
>> _______________________________________________
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
>> https://mailman.videolan.org/listinfo/vlc-devel
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
> 


More information about the vlc-devel mailing list