[CinCV TNG] [PATCH] Change the type of sample_rate to int

Einar Rünkaru einarrunkaru at gmail.com
Sun Aug 27 17:37:11 CEST 2017



On 08/26/2017 11:51 AM, Johannes Sixt wrote:
> Am 26.08.2017 um 10:34 schrieb Einar Rünkaru:
>> Hi.
>>
>> We can assume that int is at least 32 bits long and
>> it is the optimal variable type on the current arch.
>> There is no need to use fixed length variable type,
>> when we dont care about the length and all possible values
>> of the variable fit to int.
> 
> Sure, but do we now convert the value to other types, e.g. int64_t, 
> implicitly? That is also something to consider!
> 

sample_rate in asset has been int at least last 10 years. Probably we 
got rid of some implict conversions.

The upcomig changes need that sample_rate everywhere is the same type.

Einar


More information about the Cinelerra mailing list