[CinCV TNG] [PATCH] New file cinelerra.h

Einar Rünkaru einarrunkaru at gmail.com
Thu Aug 24 22:15:40 CEST 2017



On 08/23/2017 11:22 PM, Johannes Sixt wrote:
> Am 23.08.2017 um 19:14 schrieb Einar Rünkaru:
>> Contains program-wide limits.
>> Limits will be used to check user input values
>> ---
>>   cinelerra/Makefile.am |  1 +
>>   cinelerra/cinelerra.h | 52 
>> +++++++++++++++++++++++++++++++++++++++++++++++++++
>>   2 files changed, 53 insertions(+)
>>   create mode 100644 cinelerra/cinelerra.h
> 
> Hm, these values are nowhere used? Can we have a patch that shows how 
> the values are going to be used?

The patch will come.
> 
> Why do you give the file such a generic name, cinelerra.h? This is 
> certainly not suited when the purpose is "contains program-wide limits". 
> Then you should better name it "cinelerralimits.h".

Not all are limits. Simply this is the place where cinelerra-wide 
constats can be put. I prefer shorter name.

Einar


More information about the Cinelerra mailing list