__findattr__ refactor

src/org/python/core/PyFileDerived.java

 /* Generated file, do not modify.  See jython/src/templates/gderived.py. */
 package org.python.core;
 
 public class PyFileDerived extends PyFile implements Slotted {
 
     public PyObject getSlot(int index) {
         return slots[index];
     }
 
     public void setSlot(int index,PyObject value) {
@@ skipping 954 matching lines @@
             PyType self_type=getType();
             PyObject impl=self_type.lookup("__call__");
             if (impl!=null)
                 return impl.__get__(this,self_type).__call__(args,keywords);
             return super.__call__(args,keywords);
         } finally {
             --ts.recursion_depth;
         }
     }
 
-    public PyObject __findattr__(String name) {
+    public PyObject __findattr_ex__(String name) {
         PyType self_type=getType();
         PyObject getattribute=self_type.lookup("__getattribute__");
         PyString py_name=null;
+        PyException firstAttributeError=null;
         try {
             if (getattribute!=null) {
                 return getattribute.__get__(this,self_type).__call__(py_name=PyString.fromInterned(name));
             } else {
-                return super.__findattr__(name);
+                // XXX: This shouldn't happen, becase PyObject exposes
+                //      __getattribute__. 
+                //
+                //      And, by the way, invoking object__getattribute__ is the 
+                //      common case. We should optimize that to avoid doing the
+                //      '__getattribute__' lookup each time. 
+                PyObject ret=super.__findattr_ex__(name);
+                if (ret!=null) {
+                    return ret;
+                } // else: pass through to __getitem__ invocation
             }
         } catch (PyException e) {
-            if (Py.matchException(e,Py.AttributeError)) {
-                PyObject getattr=self_type.lookup("__getattr__");
-                if (getattr!=null)
-                    try {
-                        return getattr.__get__(this,self_type).__call__(py_name!=null?py_name:PyString.fromInterned(name));
-                    } catch (PyException e1) {
-                        if (!Py.matchException(e1,Py.AttributeError))
-                            throw e1;
-                    }
-                return null;
+            if (!Py.matchException(e,Py.AttributeError)) {
+                throw e;
+            } else {
+                firstAttributeError=e; // saved to avoid swallowing custom AttributeErrors
+            // and pass through to __getattr__ invocation.
             }
-            throw e;
+        }
+        PyObject getattr=self_type.lookup("__getattr__");
+        if (getattr!=null) {
+            return getattr.__get__(this,self_type).__call__(py_name!=null?py_name:PyString.fromInterned(name));
+        }
+        if (firstAttributeError!=null) {
+            throw firstAttributeError;
+        } else {
+            return null;
         }
     }
 
     public void __setattr__(String name,PyObject value) {
         PyType self_type=getType();
         PyObject impl=self_type.lookup("__setattr__");
         if (impl!=null) {
             impl.__get__(this,self_type).__call__(PyString.fromInterned(name),value);
             return;
         }
@@ skipping 123 matching lines @@
         if (impl!=null) {
             PyObject res=impl.__get__(this,self_type).__call__();
             if (!(res instanceof PyString))
                 throw Py.TypeError("__repr__ returned non-string (type "+res.getType().fastGetName()+")");
             return((PyString)res).toString();
         }
         return super.toString();
     }
 
 }