fix AS-02 audio unwrapping behavior

diff --git a/src/AS_02.h b/src/AS_02.h
index ff6b692525a433c5143c2dcfdb4240ffb840f55f..e534dd4564477feb76bb0fdd71d8106865ffdf2f 100644 (file)
--- a/src/AS_02.h
+++ b/src/AS_02.h
@@ -118,6 +118,14 @@ namespace AS_02
       return CalcSampleSize(d) * CalcSamplesPerFrame(d, edit_rate);
     }
 
+    // Returns number of frames for data described by ADesc, given a duration in samples and an edit rate
+    inline ui32_t CalcFramesFromDurationInSamples(const ui32_t durationInSamples, const ASDCP::MXF::WaveAudioDescriptor& d, const ASDCP::Rational& edit_rate)
+    {
+      return static_cast<ui32_t>(static_cast<ui64_t>(durationInSamples) *
+                                static_cast<ui64_t>(d.AudioSamplingRate.Denominator * edit_rate.Numerator) /
+                                static_cast<ui64_t>(d.AudioSamplingRate.Numerator * edit_rate.Denominator));
+    }
+
   } // namespace MXF
 
   //---------------------------------------------------------------------------------

diff --git a/src/AS_02_PCM.cpp b/src/AS_02_PCM.cpp
index 2a63aa3b9243e5cc66277f24097c7cb0d4f8e8a0..512bcdb0b9cd365972ca892624ab356084aa98ef 100644 (file)
--- a/src/AS_02_PCM.cpp
+++ b/src/AS_02_PCM.cpp
@@ -144,6 +144,8 @@ ASDCP::Result_t
 AS_02::PCM::MXFReader::h__Reader::ReadFrame(ui32_t FrameNum, ASDCP::PCM::FrameBuffer& FrameBuf,
                                            ASDCP::AESDecContext* Ctx, ASDCP::HMACContext* HMAC)
 {
+  ASDCP::MXF::WaveAudioDescriptor* wave_descriptor = 0;
+
   if ( ! m_File.IsOpen() )
     {
       return RESULT_INIT;
@@ -156,7 +158,21 @@ AS_02::PCM::MXFReader::h__Reader::ReadFrame(ui32_t FrameNum, ASDCP::PCM::FrameBu
 
   assert(m_ClipEssenceBegin);
   Result_t result = RESULT_OK;
-  ui64_t position = m_ClipEssenceBegin + ( FrameNum * m_SamplesPerFrame );
+
+  if( KM_SUCCESS(result) )
+    {
+      if ( KM_SUCCESS(m_HeaderPart.GetMDObjectByType(OBJ_TYPE_ARGS(WaveAudioDescriptor),
+                                                    reinterpret_cast<InterchangeObject**>(&wave_descriptor))) )
+       {
+         if ( wave_descriptor == 0 )
+           {
+             DefaultLogSink().Error("WaveAudioDescriptor object not found.\n");
+             return RESULT_AS02_FORMAT;
+           }
+       }
+    }
+
+  ui64_t position = m_ClipEssenceBegin + ( FrameNum * m_SamplesPerFrame * wave_descriptor->ChannelCount * wave_descriptor->BlockAlign );
 
   if ( m_File.Tell() != position )
     {
@@ -165,12 +181,12 @@ AS_02::PCM::MXFReader::h__Reader::ReadFrame(ui32_t FrameNum, ASDCP::PCM::FrameBu
 
   if ( KM_SUCCESS(result) )
     {
-      result = m_File.Read(FrameBuf.Data(), m_SamplesPerFrame);
+      result = m_File.Read(FrameBuf.Data(), m_SamplesPerFrame * wave_descriptor->ChannelCount * wave_descriptor->BlockAlign);
     }
 
   if ( KM_SUCCESS(result) )
     {
-      FrameBuf.Size(m_SamplesPerFrame);
+      FrameBuf.Size(m_SamplesPerFrame * wave_descriptor->ChannelCount * wave_descriptor->BlockAlign);
     }
 
   return result;
@@ -394,8 +410,6 @@ AS_02::PCM::MXFWriter::h__Writer::SetSourceStream(const ASDCP::Rational& edit_ra
       return RESULT_STATE;
     }
 
-  fprintf(stderr, "edit_rate=%d/%d\n", edit_rate.Numerator, edit_rate.Denominator);
-
   memcpy(m_EssenceUL, m_Dict->ul(MDD_WAVEssenceClip), SMPTE_UL_LENGTH);
   m_EssenceUL[15] = 1; // set the stream identifier
   Result_t result = m_State.Goto_READY();
@@ -407,8 +421,6 @@ AS_02::PCM::MXFWriter::h__Writer::SetSourceStream(const ASDCP::Rational& edit_ra
       m_SamplesPerFrame = AS_02::MXF::CalcSamplesPerFrame(*m_WaveAudioDescriptor, edit_rate);
       m_WaveAudioDescriptor->ContainerDuration = 0;
 
-      fprintf(stderr, "m_BytesPerFrame=%d, m_SamplesPerFrame=%d\n", m_BytesPerFrame, m_SamplesPerFrame);
-
       result = WriteAS02Header(PCM_PACKAGE_LABEL, UL(m_Dict->ul(MDD_WAVWrapping)),
                               SOUND_DEF_LABEL, UL(m_EssenceUL), UL(m_Dict->ul(MDD_SoundDataDef)),
                               m_EssenceDescriptor->SampleRate, derive_timecode_rate_from_edit_rate(edit_rate), m_BytesPerFrame);
@@ -475,7 +487,6 @@ AS_02::PCM::MXFWriter::h__Writer::Finalize()
 
   if ( KM_SUCCESS(result) )
     {
-      fprintf(stderr, "m_FramesWritten=%d, m_SamplesPerFrame=%d\n", m_FramesWritten, m_SamplesPerFrame);
       m_FramesWritten = m_FramesWritten * m_SamplesPerFrame;
       WriteAS02Footer();
     }

diff --git a/src/as-02-unwrap.cpp b/src/as-02-unwrap.cpp
index 42d48205f292f8fe0a6a3d1697b74da7b0914147..b85781d065ac9ef08116eaf4b1127b652345d274 100755 (executable)
--- a/src/as-02-unwrap.cpp
+++ b/src/as-02-unwrap.cpp
@@ -453,6 +453,7 @@ read_PCM_file(CommandOptions& Options)
        }
 
       FrameBuffer.Capacity(AS_02::MXF::CalcFrameBufferSize(*wave_descriptor, Options.edit_rate));
+      last_frame = AS_02::MXF::CalcFramesFromDurationInSamples(last_frame, *wave_descriptor, Options.edit_rate);
 
       if ( Options.verbose_flag )
        {
@@ -522,7 +523,10 @@ read_PCM_file(CommandOptions& Options)
       if ( ASDCP_SUCCESS(result) )
        {
          if ( Options.verbose_flag )
-           FrameBuffer.Dump(stderr, Options.fb_dump_size);
+           {
+             FrameBuffer.FrameNumber(i);
+             FrameBuffer.Dump(stderr, Options.fb_dump_size);
+           }
 
          result = OutWave.WriteFrame(FrameBuffer);
        }