[FFmpeg-devel] [PATCH 4/5] audioconvert: use + in av_get_channel_layout_string.

Michael Niedermayer michaelni at gmx.at
Sun Nov 6 01:40:45 CET 2011


On Sun, Nov 06, 2011 at 01:26:28AM +0100, Stefano Sabatini wrote:
> On date Saturday 2011-11-05 21:21:50 +0100, Nicolas George encoded:
> > When constructing a channel layout string from individual channel names,
> > + is a more practical separator than |, because it is usually not a shell
> > special character.
> > 
> > Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> > ---
> >  libavutil/audioconvert.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/libavutil/audioconvert.c b/libavutil/audioconvert.c
> > index 828780c..9d236f3 100644
> > --- a/libavutil/audioconvert.c
> > +++ b/libavutil/audioconvert.c
> > @@ -143,7 +143,7 @@ void av_get_channel_layout_string(char *buf, int buf_size,
> >                  const char *name = get_channel_name(i);
> >                  if (name) {
> >                      if (ch > 0)
> > -                        av_strlcat(buf, "|", buf_size);
> > +                        av_strlcat(buf, "+", buf_size);
> >                      av_strlcat(buf, name, buf_size);
> >                  }
> >                  ch++;
> 
> LFTM.

F=Fine ?
i assume so, thus applied, thx

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

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- 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/20111106/b164ecab/attachment.asc>


More information about the ffmpeg-devel mailing list