Author: troycurtisjr
Date: Wed Dec 27 02:43:05 2017
New Revision: 1819312

URL: http://svn.apache.org/viewvc?rev=1819312&view=rev
Log:
On branch swig-py3: Do not hide AttributeError exception in called function.

* subversion/bindings/swig/include/proxy.swg
  (_assert_valid_deep): Move the call to assert_valid() out of the try block to
   prevent masking AttributeError that happen within the method call.

Modified:
    subversion/branches/swig-py3/subversion/bindings/swig/include/proxy.swg

Modified: 
subversion/branches/swig-py3/subversion/bindings/swig/include/proxy.swg
URL: 
http://svn.apache.org/viewvc/subversion/branches/swig-py3/subversion/bindings/swig/include/proxy.swg?rev=1819312&r1=1819311&r2=1819312&view=diff
==============================================================================
--- subversion/branches/swig-py3/subversion/bindings/swig/include/proxy.swg 
(original)
+++ subversion/branches/swig-py3/subversion/bindings/swig/include/proxy.swg Wed 
Dec 27 02:43:05 2017
@@ -59,9 +59,11 @@
     # assert_valid attribute, but it can not be called without an instance.
     elif type(value) != type:
       try:
-        value.assert_valid()
+        fn = value.assert_valid
       except AttributeError:
         pass
+      else:
+        fn()
 %}
 
 /* Default code for all wrapped proxy classes in Python.


Reply via email to