[FFmpeg-devel] [PATCH] lavf: add SFTP protocol via libssh

James Almer jamrial at gmail.com
Thu Sep 19 18:58:29 CEST 2013

On 19/09/13 1:39 PM, Derek Buitenhuis wrote:
> On 9/18/2013 3:27 PM, Lukasz Marek wrote:
>> +Example: play a file stored on remote server.
> s/play/Play/
>> +#define DEBUG 1
> This looks quite wrong
>> +#include "libavutil/avstring.h"
>> +#include "avformat.h"
>> +#include "internal.h"
>> +#include "url.h"
>> +#include "libavutil/opt.h"
> nit: Order.
>> +#include <libssh/sftp.h>
>> +#include <fcntl.h>
> fcntl.h is not portable. Use the proper guards provided in config.h

There's no configure check for fcntl.h, only for fcntl().
Every file i could find that included that header did it without guards.

> D and E are far to short and potentially pollute and conflict
> with the namespacing of system headers.

options_table.h defines E and D the same way, so i don't think it will 
be a problem.

More information about the ffmpeg-devel mailing list