[FFmpeg-devel] [PATCH 1/6] lavf: add directory listing API

[Lukasz Marek]

On 05.07.2014 18:48, wm4 wrote:
> On Sat,  5 Jul 2014 18:11:59 +0200
> Lukasz Marek <lukasz.m.luki2 at gmail.com> wrote:
>> + */
>> +int avio_open_dir(void **s, const char *url, AVDictionary **options);
>> +
>> +/**
>> + * Get next directory entry.
>> + *
>> + * Returned entry must be freed with avio_free_directory_entry().
>> + *
>> + * @param s         directory read context.
>> + * @param[out] next next entry or NULL when no more entries.
>
> Doesn't explain the lifetime of the AVIODirEntry object.
>
> Also, is no more entries a success?

I changed it a bit. But I left returning NULL on end and no error.
Rest of your remarks should be fixed.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavf-add-directory-listing-API.patch
Type: text/x-patch
Size: 6176 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140707/2cda08a1/attachment.bin>