On Sat, Jul 09, 2011 at 11:20:15AM +0200, Anton Khirnov wrote:
> From: Benjamin Larsson <[email protected]>
> 
> Signed-off-by: Anton Khirnov <[email protected]>
> ---
>  Changelog                |    4 ++
>  doc/general.texi         |    1 +
>  libavformat/Makefile     |    1 +
>  libavformat/allformats.c |    1 +
>  libavformat/wav.c        |   82 
> +++++++++++++++++++++++++++++++++++++++++++++-
>  5 files changed, 88 insertions(+), 1 deletions(-)
> 
> diff --git a/Changelog b/Changelog
> index b785197..afd1b85 100644
> --- a/Changelog
> +++ b/Changelog
> @@ -2,6 +2,10 @@ Entries are sorted chronologically from oldest to youngest 
> within each release,
>  releases are sorted from youngest to oldest.
>  
>  
> +vesion <next>:

veRsion

> +- bwf muxer

BWF

> --- a/libavformat/wav.c
> +++ b/libavformat/wav.c
> @@ -38,7 +40,65 @@ typedef struct {
> +
> +static inline void bwf_write_bext_string(AVFormatContext *s, const char 
> *key, int maxlen)

long line

> +    if (tmp_tag = av_dict_get(s->metadata, "umid", NULL, 0)) {
> +        unsigned char umidpart_str[17] = {0};
> +        int i;
> +        uint64_t umidpart;
> +        int len = strlen(tmp_tag->value+2);
> +
> +        for (i = 0; i < len/16; i++) {
> +            memcpy(umidpart_str, tmp_tag->value + 2 + (i*16), 16);

nit: pointless ()

> --- a/libavformat/wav.c
> +++ b/libavformat/wav.c
> @@ -38,7 +40,65 @@ typedef struct {
>  
> -#if CONFIG_WAV_MUXER
> +#if CONFIG_BWF_MUXER
> +extern AVOutputFormat ff_bwf_muxer;
> +
> @@ -65,6 +125,11 @@ static int wav_write_header(AVFormatContext *s)
>  
> +#if CONFIG_BWF_MUXER
> +    if (s->oformat == &ff_bwf_muxer)
> +        bwf_write_bext_chunk(s);
> +#endif
> +
> @@ -140,6 +207,19 @@ AVOutputFormat ff_wav_muxer = {
>  
> +#if CONFIG_BWF_MUXER
> +AVOutputFormat ff_bwf_muxer = {

The forward declaration is slightly ugly, but probably unavoidable.

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

Reply via email to