New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[TIMOB-23925] iOS: Wrap platform-API's in precompile-macros #8456
Conversation
… TIMOB-23925 # Conflicts: # iphone/Classes/MediaModule.m
@@ -205,7 +214,8 @@ -(NSString*)apiName | |||
MAKE_SYSTEM_PROP_DEPRECATED_REPLACED_REMOVED(audioLineType,-10,@"Media.audioLineType",@"3.4.2",@"3.6.0",@"Media.currentRoute"); | |||
|
|||
//Constants for currentRoute | |||
MAKE_SYSTEM_STR(AUDIO_SESSION_PORT_LINEIN,AVAudioSessionPortLineIn) | |||
#ifdef USE_TI_MEDIAAUDIOPLAYER | |||
MAKE_SYSTEM_STR(text,AVAudioSessionPortLineIn) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I maybe wrong, but this can't be text
right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! Addressed.
|
||
#import "TiMediaAudioPlayerProxy.h" | ||
#import "TiUtils.h" | ||
#ifdef USE_TI_MEDIAAUDIOSESSION |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using KitchenSink, the build fails because of this. Apparently, USE_TI_MEDIAAUDIOSESSION
will not be generated in defines.h when building to device. commenting this out makes it build normally.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed.
#define USE_TI_MEDIAISMEDIATYPESUPPORTED | ||
#define USE_TI_MEDIASAVETOGALLERY | ||
#define USE_TI_MEDIASTARTVIDEOEDITING | ||
#define USE_TI_MEDIAAUDIOSESSION |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Based on testing on device using KitchenSInk, This define is not necessary since it doesn't get generated when building to device, even when the app manages audio player.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed.
KitchenSink builds and Media-related code runs fine after these issues were addressed. |
CR and FT passed! |
JIRA: https://jira.appcelerator.org/browse/TIMOB-23925