| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Change-Id: I9d499190e49fed16aac686ae5da79ab870605ff4
|
|
|
|
|
|
|
|
| |
Fix uninitialized variable that caused ADTS recording to incorrectly
specify LTP object in the header.
Bug: 17895547
Change-Id: I75650f4963f4f371cd191bc635e32c9ab17fa8c9
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Recording into an existing file of non-zero length could leave old
data in the file after the recording ends. Best case that wastes
space, worst case it results in a file that's considered corrupt
on playback.
Change-Id: I2c77a107603b2b36790958360ff0856c2b28d677
|
| |
| |
| |
| |
| |
| | |
Clang complains about mismatch of argument type and print format.
Change-Id: Ib07da09d8b1b62b3018033f9eaf7aa01bf7f7f9c
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This removes the deprecated register keyword in many places, fixes one
instance of passing '\0' where a char* is expected, and 2 switch type
narrowing issues.
Change-Id: Ie265ac26ff6a61267ad6406dfe237226d97ca9ac
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
|
|\ \ \ |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
So the "%lld" format will not have Clang warnings in 32-bit and
64-bit builds. Alternative is to use PRId64.
Change-Id: I5fafc506fae012748a04ec261af62d9d13475cde
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
|
|\ \ \ \ |
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
In the -DNDEBUG case, nextOffset is unused, causing clang to spew a
warning in code build with -Werror.
Change-Id: If0101dcdf735672050f985a1de25a983edb02a52
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
|
|\ \ \ \ |
|
| | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
If ClientProxy was interrupted by AudioTrack::pause() just before futex syscall() in obtainBuffer() was called,
It will not call releaseBuffer to wake up AudioTrackThread.
It puts the AudioTrackThread to sleep and then a deadlock occurs.
In this case, CBLK_INTERRUPT flags can't prevent a deadlock,
so this patch set mFutex to FUTEX_WAKE during interrupt() to avoid deadlock.
A similar problem could occur due to re-route or recovery after mediaserver death.
Bug: 18641665
Change-Id: I66fcae43af9a91eb55f6cdb52c644ee6c0999772
|
| |/ /
|/| |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
For build-system CFLAGS clean-up, fix unused variables.
These variables are unused in aarch64.
Bug: 18632512
Change-Id: I6ae41f64e721a0f3188f0ee045a734b366ee72a6
|
|\ \ \
| | | |
| | | |
| | | | |
OutputPortSettingsChangedState"
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Change-Id: I9719f1698bf411914eb753d3c349a15b9fbf9df6
Signed-off-by: Per Persson <per.persson@intel.com>
Signed-off-by: Persson, Mikael <mikael.persson@intel.com>
Signed-off-by: Fagerstedt, Axel <axel.fagerstedt@intel.com>
Signed-off-by: Yong Yao <yong.yao@intel.com>
|
|\ \ \ \ |
|
| | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
If track is restored when it set its volume to a non-unity value (such as mute),
the volume was forced to unity again. Now, the previous volume is restored.
Bug: 18570664
Change-Id: I79f34c6c7d6f535baba813fd3b8ba8e55cac8ad7
|
|\ \ \ \
| |/ / /
|/| | | |
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The OggExtractor wasn't extracting the YEAR key when the metadata in
the file was parsed. The YEAR key is now extracted when the file
metadata is parsed.
Change-Id: Idf0d1a37fb141491dce5de527703ca1b27f621e7
|
|\ \ \ |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
For build-system CFLAGS clean-up, remove unused functions and
variables.
Change-Id: Ic3dee56b589ea9a693efa1d72ba394036efff168
|
|\ \ \ \ |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
An overloaded function should take the same parameters as the function
it is overloading.
Change-Id: I8327fe1b363917515cf76c8f76bdbc05b2c0fbf0
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
In C++11 mode, clang errors out on the register keyword.
Change-Id: I034ba5815e37e3ff5e017551977e97b8516123e8
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
|
|\ \ \ \ \ |
|
| | |/ / /
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | | |
For build-system CFLAGS clean-up, fix unused variables.
Change-Id: I589a73780f022271fce40f64305b29a84c382e5f
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
For build-system CFLAGS clean-up, fix unused variable.
Change-Id: I6aba5f011d47271fae0ac5eca1d35a398dd56856
|
|\ \ \ \ \ |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Remove unused parameter clang complains about
Change-Id: I6710645658a802438e41d881d262231a10b091d3
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
|
|\ \ \ \ \ \
| |_|/ / / /
|/| | | | | |
|
| | |/ / /
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Remove forward declarations of MediaBuffer -- they're not needed because
MediaBuffer.h is included earlier in the file anyway, and they
incorrectly declared MediaBuffer as a struct instead of a class.
Change-Id: Ia35ea8f938553e39244032b6fd6c061aa388859f
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
|
| | | | |
| | | | |
| | | | |
| | | | | |
Change-Id: Ie2fef95a45dd124572bf062011a272898a371924
|
|\ \ \ \ \ |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
offset_for_ref_frame is an array rather than a pointer - it can't be
NULL (and the check causes a warning in clang).
Change-Id: I9b96e1e723e3b8caebd41821137ed6fb669d8cd4
Signed-off-by: Bernhard Rosenkraenzer <Bernhard.Rosenkranzer@linaro.org>
|
|\ \ \ \ \ \ |
|
| | |/ / / /
| |/| | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
* Fix string literal concatenation to not be interpreted as UD
literals.
* Add constexpr compatibility for non-integral static members.
* Use __typeof__ instead of typeof (should become decltype once this
actually becomes C++11).
* Add an appropriate cast for atomic_uintptr_t, since moving to C++11
means moving from <stdatomic.h> to <atomic>, which has better
typechecking (hooray for not macros!).
Bug: 18466763
Change-Id: I9561dcb2526578687819ff85421ba80d8e1a9694
|
|/ / / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Remove extraneous parentheses that cause compiler warnings in clang
Change-Id: I88a3d84738433db85e4a04b673f82da7b0b1be10
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
|
|\ \ \ \ \ |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Clang compiler gives warning to old gnu syntax.
BUG: 18193625
Change-Id: I1b5237d60866fc357fc808f6a6955ab20418a7d8
|
|\ \ \ \ \ \ |
|
| | |_|/ / /
| |/| | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Remove extraneous parentheses - they cause compiler warnings in clang
Change-Id: Ie2efd9714f9e2251cf3b204b5c5dd1a8b8aaca13
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
|
|/ / / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
In the case of !(err == OK && !mCancelled), sdpSize is used
uninitialized.
Change-Id: I33c6365ca6df13b0051491771d3eee3e35346b6d
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
|
| | | | |
| | | | |
| | | | |
| | | | | |
Change-Id: I79f9a53b95dcf84dff62ed4b3c0661c7c1e1fc64
|
|\ \ \ \ \ |
|
| | |_|_|/
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Fix something being declared a struct and forward-declared a class and
vice versa.
gcc doesn't care, clang complains about the mismatch.
Change-Id: I7a7a8e9040e3c86fa1d73eb112631ea140861657
Signed-off-by: Bernhard Rosenkränzer <Bernhard.Rosenkranzer@linaro.org>
|
|/ / / /
| | | |
| | | |
| | | | |
Change-Id: Ib91ff0696ab2472c62168eb5261bbda5d22f623e
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
to use the new static version.
Change-Id: Ia7b10eb38ca55b72278bfd33d3bf647f338b4e6a
Conflicts:
media/libmedia/IAudioFlinger.cpp
media/libmedia/IMediaPlayer.cpp
media/libstagefright/CameraSource.cpp
|
|/ / /
| | |
| | |
| | |
| | | |
Bug: 15193147
Change-Id: If59c4a53a497f501881d26d58b80b11371906527
|
| |/
|/|
| |
| |
| |
| | |
log/log.h
Change-Id: Ieb07a4f58c96326553e87dacca11e3980e283dc8
|