[Ffmpeg-devel] [PATCH] Wildcard and catalog image sequences
Michel Bardiaux
mbardiaux
Wed Aug 30 14:00:38 CEST 2006
Michael Niedermayer wrote:
> Hi
>
> On Wed, Aug 30, 2006 at 12:38:30PM +0200, Michael Niedermayer wrote:
> [...]
>>>>> +static void free_catalog(int last_index, char*** pindex_vector)
>>>>> +{
>>>>> + int i;
>>>>> + char** index_vector = *pindex_vector;
>>>>> + for(i=0;index_vector&&i<=last_index;i++)
>>>>> + av_free(index_vector[i]);
>>>>> + av_free(index_vector);
>>>>> + *pindex_vector = NULL;
>>>> av_freep()
>>>> and the index_vector variable seems unneeded
>>> Wilco, but my code was actually correct. It seems to me you *require*
>>> compact code and are ready to reject a patch simply because you find the
>>> coding style too verbose for your taste!
>> yes, code must be simple, patches must be minimal, no superfluous changes
>
> the coding rules also say that:
> "Main priority in FFmpeg is simplicity and small code size (=less bugs). "
> this has been written by fabrice IIRC and was there since a very long time
>
> [...]
I would dispute that a 4-liner instead of a 1-liner using ?: generates
more object code; and it does not seem *simpler* to me. I think you tend
too much towards compactness even at the price of obfuscation. But, OK,
I'll try to respect your wishes.
--
Michel Bardiaux
R&D Director
T +32 [0] 2 790 29 41
F +32 [0] 2 790 29 02
E mailto:mbardiaux at mediaxim.be
Mediaxim NV/SA
Vorstlaan 191 Boulevard du Souverain
Brussel 1160 Bruxelles
http://www.mediaxim.com/
More information about the ffmpeg-devel
mailing list