[osx] drop LPCM mode from CDVDAudioCodecFFmpeg - it's not wanted or needed
authorJonathan Marshall <jmarshall@xbmc.org>
Sun, 2 Feb 2014 06:26:22 +0000 (19:26 +1300)
committerJonathan Marshall <jmarshall@xbmc.org>
Fri, 21 Feb 2014 20:04:48 +0000 (09:04 +1300)
xbmc/cores/dvdplayer/DVDCodecs/Audio/DVDAudioCodecFFmpeg.cpp
xbmc/cores/dvdplayer/DVDCodecs/Audio/DVDAudioCodecFFmpeg.h

index 2453c65..af897fe 100644 (file)
@@ -45,8 +45,6 @@ CDVDAudioCodecFFmpeg::CDVDAudioCodecFFmpeg() : CDVDAudioCodec()
   m_channels = 0;
   m_layout = 0;
   
-  m_bLpcmMode = false;
-
   m_pFrame1 = NULL;
   m_iSampleFormat = AV_SAMPLE_FMT_NONE;
 }
@@ -73,11 +71,6 @@ bool CDVDAudioCodecFFmpeg::Open(CDVDStreamInfo &hints, CDVDCodecOptions &options
     return false;
   }
 
-#if defined(TARGET_DARWIN_OSX)
-  if (CSettings::Get().GetInt("audiooutput.channels") >  AE_CH_LAYOUT_2_0)
-    m_bLpcmMode = true;
-#endif
-
   m_pCodecContext = m_dllAvCodec.avcodec_alloc_context3(pCodec);
   m_pCodecContext->debug_mv = 0;
   m_pCodecContext->debug = 0;
@@ -198,7 +191,7 @@ int CDVDAudioCodecFFmpeg::Decode(uint8_t* pData, int iSize)
     default:
       convert = true;
   }
-  if(m_bLpcmMode || convert)
+  if(convert)
     ConvertToFloat();
 
   return iBytesUsed;
@@ -310,25 +303,18 @@ int CDVDAudioCodecFFmpeg::GetEncodedSampleRate()
 
 enum AEDataFormat CDVDAudioCodecFFmpeg::GetDataFormat()
 {
-  if(m_bLpcmMode)
-  {
-    return AE_FMT_LPCM;
-  }
-  else
+  switch(m_pCodecContext->sample_fmt)
   {
-    switch(m_pCodecContext->sample_fmt)
-    {
-      case AV_SAMPLE_FMT_U8 : return AE_FMT_U8;
-      case AV_SAMPLE_FMT_S16: return AE_FMT_S16NE;
-      case AV_SAMPLE_FMT_S32: return AE_FMT_S32NE;
-      case AV_SAMPLE_FMT_FLT: return AE_FMT_FLOAT;
-      case AV_SAMPLE_FMT_DBL: return AE_FMT_DOUBLE;
-      case AV_SAMPLE_FMT_NONE:
-        CLog::Log(LOGERROR, "CDVDAudioCodecFFmpeg::GetDataFormat - invalid data format");
-        return AE_FMT_INVALID;
-      default:
-        return AE_FMT_FLOAT;
-    }
+    case AV_SAMPLE_FMT_U8 : return AE_FMT_U8;
+    case AV_SAMPLE_FMT_S16: return AE_FMT_S16NE;
+    case AV_SAMPLE_FMT_S32: return AE_FMT_S32NE;
+    case AV_SAMPLE_FMT_FLT: return AE_FMT_FLOAT;
+    case AV_SAMPLE_FMT_DBL: return AE_FMT_DOUBLE;
+    case AV_SAMPLE_FMT_NONE:
+      CLog::Log(LOGERROR, "CDVDAudioCodecFFmpeg::GetDataFormat - invalid data format");
+      return AE_FMT_INVALID;
+    default:
+      return AE_FMT_FLOAT;
   }
 }
 
index ab461ab..1646e00 100644 (file)
@@ -50,7 +50,6 @@ protected:
   SwrContext*         m_pConvert;
   enum AVSampleFormat m_iSampleFormat;  
   CAEChannelInfo      m_channelLayout;
-  bool                m_bLpcmMode;
 
   AVFrame* m_pFrame1;
   int      m_iBufferSize1;