3eaa329775fc522d8ea4acd4edc379eabf9ac332 |
|
12-Nov-2012 |
Glenn Kasten <gkasten@google.com> |
Fix build warnings narrowing conversion of '-1' from 'int' to 'SLuint32 {aka unsigned int}' inside { } is ill-formed in C++11 [-Wnarrowing] converting to non-pointer type 'pthread_t {aka long int}' from NULL [-Wconversion-null] warning: 'currentRate' may be used uninitialized in this function [-Wmaybe-uninitialized] Change-Id: I066260cc216ff9f15c787f65e3d3efbda8f8ef35
/frameworks/wilhelm/src/android/CallbackProtector.cpp
|
15c706b25a8b9993b5391de574ed52c1628060be |
|
04-Oct-2011 |
Glenn Kasten <gkasten@google.com> |
Reduce log spam from callback protectors Change-Id: I83f40370bf86878e6d8dcabfff3847cab2cbbd6c
/frameworks/wilhelm/src/android/CallbackProtector.cpp
|
f4b45a37248899ae2d27bb172f8387fbf1edff8e |
|
12-Sep-2011 |
Glenn Kasten <gkasten@google.com> |
Bug 5090073 Callback protectors Add callback protector for decode to PCM usage cases. Move callback protection up earlier for URI decode use case only. Other: Remove redundant mCallbackProtector field. Optimization: on exit from callback protector, only broadcast if !mSafeToEnterCb, that is if destroy has requested callback protectors to finish up. More callback protector logging. Add callback protector requestCbExit() [no wait], not yet used. Unrelated: 3rd parameter of adecoder_writeToBufferQueue is always CAudioPlayer * instead of void *. Change-Id: I57a46acf0e5ecb213540b13ca08098177ad7ad6e
/frameworks/wilhelm/src/android/CallbackProtector.cpp
|
485a038f9f0f898227b8ab4218e94c5d56b6ed0b |
|
24-Aug-2011 |
Glenn Kasten <gkasten@google.com> |
Bug 5193695 Fix crash after MediaPlayer destroyed The StreamSource callback thread was continuing to run after the MediaPlayer object was destroyed. Fixed by adding a callback protector and a pre-destroy hook. GenericMediaPlayer::preDestroy now also calls MediaPlayer::stop just in case. Change-Id: I5bd771d4d1936f433d2a8c9959593782c96daed9 Miscellaneous: - added an explicit destructor on CMediaPlayer::mAVPlayer for consistency (probably not a bug since there was a clear on it earlier) - updated comments for CallbackProtector - made some CallbackProtector fields private since no sub-classes yet
/frameworks/wilhelm/src/android/CallbackProtector.cpp
|
6cce136651f6fd2c7aecd45bc553270152d75462 |
|
28-Jun-2011 |
Jean-Michel Trivi <jmtrivi@google.com> |
Fix race condition when deleting an AudioPlayer When deleting an AudioPlayer that is used as a PCM decoder, there can be callback underway, for instance when trying to delete a player just as the notification of its preparation arrives. The fix consists in: - flagging all callback with the CallbackProtector mechanism, - only entering callback when it is valid to do so - renaming AudioTrackProtector to CallbackProtector as this mechanism is not exclusively used for the AudioTrack callbacks. Change-Id: I9336a75981de43f71a983c1300f3a0ff314ac1e0
/frameworks/wilhelm/src/android/CallbackProtector.cpp
|