[FFmpeg-devel] [PATCH 4/6] lavu/avstring: add av_append_path_component() funcion
lukasz.m.luki2 at gmail.com
Mon Jul 7 22:48:12 CEST 2014
On 05.07.2014 18:52, wm4 wrote:
> On Sat, 5 Jul 2014 18:12:02 +0200
> Lukasz Marek <lukasz.m.luki2 at gmail.com> wrote:
>> TODO: bump minor, update doc/APIchanges
>> Convinient function to build paths.
>> Signed-off-by: Lukasz Marek <lukasz.m.luki2 at gmail.com>
>> libavutil/avstring.c | 43 +++++++++++++++++++++++++++++++++++++++++++
>> libavutil/avstring.h | 10 ++++++++++
>> 2 files changed, 53 insertions(+)
>> + * Append path component to the existing path.
>> + * Path separator '/' is placed between when needed.
>> + * Resulting string have to be freed with av_free().
>> + * @param path base path
>> + * @param component component to be appended
>> + * @return new path or NULL on error.
>> + */
>> +char *av_append_path_component(const char *path, const char *component);
> IMO the name is awkward. Maybe av_path_join()?
> Did you have a look at Python's os.path.join()? IMO it has pretty good
> and well documented semantics, which you could steal.
I had look at it after your comment. I don't know it is better. It seems
a bit complicated for a simple needs of ffmpeg.
More information about the ffmpeg-devel