1. Mike Miller
  2. pytave
  3. Pull requests

Pull requests

#17 Merged
Repository
Deleted repository
Branch
default (bbf0b7a4e117)
Repository
pytave
Branch
default

Return pyobject from pycall too, similar to pyeval (fixes issue #28)

Author
  1. Abhinav Tripathi
Reviewers
Description
  • pycall.cc: Add the same logic as pyeval to convert return value to pyobject in case of object_convert_exception. Add a test to verify.
  • pyeval.cc: Add a similar test to verify pyobject is returned.
  • Issues #28: pycall does not return Python objects resolved

Comments (2)

    1. Abhinav Tripathi author

      Done. Sorry for the delay. Also I read PEP8 docs and although I couldn't find explicit mention but it seems that there should be no spaces between function name and parentheses. So I changed the embedded python code to object() from object () I hope it's correct.