[FFmpeg-devel] Hello,

[Baptiste Coudurier]

On 07/16/2009 11:02 AM, Daniel Verkamp wrote:
> On Thu, Jul 16, 2009 at 1:53 PM, Jean-Daniel
> Dupas<devlists at shadowlab.org>  wrote:
> [...]
>> And so, to try to get into the code, I did a little patch to improve alias
>> support in dref in mov files on OS X. This patch uses native Mac OS X API to
>> resolve the alias if possible instead of trying to infer the path from
>> opaque data.
> [...]
>
>> @@ -246,7 +251,7 @@
>> {
>>      AVStream *st;
>>      MOVStreamContext *sc;
>> -    int entries, i, j;
>> +    int entries, i;
>>
>>      if (c->fc->nb_streams<  1)
>>          return 0;
>
>> @@ -294,6 +318,7 @@
>>                  if (len&1)
>>                      len += 1;
>>                  if (type == 2) { // absolute path
>> +                    int j;
>>                      av_free(dref->path);
>>                      dref->path = av_mallocz(len+1);
>>                      if (!dref->path)
>
> These hunks are unrelated to the rest of the patch...

I think it is, I think he moved the variable to avoid the unused warning 
which is ok :)

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org