diff options
author | Glenn Kasten <gkasten@google.com> | 2012-03-06 11:23:32 -0800 |
---|---|---|
committer | Glenn Kasten <gkasten@google.com> | 2012-03-07 07:28:12 -0800 |
commit | 761415bd7f5611812336a3d39b3f6baac22a1b88 (patch) | |
tree | 61fbb2ef2888252a8722af5730745c820b225599 /services/audioflinger/AudioFlinger.h | |
parent | ff249c0dc814dfc22d8575ec1ef3af03831dfaaa (diff) | |
download | frameworks_base-761415bd7f5611812336a3d39b3f6baac22a1b88.zip frameworks_base-761415bd7f5611812336a3d39b3f6baac22a1b88.tar.gz frameworks_base-761415bd7f5611812336a3d39b3f6baac22a1b88.tar.bz2 |
Merge the calls to prepareTracks_l
Change-Id: I1dd759581333e2908d980180d44db7bf5ed6591d
Diffstat (limited to 'services/audioflinger/AudioFlinger.h')
-rw-r--r-- | services/audioflinger/AudioFlinger.h | 19 |
1 files changed, 7 insertions, 12 deletions
diff --git a/services/audioflinger/AudioFlinger.h b/services/audioflinger/AudioFlinger.h index 336c777..edf083f 100644 --- a/services/audioflinger/AudioFlinger.h +++ b/services/audioflinger/AudioFlinger.h @@ -584,7 +584,7 @@ private: // standby mode does not have an enum value // suspend by audio policy manager is orthogonal to mixer state #if 1 - // FIXME remove these hacks for threadLoop_prepareTracks_l + // FIXME remove this hack for prepareTracks_l() , MIXER_CONTINUE // "continue;" #endif }; @@ -817,12 +817,11 @@ protected: // Non-trivial for DIRECT only virtual void applyVolume() { } - // FIXME merge these - // Non-trivial for MIXER and DUPLICATING only - virtual mixer_state prepareTracks_l(Vector< sp<Track> > *tracksToRemove) { return MIXER_IDLE; } - // Non-trivial for DIRECT only - virtual mixer_state threadLoop_prepareTracks_l(sp<Track>& trackToRemove) - { return MIXER_IDLE; } + // prepareTracks_l reads and writes mActiveTracks, and also returns the + // pending set of tracks to remove via Vector 'tracksToRemove'. The caller is + // responsible for clearing or destroying this Vector later on, when it + // is safe to do so. That will drop the final ref count and destroy the tracks. + virtual mixer_state prepareTracks_l(Vector< sp<Track> > *tracksToRemove) = 0; public: @@ -970,10 +969,6 @@ public: virtual status_t dumpInternals(int fd, const Vector<String16>& args); protected: - // prepareTracks_l reads and writes mActiveTracks, and also returns the - // pending set of tracks to remove via Vector 'tracksToRemove'. The caller is - // responsible for clearing or destroying this Vector later on, when it - // is safe to do so. That will drop the final ref count and destroy the tracks. virtual mixer_state prepareTracks_l(Vector< sp<Track> > *tracksToRemove); virtual int getTrackName_l(); virtual void deleteTrackName_l(int name); @@ -1006,7 +1001,7 @@ public: virtual uint32_t suspendSleepTimeUs(); // threadLoop snippets - virtual mixer_state threadLoop_prepareTracks_l(sp<Track>& trackToRemove); + virtual mixer_state prepareTracks_l(Vector< sp<Track> > *tracksToRemove); virtual void threadLoop_mix(); virtual void threadLoop_sleepTime(); virtual void applyVolume(); |