[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