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/bind: A field named be "seq" will make gomobile bind fail to build code #14168

Closed
ghost opened this issue Jan 31, 2016 · 1 comment
Closed

x/mobile/bind: A field named be "seq" will make gomobile bind fail to build code #14168

ghost opened this issue Jan 31, 2016 · 1 comment
Labels
FrozenDueToAge

Comments

@ghost
Copy link

ghost commented Jan 31, 2016

Hi, if I have a filed name "seq", the gomobile will report a error message:

Example:
type ChatClientEvent interface {
ChatMessage2(seq int32, from string, msg string)
}

Error message when "gomobile bind -target=android xxxx"

gomobile: go build -p=8 -pkgdir=/Users/holden/go/pkg/gomobile/pkg_android_arm -tags="" -buildmode=c-shared -o=/var/folders/r6/8n2n5zfj1qz_09jf1_yh2qgc0000gn/T/gomobile-work-011628298/android/src/main/jniLibs/armeabi-v7a/libgojni.so /var/folders/r6/8n2n5zfj1qz_09jf1_yh2qgc0000gn/T/gomobile-work-011628298/androidlib/main.go failed: exit status 2

_/var/folders/r6/8n2n5zfj1qz_09jf1_yh2qgc0000gn/T/gomobile-work-011628298/go_client

/var/folders/r6/8n2n5zfj1qz_09jf1_yh2qgc0000gn/T/gomobile-work-011628298/go_client/go_clientmain.go:234: seq.Buffer undefined (type int has no field or method Buffer)
/var/folders/r6/8n2n5zfj1qz_09jf1_yh2qgc0000gn/T/gomobile-work-011628298/go_client/go_clientmain.go:238: seq.Transact undefined (type int has no field or method Transact)
/var/folders/r6/8n2n5zfj1qz_09jf1_yh2qgc0000gn/T/gomobile-work-011628298/go_client/go_clientmain.go:238: seq.Ref undefined (type int has no field or method Ref)
@gopherbot
Copy link

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

@golang golang locked and limited conversation to collaborators Feb 28, 2017
imWildCat pushed a commit to imWildCat/go-mobile that referenced this issue Apr 10, 2021
@hyangah
bind: rename seq package with name _seq in generated Go code
d74c6d5 
Avoid taking a good name (seq) away from users.

Fixes golang/go#14168

Change-Id: I88e90cb74b479e348c642a1caa27096ed4a6d68e
Reviewed-on: https://go-review.googlesource.com/19601
Reviewed-by: David Crawshaw <crawshaw@golang.org>
imWildCat pushed a commit to imWildCat/go-mobile that referenced this issue Apr 11, 2021
@hyangah
bind: rename seq package with name _seq in generated Go code
0f12aef 
Avoid taking a good name (seq) away from users.

Fixes golang/go#14168

Change-Id: I88e90cb74b479e348c642a1caa27096ed4a6d68e
Reviewed-on: https://go-review.googlesource.com/19601
Reviewed-by: David Crawshaw <crawshaw@golang.org>
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
No milestone
Development

No branches or pull requests
1 participant
@gopherbot