[FFmpeg-devel] [PATCH 1/3] lavf/file: file_check: Handle paths that start with file:

Alexander Strasser eclipse7 at gmx.net
Sun Jan 5 00:21:24 CET 2014


On 2014-01-04 14:53 +0100, Stefano Sabatini wrote:
> On date Friday 2014-01-03 21:53:02 +0100, Alexander Strasser encoded:
[...]
> >   For your other remark about the {}, the #else part still declares a
> > variable. This avoids having statements and declarations mixed in a
> > block after my patch. I did not add them into the #else part because
> > this way it will need no changes if variables are added again in the
> > #if part.

  I found an error I had in the #else part. I moved the variable ret
into the parent scope but forgot to adopt the #else part.

> I think the patch is fine as is once you fix/extend the commit
> message, feel free to push if there are no further comments. Thanks.

  Fixed & pushed.

Thanks,
  Alexander
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140105/258a0ced/attachment.asc>


More information about the ffmpeg-devel mailing list