Skip to content
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

Jump to bottom

x/mobile/cmd/gobind: Go's int is always exported as int (32bit) in Objective-C #16182

Closed
hajimehoshi opened this issue Jun 25, 2016 · 2 comments
Closed

x/mobile/cmd/gobind: Go's int is always exported as int (32bit) in Objective-C #16182

hajimehoshi opened this issue Jun 25, 2016 · 2 comments
Labels
FrozenDueToAge
Milestone
Unreleased

Comments

@hajimehoshi
Copy link
Member

Whether the architecture is 32bit or 64bit in Objective-C side, Go's int is exported as Objective-C's int, which is 32bit.
@hyangah
Copy link
Contributor

hyangah commented Jun 29, 2016

Any objection to change to long?
On the other hand, Go's int type is mapped to long for java (android), which causes a different issue #11421. Fortunately, for iOS, it seems 'long' matches Go's int type. Best practice is to avoid int type though.

@hyangah hyangah added this to the Unreleased milestone Jul 7, 2016
@gopherbot
Copy link

CL https://golang.org/cl/24792 mentions this issue.

@golang golang locked and limited conversation to collaborators Jul 7, 2017
imWildCat pushed a commit to imWildCat/go-mobile that referenced this issue Apr 10, 2021
@hyangah
bind: map Go's int type to Objective-C's long type
4dbdde7 
https://developer.apple.com/library/ios/documentation/General/Conceptual/CocoaTouch64BitGuide/Major64-BitChanges/Major64-BitChanges.html

Fixes golang/go#16182

Change-Id: I312749e8a7113bf9231eb3896f801e450f46a410
Reviewed-on: https://go-review.googlesource.com/24792
Reviewed-by: Elias Naur <elias.naur@gmail.com>
imWildCat pushed a commit to imWildCat/go-mobile that referenced this issue Apr 11, 2021
@hyangah
bind: map Go's int type to Objective-C's long type
912e014 
https://developer.apple.com/library/ios/documentation/General/Conceptual/CocoaTouch64BitGuide/Major64-BitChanges/Major64-BitChanges.html

Fixes golang/go#16182

Change-Id: I312749e8a7113bf9231eb3896f801e450f46a410
Reviewed-on: https://go-review.googlesource.com/24792
Reviewed-by: Elias Naur <elias.naur@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
Unreleased
Development

No branches or pull requests
3 participants
@hajimehoshi @hyangah @gopherbot