fix merge duplication

For fixing svace issue about duplication of free `g_default_language`.
This patch removes one of them.

Change-Id: I2100e35e70c60921e4753e85125b12d83d5171f8

Update version (1.90.0)

Change-Id: Ie3a73a17905455038397268ac5390a652ed3f280

Merge "Fix svace issue (add null checker)" into tizen

Fix defects detected by static analysis tool

Change-Id: Ic53d90b9783c0fe2cde0e47b2baca984bae544e6

Fix svace issue (add null checker)

Change-Id: I6c7c6b11589b24bb6ea2b9587a8ba8bc9857b247
Signed-off-by: sooyeon <sooyeon.kim@samsung.com>

Update since_tizen in header files

Change-Id: Ia969c88684251e968d7d006fc515e6cfe7e019e5
Signed-off-by: sooyeon <sooyeon.kim@samsung.com>

Include stddef.h for size_t type

Change-Id: Ie2a56f31318d7abf00c69b2bd47e328e95f479d5
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Merge "Fix test case name and scenario" into tizen

Restore the internal definition temporarily

Change-Id: I0415eaff349f3fbd480a62bbeefd109d0eacd649
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Fix test case name and scenario

- Issue:
Test case name is not matched with the name of the function which is
tested by the test case.
stt_get_audio_id has no test cases for testing invalid state error.

- Solution:
This patch fixes the name of the test cases and scenario. Through this
patch the name of each TCs will be matched with the tested function. And
also, negative TC for stt_get_audio_id will properly test the invalid
state scenario.

Change-Id: Ie7668a9516ab788cfda4c88ddc18af97c974874d
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Update version (1.80.3)

Change-Id: I935a25a7fdc9d9732b173603f0d3d19cd0216f55
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Pop all dbus message

- Issue:
When the test case for audio streaming, stt_cancel() can be ignored
sometimes.

- Solution:
This patch adds while loop in a dbus listener callback to get all the
message from the handle. If there are multiple message requests from the
client library, the server process can receive multiple message at once
while single invocation of listener callback. However, the code does not
pop the all message from the listener handle. This may cause ingoring
some requests from the client. So, this patch fixes this situation using
while loop. Through this patch, listener callback always pop the all
messgae from the listener handle.

Change-Id: I258e4f776956efdb1e2c81f93658ba58e2070052
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Fix typo from doxygen description

Change-Id: I1fd4c6a5823b078343abcfc6197fe38b85a58d25
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Add parameter checking code

- Issue:
Some functions do not have code for parameter checking even though their
description says they can return invalid parameter error.

- Solution:
This patch adds parameter checking code. Through this patch, the
functions can check the parameter validation and returns the error
properly.

Change-Id: Ie7ffe4ab2ebb7df19b63b09bb17315208af064a2
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Update version (1.80.2)

Change-Id: I5c1215620fb7c46cfda095557182dcf1163c3395
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Move setter and getter about audio id to public header file

Change-Id: Ie327efff75ff540fac2182a49e1f2448554472e4
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Update version (1.80.1)

Change-Id: Ic40e38370c4ea8801a1b3ea717e2d9014ded9f0f
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Move functions about audio streaming to public header file

Change-Id: Ibb6a5473aabdb36d7936a79fc99b66d36025e3da
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Fix typo in function and variable names

Change-Id: I39bf65c30a2d95b8e4d10f97f77890140a88340d
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Fix coding style for internal function

- Contents:
This patch removes the two underscore identifier from function name.

Change-Id: Icc732bb3afbc59e1341eb9e830a86fa0756ed31b
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Refactor unused and unreachable code

- Contents:
This patch fixes some unused and unrechable code. Through this patch,
unvaluable code is removed and also test cases covers more code and
logics.

Change-Id: I7ce5698cccb39491b50738d8d6e5301ae11ad098
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Add new test cases to increse the coverage

- Contents:
This patch adds new test cases for uncovered line. Through this patch,
developers can check the behavior about specific situations.

Change-Id: I5289f5b80609768c7eaa9227eb1ce9e7d9032207
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Refactor unit tests with utility class

- Contents:
This patch fixes all unit tests using new utility class from previous
commit. Through this change, many duplicated codes in test cases can be
removed.

Change-Id: Icff147fe2b492f8dfb08b267ad374f9c5a20151f
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Add test utility class

- Contents:
This patch introduces new utility class for test case. Through this
class, developers can easily make the test cases. This new class
provides many useful utility methods and frequently used code blocks.

Change-Id: Ia76c9eed7345f98421a89da7c302c59794b311d3
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Fix condition for checking return value of play_sound()

- Issue:
sttd_server_start() did not progress even though all behavior is
properly worked.

- Solution:
play_start_sound_for_uid() return true if start sound is playing.
However, previous code uses this return value reversely. So, the
sttd_server_start() worked inproperly.
To solve this issue, this patch fixes the condition properly. Through
this patch, sttd_server_start() will work properly.

Change-Id: Iba90d239b917182fbffea54cb526ebe5629262a7
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Refactor sttd_server module according to review

- Contents:
This patch includes these changes:
  - Use play_sound() in sttd_server_start() and sttd_server_stop()
  - Change from strcmp() to strncmp() for secure access
  - Remove play_sound() from the functions related about streaming mode

Change-Id: I0eef55f3034d6d75dcc33f806a83c4dba863f24d
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Add test cases for audio streaming functions

- Requirements:
The test cases for testing new functions are neccessary.

- Contents:
This patch adds new test cases for audio streaming functions. With this
patch, the developer can validate new functions.

Change-Id: Ie07218cca3ba169967400f58a0b0f4bf8c5531c8
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Implement new API functions for audio streaming

- Requirements:
The client library needs to provide new API for starting, sending,
stopping audio streaming feature. And also, the client library needs to
provide new API for getting audio format information.

- Contents:
To provide new API functions for audio streaming, this patch adds
implementation for these new functions. Through this patch, this module
has prepared to provide new API for audio streaming.

Change-Id: I88016a602f44c0325e7e58fc63c03872a8533f6d
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Add dbus interface for audio streaming

- Requirements:
The client and server library must be able to exchange the requests for
audio streaming.

- Contents:
To handle requests for audio streaming, this patch adds new dbus
interface. With this patch, the client and server library can exchange
the requests for audio streaming.

Change-Id: I559f8f7693e87539549b81c5b90cda78feca7270
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Add new functions for handling audio streaming

- Requirements:
The engine library needs to recognize audio streaming from the clients.

- Contents:
To handle audio streaming from the clients, this patch adds new
functions. Through this new functions, the engine library can sends the
raw PCM data from the clients to the engine.

Change-Id: If54a437d0bf43cf2ead88f65a264225e645f57da
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Add new functions for getting audio format from engine

- Requirements:
The app needs to know the audio format information that engine can
handle.

- Contents:
This patch adds new function to get audio format information from the
engine. Through this patch, the engine library can ready to send the
audio format information to the clients.

Change-Id: I84b96d93de651740587b3630c84086beeb29ba58
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Update version (1.80.0)

Change-Id: I8c96e0bbae4a4a1dc9f9a2aea523a6161d0f2c05

Add binary file to support risc64 build

Change-Id: I3efc14c89de555e6e24a0a30649fc33f29167d7e

Update version (1.75.3)

Change-Id: I443936b76c1ccbef95afec85138c022c5a69ec30
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Fix static analysis issue

This patch includes this changes to resolve potential issues:
- Add code for checking return value of the function.
- Move the local variable definition to minimize the valid scope of each
variables.

Change-Id: I8b7b977d45e37eaec27177f3de4e1c5cc6e5cc37
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Update version (1.75.2)

Change-Id: I749e680ae4378ae3b0ee2969f066c081f0aa6084
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Clean up the sttd_server code

This patch includes these changes to get more readbility:
- Add static keyword to every internal functions and variables
- Remove unvaluable comments and unused code
- Remove duplicated code

Change-Id: I1592ae524e6524d66bd6143af666434f52380d34
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Cancel stop by silence timer when new recording is started

- Issue:
New recording can be stopped by previously reserved stop by silence
timer.

- Solution:
This patch adds new code for managing stop by silence timer. And also,
this patch adds new code for canceling this timer when
sttd_server_start() is invoked. Through this patch, recording can be
started on perfectly clear state.

Change-Id: I578b66819ccb983a9c8c98203e058ecdb69b5c21
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Consider processing state as no error

- Issue:
When stt_stop() is invoked while silence detection is enabled, sometimes
the framework makes error.

- Solution:
stt_stop() function works asynchronously and silence detection also has
asynchronous behavior. So, these two logic can conflict with each other.
To avoid this situation, this patch skip the remained logic if the state
is already changes to processing. Through this patch, even if the two
logic is invoked at once, one of the logic will properly stop the
recording and another one will pass the logic with no error.
Post condition of these two process is same, so it is fine to skip the
logic with no error handling.

Change-Id: I7d5e143153b7167f742ec64daf50f73041491fab
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Remove unused enum definition

- Issue:
The enum value is unvaluable in server logic.

- Solution:
Enum values in sttd_result_state_e are not valuable, because there is no
logic to handle those value. So, this patch removes this enum definition
from the code.

Change-Id: I2d2b57d11fd5c0508346783c1a4c70d76d1ca1c2
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Remove duplicated logic int stop behavior

- Issue:
In __stop_by_silence() function, there is duplicated logic.

- Solution:
In previous code, __stop_by_silence() function send message to client
according to the return value of sttd_server_stop(). However,
sttd_server_stop() is already send message when it is return STATE DONE
value. Thus, the bahvior on __stop_by_silence() is actually unvaluable.
Thus, this patch removes that code to reduce duplicated logic.

Change-Id: I61948b1ad6fe6d243b1240fb8ae484639b072e0e
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Update version (1.75.1)

Change-Id: I39f7bcac8a1d91bd69621d70d8a8423473d4abb2
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Support audio I/O on TV code

- Issue:
On TV, STT can not record voice with USB MIC.

- Solution:
In previous code, STT framework did not use audio I/O framework on TV
platform code. So, on TV, STT framework did not record sound using audio
I/O framework. This patch makes the framework use audio I/O framework on
TV. Through this patch, STT framework choose the recording source by
audio ID which is set by clients. Of course, even though this patch is
applied, the code for TV only will be still blocked in common image.

Change-Id: I45e68b614fd48a818c06822979738c0021e4532a
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Add missing log and pointer reset code

- Issue:
In some condition, the file pointer did not closed and made a memory
leakage.

- Solution:
In previous patch, a log for checking error status was missed. So, the
behavior of the code was not predictable. Thus, this patch restores the
log and also adds pointer reset code in order to prevent memory leakage.
Through this patch, the code will work as intented.

Change-Id: Id2524f579bd68e0f757910dbdf93e482dc402755
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Update version (1.75.0)

Change-Id: If55dd545b1bce3439b747024c62aac2a0ca6c67b
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Add checker for feature and privilege to engine API

- Issue:
The engine API did not check feature and privilege of the application
even if it was described on header file.

- Solution:
This patch adds new logic for checking the feature and privilege for
using STT. Through this patch, engine API can return error values about
feature and privilege.

Change-Id: I4423ecac6a2ef20c92c535252bdd401e899b34f9
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Remove unnecessary parameter for stt_start_file function

The audio format types are not changable in current STT engine.
Therefore sample rate, channel and audio type are not necessary to transmit the pcm data to cloud.
The application should send pcm data, which consits of 1 channel, 16000 sample rate and signed 16 bit pcm data.

Change-Id: I4f9ad034419bf088d6de9d2a1b4a54adfc6e718a

Send pcm data as real time from file, which app wants to recognize the speech

Change-Id: I941d16223458020a546575156a6f5f8917f1a626

Update version (1.70.9)

Change-Id: I6c201374b750622edd50a462d7321b3a639c85ea
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Make atomic variable to avoid race condition

- Issue:
Sometimes, the engine library can access invalid memory location when
the engine app tries to access callback.

- Solution:
Some engine API can access callback function in sttd_engine_agent.
However, these callbacks are not protected by multi threading. Thus,
thee engine app can access invalid pointer if engine library terminates
the engine internally and the app tries too access callback in sub
thread.
Therefore, this patch changes the global variable to atomic variable for
protecting those callback pointers and flags. And also this patch adds
null check for those pointers. Through this patch, engine library can
access those global variables in a safe way even if it is on multi thread
environment.

Change-Id: Ie501985b0ad5b1fdaf0937649cbacc37ab1e4b42
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Fix null pointer access issue when check audio_id

- Issue:
The first time to start recording, the engine can get null audio_id from
stt engine module, if the engine app did not set audio id.

- Solution:
Because of the order of calling setter and getter of audio id, getter
can provide null pointer as audio id, but the implementation does not
check it. Thus, this patch adds null pointer checker and fix some logic
to prevent crash by accessing null pointer.

Change-Id: Id68f17f6267adf3bffadd07ad797978b4800045b
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Add to set/get audio type on client side

Cause: FFV callback was called, although audio type of sending audio was from Blutetooth.

Solution: Provides a way to set audio type on client side.

Change-Id: If9d2177cfe7aa61bba9fa0480ea750ea9c0d7ceb

Fix to transfer wrong argument

Change-Id: Ic831091feb2d9c092837d3c07de9e29c926f6491
Signed-off-by: sooyeon <sooyeon.kim@samsung.com>

Update version (1.70.8)

Change-Id: I37ac61df076c2f946da5545b7f48056219d1d206
Signed-off-by: sooyeon <sooyeon.kim@samsung.com>

Enable FFV on TV and Use 2nd channel data in ffv audio callback

Change-Id: Idd2514eda85a1896ebd8870a7e5b7e4ab1c77acb
Signed-off-by: sooyeon <sooyeon.kim@samsung.com>

Update version (1.70.7)

Change-Id: I278618cae558c5a81542c3873b71b316dec1126a
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Extend timeout value for recognition processing

- Issue:
In emulator, sometimes, the client can not get final result.

- Solution:
This patch extends timeout value for recognition processing. Through
this patch probability of getting final result is increased even if the
processing requires more time and resources.

Change-Id: Iae9be55df0bfa3da9ec6d8807a06c58e4a713be7
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Merge "Fix defect detected by static analysis tool" into tizen

Fix defect detected by static analysis tool

Change-Id: I247fc0379958d61a607b077b7f743e322435233d

Update version (1.70.6)

Change-Id: I9636b8cbe8842094a0a8cf5ec26397696a3e02fa
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Reorder precondition checker

- Issue:
Some unit tests are failed.

- Solution:
The reason why the tests are failed is because the order for checking
precondition is changed from previous code. All APIs first check
parameters and then check state. However, some function check state
first and the parameters after next patch.
https://review.tizen.org/gerrit/c/platform/core/uifw/stt/+/277656

This patch revert the order of precondition check, so every APIs check
precondition with same way.

Change-Id: I91d1ef4e3fd0910070e182bad39156d45b1c69fc
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Merge "Fix line coverage scope and test option" into tizen

Add metadata key

Change-Id: I2b3be6281883bc69739159cdef3dc6461331dd49

Fix line coverage scope and test option

This patch reduces the scope for checking line coverage. The line
coverage is for checking client API, so other files in this repo is not
target of the line coverage test.

Change-Id: Ib93c7f6a49dc72d4f6c580c3d69ab393f9ed5294
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Update version to 1.70.5

Change-Id: I219d8e98f4a4fe9734eddc0163aabc6111f4372b

Extract codes to check retrying dbus request

Change-Id: I98b4b6348f974c82f87dc61f601143730f4b32e5

Add codes to check precondition with state

Change-Id: I474382205ba3f479d7696fed8e06c8f61bfc5a90

Extract to check precondition on engine agent

Change-Id: I21d7729cf7297428ee07be0d79d74f23d2f0a777

Merge "Extract to get base node from config" into tizen

Extract to get base node from config

Change-Id: Icd8c0c2270f7aa3ce1dbdd05606c50078177a31f

Extract dbus_connection_send

Change-Id: I32f75c3f8e98da1e720f3d805985d3f4dcc97c26

Fix value for call by reference and simplify codes for logs

Change-Id: Ie1ff065388e44e7cfa964e545477f28f7d7e0224

Add function to check precondition

Change-Id: If458a13a8f6b937fa3c7c3caab28213e6eadfa71

Merge "Extract and simplify codes to check precondition" into tizen

Merge "Extract precondition for start" into tizen

Extract and simplify codes to check precondition

Change-Id: Id468feec452fc7a8883585258cb8d709a6706ec0

Extract precondition for start

Change-Id: I39dd1765782564a956d7edf92d799aeba2b17789

Merge "Remove unreachable codes" into tizen

Remove unreachable codes

Change-Id: I355058d68d6a2f850ab346dd4e934cad3d85ff9f

Extract dbus message return for reducing duplicated codes

Change-Id: I31a7bdbf6433ef768e4fb65c2a82eee5c14cdfd8

Remove duplicated codes

Change-Id: I8a9b6e53fa371936b50bcfe749ade2dc85c12508

Remove duplicates regarding dbus_connection_send_with_reply_and_block

Change-Id: Iaec79cc7a2801fd10e45349bff77ea96552763de

Remove duplicated codes for set/unset sound for start and stop

Change-Id: I927c0cd1ec1e022a9564a259e82a58d1d2936a11

Remove duplicated code for SAM score

Change-Id: I2ae51acc83aefb32a0eb579ab560c74080c3be41

Update version (1.70.4)

Change-Id: I6f00ac16c432200d852f0bf3e1613d6822e280a6
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Fix unittest for line coverage automation

Change-Id: I55a086595b43ea33f6d9aaf1304f90edc99eb8f9
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Fix gcov package contents for automation

Change-Id: Icec96eaa943ddd88a7516af91b1112cc4352a3e1
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Update version to 1.70.3

Change-Id: Ic2777ff15832c7e0eec2fe643c3856c5a748d6d6

Add metadata profile xml file

Change-Id: I56b09585dc5396e5b46d505f18bf3d43cb2017ee

Update version (1.70.2)

Change-Id: I8b51a78b736e1eb656d1f1d1cc8115f9c0a1296b
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Prevent state change even if no recognition result callback is registered

Current code changes the state to ready when the result comes from the server without registered
recognition result callback. However, this behavior can make difference between client state and
real server state.

To handle this potential issue, this patch prevents the state change in this case. By this change,
client state is always matched with server state.

Change-Id: I4df2438fb09d00185318b487b8f233ba0918798b
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Fix the condition for checking dbus data type of uid

Type of uid is unsigned integer, but current code checks whether the uid type is signed integer.
By this behavior, the result from server is not received.

This patch changes the condition for checking dbus data type of uid from int32 to uint32.
By this change, the result is properly delivered to client application by the callback.

Change-Id: I0b0e75fcc5f4903dd5085fa208b21b0e5e30f530
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Update version to 1.70.1

Change-Id: Ia85a00cb18d0b45854a85126bb0d5ab9ed141f19

Merge "Coverage support for robot profile" into tizen

Coverage support for robot profile

Change-Id: I4e18863cac25c9db04442f2801cd09874de30a74

Merge branch 'tizen_6.5' into tizen

Change-Id: Id7b2ac0c4e921eaf1b65efcf67870a693857d450

Update version (1.70.0)

Change-Id: I6da39a0b9dd36b54b6769761f233df851a30cd30
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Update version (1.65.2)

Change-Id: Iafa4ccde1abd6169c4feb8d5abc10474917d820c
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Get UID from list using stt_config_client_s structure

Current code converts the type of element as integer. However, the element of the list is instance
of stt_config_client_s structure. Because the first member of the structure is uid, so current code
has no problem until now. But this can occur problem if the type of uid or the order of member is
changed.

To solve this problem, this patch changes the type from integer to the structure. This can provide
safe access to member.

Change-Id: Ica103855b96bfc4196044772343216f75d38f9c0
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Fix the data type for uid

On 32bit architecture, signed integer is enough to store uid value. However, on 64bit architecture,
signed integer is not enough because maximum value of pid is 4194304 and value of pid * 1000 is
bigger than the limit of signed integer. And it cause overflow problem.

To solve this issue, this patch changes the data type of variable that stores uid.
Unsigned integer can provides enough space to store uid value on 64bit os.

Change-Id: I32fd8cbc81e2553524cb43cd357eda077f9ceeea
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>