diff options
| author | Glenn Kasten <gkasten@google.com> | 2011-12-15 15:46:46 -0800 |
|---|---|---|
| committer | Glenn Kasten <gkasten@google.com> | 2011-12-15 15:55:51 -0800 |
| commit | fba380a0b2c62684ce4272a05165965bae29d6a1 (patch) | |
| tree | d0a89a2c45aa09a838c5f93108a3be8f5a80d4e5 /services/audioflinger/AudioMixer.h | |
| parent | eebeceec684a36222b4559e3157b0db04c0a67ed (diff) | |
| download | frameworks_av-fba380a0b2c62684ce4272a05165965bae29d6a1.zip frameworks_av-fba380a0b2c62684ce4272a05165965bae29d6a1.tar.gz frameworks_av-fba380a0b2c62684ce4272a05165965bae29d6a1.tar.bz2 | |
setActiveTrack and setBufferProvider can't fail
Return void, not status_t, from setActiveTrack and setBufferProvider.
These methods returned status_t, but the callers never checked the
return value. Since these aren't externally visible APIs, they now
return void, and assert on bad input parameters.
Change-Id: I530ed29484596ae41e8659826ca425149c51c2a1
Diffstat (limited to 'services/audioflinger/AudioMixer.h')
| -rw-r--r-- | services/audioflinger/AudioMixer.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/services/audioflinger/AudioMixer.h b/services/audioflinger/AudioMixer.h index 75c9170..6f31c5d 100644 --- a/services/audioflinger/AudioMixer.h +++ b/services/audioflinger/AudioMixer.h @@ -81,10 +81,10 @@ public: status_t enable(int name); status_t disable(int name); - status_t setActiveTrack(int track); + void setActiveTrack(int track); status_t setParameter(int target, int name, void *value); - status_t setBufferProvider(AudioBufferProvider* bufferProvider); + void setBufferProvider(AudioBufferProvider* bufferProvider); void process(); uint32_t trackNames() const { return mTrackNames; } |