C++20 will require members in a designated initializer to be in order
unlike C99.
Bug: 139945549
Test: mm
Change-Id: Ie129344a36893adc82e67181012e3af817bda9f4
Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>
AudioPlayer was only used by the commandline utilities, so move it
out of libstagefright.
Test: build, run
Change-Id: I561cccd323206de7415bd235b72711194080aaea
INIT_CHECK() expands to `if (foo) return bar;`, and `EventTimer`'s
destructor uses a value that's only set if `SetAttribute` is called.
This CL flips the INIT_CHECK/EventTimer lines to match getKeyRequest.
Caught by clang's static analyzer:
frameworks/av/include/media/EventMetric.h:155:7: warning: 2nd function
call argument is an uninitialized value
[clang-analyzer-core.CallAndMessage]
Bug: None
Test: TreeHugger
Change-Id: Ie0c4fb8c99a56082e234475e539c2ec4bc8fd948
'pkg' was std::move()d a few lines before, so we avoid reading
from it for our logging statement.
Test: TreeHugger
Change-Id: Idd25360ee0b1fbfee3ae63fb8f6c06f95d302792
am: 38ea0696e9 -s ours
am skip reason: change_id I14b8241c5eb6c789ebc5d6725db716b9b0f8f31f with SHA1 a33446ada9 is in history
Change-Id: I35d55a6cc37bf8ebeae2f98976a976beee99533f
Restore pre Q behavior for HOTWORD capture for non
Assistant applications. This allows non Assistant privileged
applications to capture audio with source HOTWORD and no visible
foreground UI as long as no other app is capturing. This capture
will be silenced as soon as another app starts capturing and will not
prevent other apps from capturing.
Bug: 140799148
Test: manual OK Google and Now Playing tests
Test: CTS AudioRecordTest, GTS HotwordTest
Change-Id: I9473851e2949e42abc02fb5974be21d351810854
This reverts commit aa05782462.
Reason for revert: The original CL should work on top of ag/9570881
Test: make, atest audiofoundation_parcelable_test
Change-Id: Ibd49f8e86d97f94b13ff59779631283c0c4452e3
Add libaudio_system_headers, libaudioclient_headers, libmedia_headers
to export_header_lib_headers for libaudiofoundation_headers so that
there is not needed to add header lib when using libaudiofoundation.
Test: make
Bug: 135621476
Change-Id: Iaa2c8046f9bce4b69398795e162d1ee39cece24b
JetPlayer.cpp was only used by the JetPlayer JNI code, so move
it to the same library.
Test: atest JetPlayerTest
Change-Id: I14b8241c5eb6c789ebc5d6725db716b9b0f8f31f
Merged-In: I14b8241c5eb6c789ebc5d6725db716b9b0f8f31f
Go ahead and write the data to the buffer if there is room,
even if it exceeds the setBufferSizeInFrames().
There is no point of holding it in the application buffer.
By making the data available to the DSP we can reduce glitches
or reduce latency.
Because this offers better glitch protection, we can allow
setBufferSizeInFrames() to accept a buffer size of zero.
This will provide the lowest possible latency.
Test: OboeTester glitch test
Change-Id: Ie706867c5dd57c29b4393c806cd51ae09198873e
JetPlayer.cpp was only used by the JetPlayer JNI code, so move
it to the same library.
Test: atest JetPlayerTest
Change-Id: I14b8241c5eb6c789ebc5d6725db716b9b0f8f31f
Marco Nelissen
TreeHugger Robot
Nick Desaulniers
George Burgess IV
Eric Laurent
Greg Kaiser
Ray Essick
Andy Hung
Jiabin Huang
Phil Burk
Mikhail Naganov