[APU] XMA: ConvertFrame always returns true

src/xenia/apu/xma_context.cc

@@ -806,7 +806,7 @@ int XmaContext::PrepareDecoder(uint8_t* packet, int sample_rate,
   return 0;
 }
 
-bool XmaContext::ConvertFrame(const uint8_t** samples, bool is_two_channel,
+void XmaContext::ConvertFrame(const uint8_t** samples, bool is_two_channel,
                               uint8_t* output_buffer) {
   // Loop through every sample, convert and drop it into the output array.
   // If more than one channel, we need to interleave the samples from each
@@ -880,8 +880,6 @@ bool XmaContext::ConvertFrame(const uint8_t** samples, bool is_two_channel,
     }
   }
 #endif
-
-  return true;
 }
 
 }  // namespace apu

src/xenia/apu/xma_context.h

@@ -186,7 +186,7 @@ class XmaContext {
   static std::tuple<int, bool> GetPacketFrameCount(uint8_t* packet);
 
   // Convert sample format and swap bytes
-  static bool ConvertFrame(const uint8_t** samples, bool is_two_channel,
+  static void ConvertFrame(const uint8_t** samples, bool is_two_channel,
                            uint8_t* output_buffer);
 
   bool ValidFrameOffset(uint8_t* block, size_t size_bytes,