On Fri, Apr 06, 2012 at 05:55:30PM +0300, Martin Storsjö wrote:
> Also add missing trailing commas.
> ---
>  63 files changed, 154 insertions(+), 154 deletions(-)

Nice, you beat me to it :)

> --- a/libavformat/aiffdec.c
> +++ b/libavformat/aiffdec.c
> @@ -328,5 +328,5 @@ AVInputFormat ff_aiff_demuxer = {
>      .read_seek      = ff_pcm_read_seek,
> -    .codec_tag= (const AVCodecTag* const []){ff_codec_aiff_tags, 0},
> +    .codec_tag      = (const AVCodecTag* const []){ff_codec_aiff_tags, 0},

Would you mind adding spaces inside {} while you're changing those
lines anyway?  Same below...

> --- a/libavformat/asfenc.c
> +++ b/libavformat/asfenc.c
> @@ -893,8 +893,8 @@ AVOutputFormat ff_asf_muxer = {
>      .write_trailer  = asf_write_trailer,
> -    .flags = AVFMT_GLOBALHEADER,
> -    .codec_tag= (const AVCodecTag* const []){codec_asf_bmp_tags, 
> ff_codec_bmp_tags, ff_codec_wav_tags, 0},
> +    .flags          = AVFMT_GLOBALHEADER,
> +    .codec_tag      = (const AVCodecTag* const []){codec_asf_bmp_tags, 
> ff_codec_bmp_tags, ff_codec_wav_tags, 0},

Long lines like this could be broken while you're at it.  Same below ...


> --- a/libavformat/mpeg.c
> +++ b/libavformat/mpeg.c
> @@ -614,5 +614,5 @@ AVInputFormat ff_mpegps_demuxer = {
>      .read_header    = mpegps_read_header,
>      .read_packet    = mpegps_read_packet,
>      .read_timestamp = mpegps_read_dts,
> -    .flags = AVFMT_SHOW_IDS|AVFMT_TS_DISCONT,
> +    .flags          = AVFMT_SHOW_IDS|AVFMT_TS_DISCONT,

spaces around | please

> --- a/libavformat/mpegts.c
> +++ b/libavformat/mpegts.c
> @@ -2266,9 +2266,9 @@ AVInputFormat ff_mpegts_demuxer = {
>      .read_timestamp = mpegts_get_pcr,
> -    .flags = AVFMT_SHOW_IDS|AVFMT_TS_DISCONT,
> +    .flags          = AVFMT_SHOW_IDS|AVFMT_TS_DISCONT,
>  
> @@ -2281,9 +2281,9 @@ AVInputFormat ff_mpegtsraw_demuxer = {
>      .read_timestamp = mpegts_get_pcr,
> -    .flags = AVFMT_SHOW_IDS|AVFMT_TS_DISCONT,
> +    .flags          = AVFMT_SHOW_IDS|AVFMT_TS_DISCONT,

same

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to