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.

Sign up for GitHub

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

Binding generation broken with c-for-go/issue#31 #9

Closed
xlab opened this issue Aug 28, 2017 · 2 comments
Closed

Binding generation broken with c-for-go/issue#31 #9

xlab opened this issue Aug 28, 2017 · 2 comments

Comments

@xlab
Copy link
Owner

xlab commented Aug 28, 2017

No description provided.

@xlab
Copy link
Owner Author

xlab commented Aug 28, 2017 •
edited
Loading

xlab/c-for-go#31 related commit d9011c637a87db1e99ec6b05812b418600dae0a1 breaks code generation

# github.com/xlab/pocketsphinx-go/pocketsphinx
../../pocketsphinx/cgo_helpers.go:1445: undefined: sizeOfStructArgValue
../../pocketsphinx/cgo_helpers.go:1445: invalid array bound m / sizeOfStructArgValue
../../pocketsphinx/pocketsphinx.go:298: undefined: Astar
../../pocketsphinx/pocketsphinx.go:306: undefined: Astar
../../pocketsphinx/pocketsphinx.go:860: undefined: LatlinkList
../../pocketsphinx/pocketsphinx.go:868: undefined: LatlinkList
../../pocketsphinx/pocketsphinx.go:886: undefined: LatlinkList
../../pocketsphinx/pocketsphinx.go:1069: undefined: NgramFileTypeE
../../pocketsphinx/pocketsphinx.go:1077: undefined: NgramFileTypeE
../../pocketsphinx/pocketsphinx.go:1884: undefined: HashIter
../../pocketsphinx/cgo_helpers.go:1445: too many errors

Basically forces to use underlying type for all typedefs, e.g. sizeOfArgValue becomes sizeOfStructArgValue and typedef struct ps_astar_s ps_nbest_t; is processed as Astar instead of Nbest.

@xlab xlab changed the title Binding generation broken with d9011c637a87db1e99ec6b05812b418600dae0a1 Binding generation broken with c-for-go/issue#100 Aug 28, 2017
@xlab xlab changed the title Binding generation broken with c-for-go/issue#100 Binding generation broken with c-for-go/issue#31 Aug 28, 2017
xlab added a commit to xlab/c-for-go that referenced this issue Aug 28, 2017
@xlab
Revert "Workaround issue #100 in cc"
9ae9e8c 
See xlab/pocketsphinx-go#9

This reverts commit d9011c6.
@xlab
Copy link
Owner Author

xlab commented Aug 28, 2017

Closing, as change has been reverted & this issue is fixed.

@xlab xlab closed this as completed Aug 28, 2017
@xlab xlab mentioned this issue Aug 28, 2017
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@xlab