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-17876][TIMOB-17929] Enable apps with 64 bit slice in titanium #6357
Conversation
Turn on GCC_WARN_64_TO_32_BIT_CONVERSION flag and fix code for the same
Modules targeting the SDK with the new JSCore will need an updated path to the headers
All supported architectures now will depend on the SDK/Xcode combination used to build the application
Resolving merge conflicts here itself so it can be merged back into master easily
@@ -101,7 +101,7 @@ | |||
TiValueRef length = TiObjectGetProperty(jsContext, obj, kTiStringLength, NULL); | |||
double len = TiValueToNumber(jsContext, length, NULL); | |||
NSMutableArray* resultArray = [[NSMutableArray alloc] initWithCapacity:len]; | |||
for (size_t c=0; c<len; ++c) | |||
for (uint c=0; c<len; ++c) | |||
{ |
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.
The variable "c" is uint
but "len" is long
. I'm ok with this, but you're comparing two variable types here.
Code reviewed. Testing now |
This issue only occurs on 64 bit simulators
The CoverFlowView does not work. Found a fix, change all the
|
Ready for review |
Hyper loop CLI hook needs this to be defined
FT pass |
[TIMOB-17876][TIMOB-17929] Enable apps with 64 bit slice in titanium
Changes for
TIMOB-17876
TIMOB-17929
This PR also makes additional changes for the following
TIMOB-17927
TIMOB-18027