Skip to content

Commit 2fa59c8

Browse files
committed
Build separate sonivox libs with and without jet
and remove more unused code. Bug: 151448144 Test: build Change-Id: I6c5ef52c3fe3a00d1bd578005a77a07016540297
1 parent 34ba480 commit 2fa59c8

File tree

10 files changed

+65
-1161
lines changed

10 files changed

+65
-1161
lines changed

arm-wt-22k/Android.bp

+22-3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
cc_library {
2-
name: "libsonivox",
1+
cc_defaults {
2+
name: "libsonivox-defaults",
33
srcs: [
44
"lib_src/eas_data.c",
55
"lib_src/eas_dlssynth.c",
@@ -30,7 +30,6 @@ cc_library {
3030
"lib_src/eas_xmf.c",
3131
"lib_src/eas_xmfdata.c",
3232
"lib_src/wt_22khz.c",
33-
"lib_src/jet.c",
3433
"host_src/eas_config.c",
3534
"host_src/eas_hostmm.c",
3635
"host_src/eas_report.c",
@@ -114,3 +113,23 @@ cc_library {
114113
],
115114
}
116115
}
116+
117+
cc_library {
118+
name: "libsonivoxwithoutjet",
119+
defaults: ["libsonivox-defaults"],
120+
apex_available: ["com.android.media"],
121+
}
122+
123+
cc_library {
124+
name: "libsonivox",
125+
defaults: ["libsonivox-defaults"],
126+
srcs: [
127+
"lib_src/jet.c",
128+
],
129+
130+
cflags: [
131+
"-DJET_INTERFACE",
132+
],
133+
}
134+
135+

arm-wt-22k/lib_src/eas_data.h

-2
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,6 @@
3131
#ifndef _EAS_DATA_H
3232
#define _EAS_DATA_H
3333

34-
#define JET_INTERFACE
35-
3634
#include "eas_types.h"
3735
#include "eas_synthcfg.h"
3836
#include "eas.h"

arm-wt-22k/lib_src/eas_imelody.c

+7
Original file line numberDiff line numberDiff line change
@@ -160,8 +160,13 @@ const S_FILE_PARSER_INTERFACE EAS_iMelody_Parser =
160160
IMY_State,
161161
IMY_Close,
162162
IMY_Reset,
163+
#ifdef JET_INTERFACE
163164
IMY_Pause,
164165
IMY_Resume,
166+
#else
167+
NULL,
168+
NULL,
169+
#endif
165170
NULL,
166171
IMY_SetData,
167172
IMY_GetData,
@@ -706,6 +711,7 @@ static EAS_RESULT IMY_Reset (S_EAS_DATA *pEASData, EAS_VOID_PTR pInstData)
706711
return EAS_SUCCESS;
707712
}
708713

714+
#ifdef JET_INTERFACE
709715
/*----------------------------------------------------------------------------
710716
* IMY_Pause()
711717
*----------------------------------------------------------------------------
@@ -777,6 +783,7 @@ static EAS_RESULT IMY_Resume (S_EAS_DATA *pEASData, EAS_VOID_PTR pInstData)
777783
pData->state = EAS_STATE_PLAY;
778784
return EAS_SUCCESS;
779785
}
786+
#endif
780787

781788
/*----------------------------------------------------------------------------
782789
* IMY_SetData()

arm-wt-22k/lib_src/eas_ota.c

+7
Original file line numberDiff line numberDiff line change
@@ -117,8 +117,13 @@ const S_FILE_PARSER_INTERFACE EAS_OTA_Parser =
117117
OTA_State,
118118
OTA_Close,
119119
OTA_Reset,
120+
#ifdef JET_INTERFACE
120121
OTA_Pause,
121122
OTA_Resume,
123+
#else
124+
NULL,
125+
NULL,
126+
#endif
122127
NULL,
123128
OTA_SetData,
124129
OTA_GetData,
@@ -675,6 +680,7 @@ static EAS_RESULT OTA_Reset (S_EAS_DATA *pEASData, EAS_VOID_PTR pInstData)
675680
return EAS_SUCCESS;
676681
}
677682

683+
#ifdef JET_INTERFACE
678684
/*----------------------------------------------------------------------------
679685
* OTA_Pause()
680686
*----------------------------------------------------------------------------
@@ -738,6 +744,7 @@ static EAS_RESULT OTA_Resume (S_EAS_DATA *pEASData, EAS_VOID_PTR pInstData)
738744
pData->state = EAS_STATE_PLAY;
739745
return EAS_SUCCESS;
740746
}
747+
#endif
741748

742749
/*----------------------------------------------------------------------------
743750
* OTA_SetData()

arm-wt-22k/lib_src/eas_public.c

+8-66
Original file line numberDiff line numberDiff line change
@@ -218,55 +218,6 @@ EAS_RESULT EAS_IntSetStrmParam (S_EAS_DATA *pEASData, EAS_HANDLE pStream, EAS_IN
218218
return EAS_SUCCESS;
219219
}
220220

221-
/*----------------------------------------------------------------------------
222-
* EAS_IntGetStrmParam()
223-
*----------------------------------------------------------------------------
224-
* This routine gets common parameters like transpose, volume, etc.
225-
* First, it attempts to use the parser EAS_GetStreamParameter interface. If that
226-
* fails, it attempts to get the synth handle from the parser and
227-
* get the parameter directly on the synth.
228-
*----------------------------------------------------------------------------
229-
*/
230-
EAS_RESULT EAS_IntGetStrmParam (S_EAS_DATA *pEASData, EAS_HANDLE pStream, EAS_INT param, EAS_I32 *pValue)
231-
{
232-
S_SYNTH *pSynth;
233-
234-
/* try to set the parameter */
235-
if (EAS_GetStreamParameter(pEASData, pStream, param, pValue) == EAS_SUCCESS)
236-
return EAS_SUCCESS;
237-
238-
/* get a pointer to the synth object and retrieve data directly */
239-
/*lint -e{740} we are cheating by passing a pointer through this interface */
240-
if (EAS_GetStreamParameter(pEASData, pStream, PARSER_DATA_SYNTH_HANDLE, (EAS_I32*) &pSynth) != EAS_SUCCESS)
241-
return EAS_ERROR_INVALID_PARAMETER;
242-
243-
if (pSynth == NULL)
244-
return EAS_ERROR_INVALID_PARAMETER;
245-
246-
switch (param)
247-
{
248-
case PARSER_DATA_POLYPHONY:
249-
return VMGetPolyphony(pEASData->pVoiceMgr, pSynth, pValue);
250-
251-
case PARSER_DATA_PRIORITY:
252-
return VMGetPriority(pEASData->pVoiceMgr, pSynth, pValue);
253-
254-
case PARSER_DATA_TRANSPOSITION:
255-
VMGetTranposition(pSynth, pValue);
256-
break;
257-
258-
case PARSER_DATA_NOTE_COUNT:
259-
*pValue = VMGetNoteCount(pSynth);
260-
break;
261-
262-
default:
263-
{ /* dpp: EAS_ReportEx(_EAS_SEVERITY_ERROR, "Invalid paramter %d in call to EAS_IntSetStrmParam", param); */ }
264-
return EAS_ERROR_INVALID_PARAMETER;
265-
}
266-
267-
return EAS_SUCCESS;
268-
}
269-
270221
/*----------------------------------------------------------------------------
271222
* EAS_AllocateStream()
272223
*----------------------------------------------------------------------------
@@ -683,23 +634,6 @@ EAS_PUBLIC EAS_RESULT EAS_OpenFile (EAS_DATA_HANDLE pEASData, EAS_FILE_LOCATOR l
683634
return EAS_ERROR_UNRECOGNIZED_FORMAT;
684635
}
685636

686-
/*----------------------------------------------------------------------------
687-
* EAS_GetFileType
688-
*----------------------------------------------------------------------------
689-
* Returns the file type (see eas_types.h for enumerations)
690-
*----------------------------------------------------------------------------
691-
* pEASData - pointer to EAS persistent data object
692-
* pStream - stream handle
693-
* pFileType - pointer to variable to receive file type
694-
*----------------------------------------------------------------------------
695-
*/
696-
EAS_PUBLIC EAS_RESULT EAS_GetFileType (S_EAS_DATA *pEASData, EAS_HANDLE pStream, EAS_I32 *pFileType)
697-
{
698-
if (!EAS_StreamReady (pEASData, pStream))
699-
return EAS_ERROR_NOT_VALID_IN_THIS_STATE;
700-
return EAS_GetStreamParameter(pEASData, pStream, PARSER_DATA_FILE_TYPE, pFileType);
701-
}
702-
703637
/*----------------------------------------------------------------------------
704638
* EAS_Prepare()
705639
*----------------------------------------------------------------------------
@@ -825,18 +759,21 @@ EAS_PUBLIC EAS_RESULT EAS_Render (EAS_DATA_HANDLE pEASData, EAS_PCM *pOut, EAS_I
825759
/* establish pointer to parser module */
826760
pParserModule = pEASData->streams[streamNum].pParserModule;
827761

762+
#ifdef JET_INTERFACE
828763
/* handle pause */
829764
if (pEASData->streams[streamNum].streamFlags & STREAM_FLAGS_PAUSE)
830765
{
831766
if (pParserModule->pfPause)
832767
result = pParserModule->pfPause(pEASData, pEASData->streams[streamNum].handle);
833768
pEASData->streams[streamNum].streamFlags &= ~STREAM_FLAGS_PAUSE;
834769
}
770+
#endif
835771

836772
/* get current state */
837773
if ((result = (*pParserModule->pfState)(pEASData, pEASData->streams[streamNum].handle, &parserState)) != EAS_SUCCESS)
838774
return result;
839775

776+
#ifdef JET_INTERFACE
840777
/* handle resume */
841778
if (parserState == EAS_STATE_PAUSED)
842779
{
@@ -847,6 +784,7 @@ EAS_PUBLIC EAS_RESULT EAS_Render (EAS_DATA_HANDLE pEASData, EAS_PCM *pOut, EAS_I
847784
pEASData->streams[streamNum].streamFlags &= ~STREAM_FLAGS_RESUME;
848785
}
849786
}
787+
#endif
850788

851789
/* if necessary, parse stream */
852790
if ((pEASData->streams[streamNum].streamFlags & STREAM_FLAGS_PARSED) == 0)
@@ -1001,6 +939,7 @@ EAS_PUBLIC EAS_RESULT EAS_Render (EAS_DATA_HANDLE pEASData, EAS_PCM *pOut, EAS_I
1001939
return EAS_SUCCESS;
1002940
}
1003941

942+
#ifdef JET_INTERFACE
1004943
/*----------------------------------------------------------------------------
1005944
* EAS_SetTransposition)
1006945
*----------------------------------------------------------------------------
@@ -1031,6 +970,7 @@ EAS_PUBLIC EAS_RESULT EAS_SetTransposition (EAS_DATA_HANDLE pEASData, EAS_HANDLE
1031970
return EAS_ERROR_NOT_VALID_IN_THIS_STATE;
1032971
return EAS_IntSetStrmParam(pEASData, pStream, PARSER_DATA_TRANSPOSITION, transposition);
1033972
}
973+
#endif
1034974

1035975
/*----------------------------------------------------------------------------
1036976
* EAS_ParseEvents()
@@ -1484,6 +1424,7 @@ EAS_PUBLIC EAS_RESULT EAS_GetLocation (EAS_DATA_HANDLE pEASData, EAS_HANDLE pStr
14841424
return EAS_SUCCESS;
14851425
}
14861426

1427+
#ifdef JET_INTERFACE
14871428
/*----------------------------------------------------------------------------
14881429
* EAS_Pause()
14891430
*----------------------------------------------------------------------------
@@ -1600,6 +1541,7 @@ EAS_PUBLIC EAS_RESULT EAS_Resume (EAS_DATA_HANDLE pEASData, EAS_HANDLE pStream)
16001541

16011542
return result;
16021543
}
1544+
#endif
16031545

16041546
/*----------------------------------------------------------------------------
16051547
* EAS_SetParameter()

arm-wt-22k/lib_src/eas_rtttl.c

+7
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,13 @@ const S_FILE_PARSER_INTERFACE EAS_RTTTL_Parser =
105105
RTTTL_State,
106106
RTTTL_Close,
107107
RTTTL_Reset,
108+
#ifdef JET_INTERFACE
108109
RTTTL_Pause,
109110
RTTTL_Resume,
111+
#else
112+
NULL,
113+
NULL,
114+
#endif
110115
NULL,
111116
RTTTL_SetData,
112117
RTTTL_GetData,
@@ -626,6 +631,7 @@ static EAS_RESULT RTTTL_Reset (S_EAS_DATA *pEASData, EAS_VOID_PTR pInstData)
626631
return EAS_SUCCESS;
627632
}
628633

634+
#ifdef JET_INTERFACE
629635
/*----------------------------------------------------------------------------
630636
* RTTTL_Pause()
631637
*----------------------------------------------------------------------------
@@ -689,6 +695,7 @@ static EAS_RESULT RTTTL_Resume (S_EAS_DATA *pEASData, EAS_VOID_PTR pInstData)
689695
pData->state = EAS_STATE_PLAY;
690696
return EAS_SUCCESS;
691697
}
698+
#endif
692699

693700
/*----------------------------------------------------------------------------
694701
* RTTTL_SetData()

arm-wt-22k/lib_src/eas_smf.c

+7
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,13 @@ const S_FILE_PARSER_INTERFACE EAS_SMF_Parser =
7979
SMF_State,
8080
SMF_Close,
8181
SMF_Reset,
82+
#ifdef JET_INTERFACE
8283
SMF_Pause,
8384
SMF_Resume,
85+
#else
86+
NULL,
87+
NULL,
88+
#endif
8489
NULL,
8590
SMF_SetData,
8691
SMF_GetData,
@@ -528,6 +533,7 @@ EAS_RESULT SMF_Reset (S_EAS_DATA *pEASData, EAS_VOID_PTR pInstData)
528533
return EAS_SUCCESS;
529534
}
530535

536+
#ifdef JET_INTERFACE
531537
/*----------------------------------------------------------------------------
532538
* SMF_Pause()
533539
*----------------------------------------------------------------------------
@@ -591,6 +597,7 @@ EAS_RESULT SMF_Resume (S_EAS_DATA *pEASData, EAS_VOID_PTR pInstData)
591597
pSMFData->state = EAS_STATE_PLAY;
592598
return EAS_SUCCESS;
593599
}
600+
#endif
594601

595602
/*----------------------------------------------------------------------------
596603
* SMF_SetData()

0 commit comments

Comments
 (0)