[FFmpeg-devel] [PATCH 3/3] lavf/mxfdec: delete trailing semi-colon

Michael Niedermayer michaelni at gmx.at
Fri Apr 4 02:30:18 CEST 2014


On Fri, Apr 04, 2014 at 12:46:28AM +0200, James Darnley wrote:
> ---
>  libavformat/mxfdec.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> index 2abd34d..0d17a02 100644
> --- a/libavformat/mxfdec.c
> +++ b/libavformat/mxfdec.c
> @@ -508,7 +508,7 @@ static int mxf_read_partition_pack(void *arg, AVIOContext *pb, int tag, int size
>          }
>          /* if no previous body partition are found point to the header
>           * partition */
> -        if (partition->previous_partition == partition->this_partition);
> +        if (partition->previous_partition == partition->this_partition)
>              partition->previous_partition = 0;
>          av_log(mxf->fc, AV_LOG_ERROR,
>                 "Overriding PreviousPartition with %"PRIx64"\n",

ive merged a variant of this

It should still be double checked though if its correct now, also
if someone has a testcase for which this makes a difference, please
post it or a link to it also a testcase for what the original case
fixed would be welcome too

Ive asked the aleged reporter of the original issue a while ago
but never got a reply, so possibly he didnt remember the exact
testcase

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

What does censorship reveal? It reveals fear. -- Julian Assange
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140404/ca72b0b2/attachment.asc>


More information about the ffmpeg-devel mailing list