gopy
gopy copied to clipboard
[Draft] Support for multiple return values from go function to python
Support for multiple return values from go function to python
Some memory leaks exist.
Yeah, would it make sense to delete this PR for the time being and rebase on the variadic version which I will test now. The variadic one does not have any leaks?
you can resubmit a PR once you have it working with out leaks I mean.
Sure. Variadic shouldn't have leaks. I may take up fixing of memory leaks later as per internal priority.
Actually, it may still make sense to review from the perspective of behavior. Esp. translating error return into exceptions. Also, deleting PRs causes context to be lost. Will suggest making it a draft PR.
ok.. I'll try to take a look in a few days. Meanwhile in the future, please run your code through gofmt -- that check is failing on test.
ps. thanks for submitting these fixes -- much appreciated!
URL changes are temporary. Made them to test. Will revert before switching PR draft status. JFYI.