c2a325746469c4c7625ec78a169b65a11dbe1e30 |
|
08-Aug-2011 |
Glenn Kasten <gkasten@google.com> |
Bug 5126938 dependency and miscellaneous bug fixes Fixing bug 5126938 will involve adding the IAndroidConfiguration interface to CMediaPlayer object. In the process of doing this, I found a few miscellaneous bugs in the error handling for IAndroidConfiguration. This CL fixes those bugs, and prepares for adding the interface CMediaPlayer by making it easier to support new objects with the interface. Details: - Generic error handling: Move some error checks from the object-specific code into the generic IAndroidConfiguration.c. Fix bug in SetConfiguration and GetConfiguration where we were not checking configKey for NULL Return SL_RESULT_FEATURE_UNSUPPORTED instead of SL_RESULT_PARAMETER_INVALID if IAConfig interface is exposed on an object which doesn't really support it (this could if the classes.c interfaces table was updated before the cases were updated). Compare with IMuteSolo for non-AudioPlayer. - Object-specific error handling: Fix bug where we weren't returning the actual value size in audioPlayer_getConfig when pValueSize was non-NULL and pConfigValue was NULL. Return result SL_RESULT_BUFFER_INSUFFICIENT instead of SL_RESULT_PARAMETER_INVALID when output buffer is specified as non-NULL but available space is too small. Compare with IAudioIODeviceCapabilities. Return result SL_RESULT_SUCCESS instead of SL_RESULT_PARAMETER_INVALID when output buffer is specified as NULL; this is intended to allow the application to discover the required space. Compare with IAudioIODeviceCapabilities. - Use 'switch' instead of 'if' for checking object types, to improve readability when adding more object types Change-Id: I8b2cdd506a36bfa9b1326a8a9966a6633bbbba6d
/frameworks/wilhelm/src/itf/IAndroidConfiguration.c
|
a8179ea15c4ff78db589d742b135649f0eda7ef2 |
|
13-Apr-2011 |
Glenn Kasten <gkasten@google.com> |
Fix build warnings caused by the 'long' change Change-Id: I34e74d723ae7fd829dd140614928d7880e669f03
/frameworks/wilhelm/src/itf/IAndroidConfiguration.c
|
bcc5c7225e3b7a1dbf2e9e830987f69167acf06f |
|
18-Jan-2011 |
Glenn Kasten <gkasten@google.com> |
Rename class__ to clazz, this to thiz The JNI naming conventions avoid conflict with C++ reserved words. Change-Id: I93ad6920bf78c0a02ac4b2650a29c9e11252f731
/frameworks/wilhelm/src/itf/IAndroidConfiguration.c
|
262059f71a68edc5e510427c63f5f1623d3672a8 |
|
12-Jan-2011 |
Glenn Kasten <gkasten@google.com> |
Directory re-organization Change-Id: I8e3f12a22e765d36ddefd87edf204735b25474f6
/frameworks/wilhelm/src/itf/IAndroidConfiguration.c
|