Skip to content

/ go

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: gobind drops package function comments #32470

Closed
i9 opened this issue Jun 6, 2019 · 2 comments
Closed

x/mobile: gobind drops package function comments #32470

i9 opened this issue Jun 6, 2019 · 2 comments
Labels
NeedsFix mobile
Milestone
Unreleased

Comments

@i9
Copy link

@i9 i9 commented Jun 6, 2019

What version of Go are you using (go version)?

$ go version
go version go1.11.10 darwin/amd64

Does this issue reproduce with the latest release?

yes

What did you do?

gomobile bind to generate Android .aar and sources.jar

What did you expect to see?

Package level function comments are kept as javadoc

What did you see instead?

Package level function comments are ignored in generated java file
@gopherbot gopherbot added this to the Unreleased milestone Jun 6, 2019
@gopherbot gopherbot added the mobile label Jun 6, 2019
@i9

This comment has been minimized.

Sign in to view
Copy link
Author

@i9 i9 commented Jun 6, 2019

identify the cause at bind/genjava.go

https://go-review.googlesource.com/c/mobile/+/181038 for review
@gopherbot

This comment has been minimized.

Sign in to view
Copy link

@gopherbot gopherbot commented Jun 6, 2019

Change https://golang.org/cl/181038 mentions this issue: bind/genjava: keep package function comments in java file
@hyangah hyangah closed this Jun 8, 2019
@dmitshur dmitshur added the NeedsFix label Jun 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
NeedsFix mobile
Projects
None yet
Milestone
  Unreleased
4 participants
@i9 @gopherbot @dmitshur @hyangah
You can’t perform that action at this time.