Fixes #148

src/libpython_clj2/python/base.clj

@@ -10,10 +10,10 @@
 (defn ->jvm
   "Copying conversion to the jvm."
   ([obj]
-   (when obj
+   (when-not (nil? obj)
      (py-proto/->jvm obj nil)))
   ([obj opts]
-   (when obj
+   (when-not (nil? obj)
      (py-proto/->jvm obj opts))))
 
 
@@ -22,7 +22,7 @@
   ([obj]
    (as-jvm obj nil))
   ([obj opts]
-   (when obj
+   (when-not (nil? obj)
      (py-proto/as-jvm obj opts))))
 
 

src/libpython_clj2/python/ffi.clj

@@ -876,8 +876,7 @@ Object's refcount is bad.  Crash is imminent"
         :str (with-decref [pyobj pyobj]
                (pystr->str pyobj))
         :bool (with-decref [pyobj pyobj]
-                (= (dt-ffi/->pointer pyobj)
-                   (py-true)))
+                (== 1 (long (PyObject_IsTrue pyobj))))
         ;;maybe copy, maybe bridge - in any case we have to decref the item
         (track-pyobject pyobj)))
     (check-error-throw)))

src/libpython_clj2/python/fn.clj

@@ -115,7 +115,8 @@
   py-proto/PyCall
   (call [callable arglist kw-arg-map]
     (call-py-fn callable arglist kw-arg-map py-base/->python))
-  (marshal-return [callable retval] retval))
+  (marshal-return [callable retval]
+    retval))
 
 
 (defn call
@@ -137,11 +138,14 @@
   (py-ffi/with-gil
     (if (string? att-name)
       (py-ffi/with-decref [attval (py-ffi/PyObject_GetAttrString item att-name)]
+        (when-not attval (py-ffi/check-error-throw))
         (->> (call-py-fn attval arglist kw-map arg-converter)
              (py-proto/marshal-return item)))
       (py-ffi/with-decref
         [att-name (py-ffi/untracked->python att-name py-base/->python)
          att-val (py-ffi/untracked->python att-name py-base/->python)]
+        (when (or (nil? att-name) (nil? att-val))
+          (py-ffi/check-error-throw))
         (->> (call-py-fn att-val arglist kw-map arg-converter)
              (py-proto/marshal-return item))))))
 

test/libpython_clj2/python_test.clj

@@ -384,6 +384,12 @@ class Foo:
          (py/->jvm (py/->python \c)))))
 
 
+(deftest numpy-all
+  (let [np (py/import-module "numpy")]
+    (is (= true (py/call-attr np "all" (py/call-attr np "array" [true true true]))))
+    (is (= false (py/call-attr np "all" (py/call-attr np "array" [true false true]))))))
+
+
 (comment
   (require '[libpython-clj.require :refer [require-python]])