commit 60a839204713e0f8258d082af83262b1eb33a6c3
author Glenn Kasten <gkasten@google.com> Thu Jun 21 12:56:37 2012 -0700
committer Glenn Kasten <gkasten@google.com> Wed Nov 14 16:45:53 2012 -0800
tree 5b8ef6ace475c96637bb6498b1c0a602611f9a58
parent 3b16c766d1ae2cfd8487e8ffb2b23936fc0a8e17

Clean up frame size in AudioTrack and AudioFlinger

TrackBase::mFrameSize, mChannelMask, and mChannelCount are now const.
Use TrackBase::mFrameSize instead of re-calculating frame size.
AudioFlinger only sees 16-bit PCM format, conversion from 8-bit is
  now entirely on the client side.  Previously a small part of the
  responsibility was on server side also.
size_t is unsigned, so use %u in logs.
Fix theoretical bug where TrackBase constructor was over-allocating space
  for non-linear AudioTrack or 8-bit PCM AudioRecord (probably benign).

Change-Id: I7cbbba0bf4dba29ea751d8af341ab8e5cbbdc206

include/media/AudioTrack.h

diff --git a/include/media/AudioTrack.h b/include/media/AudioTrack.h
index 99d583d..6fd1b9e 100644
--- a/include/media/AudioTrack.h
+++ b/include/media/AudioTrack.h
@@ -520,7 +520,7 @@
             // address space.  AudioFlinger::TrackBase::mBuffer is for the server address space.
     void*                   mBuffers;
 
-    audio_format_t          mFormat;
+    audio_format_t          mFormat;                // as requested by client, not forced to 16-bit
     audio_stream_type_t     mStreamType;
     uint8_t                 mChannelCount;
     uint8_t                 mMuted;