Merge pull request #9109 from parleur/minorbug

Prevent bad `__bool__` from  crashing pinfo

IPython/core/oinspect.py

@@ -832,7 +832,7 b' class Inspector:'
         else:
             callable_obj = None
 
-        if callable_obj:
+        if callable_obj is not None:
             try:
                 argspec = getargspec(callable_obj)
             except (TypeError, AttributeError):

IPython/core/tests/test_oinspect.py

@@ -172,6 +172,19 b' class Awkward(object):'
     def __getattr__(self, name):
         raise Exception(name)
 
+class NoBoolCall:
+    """
+    callable with `__bool__` raising should still be inspect-able.
+    """
+
+    def __call__(self):
+        """does nothing"""
+        pass
+
+    def __bool__(self):
+        """just raise NotImplemented"""
+        raise NotImplementedError('Must be implemented')
+
 
 def check_calltip(obj, name, call, docstring):
     """Generic check pattern all calltip tests will use"""
@@ -281,6 +294,9 b' def test_info_awkward():'
     # Just test that this doesn't throw an error.
     i = inspector.info(Awkward())
 
+def test_bool_raise():
+    inspector.info(NoBoolCall())
+
 def test_calldef_none():
     # We should ignore __call__ for all of these.
     for obj in [f, SimpleClass().method, any, str.upper]: