[FFmpeg-cvslog] mp3enc: clarify meaning of "Unsupported" messages.

Reimar Döffinger git at videolan.org
Sat Apr 14 18:22:03 CEST 2012


ffmpeg | branch: master | Reimar Döffinger <Reimar.Doeffinger at gmx.de> | Sat Apr 14 00:47:53 2012 +0200| [4c53c66a21f18b2633bfd4894623f163258b2eee] | committer: Reimar Döffinger

mp3enc: clarify meaning of "Unsupported" messages.

They will only cause us to skip writing the Xing header,
not cause any serious breakage.
Related to trac issue #1027.

Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=4c53c66a21f18b2633bfd4894623f163258b2eee
---

 libavformat/mp3enc.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavformat/mp3enc.c b/libavformat/mp3enc.c
index 22810b5..29c0780 100644
--- a/libavformat/mp3enc.c
+++ b/libavformat/mp3enc.c
@@ -131,14 +131,14 @@ static int mp3_write_xing(AVFormatContext *s)
             break;
         }
     if (i == FF_ARRAY_ELEMS(avpriv_mpa_freq_tab)) {
-        av_log(s, AV_LOG_ERROR, "Unsupported sample rate.\n");
+        av_log(s, AV_LOG_WARNING, "Unsupported sample rate, not writing Xing header.\n");
         return -1;
     }
 
     switch (codec->channels) {
     case 1:  channels = MPA_MONO;                                          break;
     case 2:  channels = MPA_STEREO;                                        break;
-    default: av_log(s, AV_LOG_ERROR, "Unsupported number of channels.\n"); return -1;
+    default: av_log(s, AV_LOG_WARNING, "Unsupported number of channels, not writing Xing header.\n"); return -1;
     }
 
     /* dummy MPEG audio header */



More information about the ffmpeg-cvslog mailing list