# HG changeset patch
# User John W. Eaton <jwe@octave.org>
# Date 1207242828 14400
# Node ID 2f659a9c89f2a951bb50023a3ff6f9a46dfe838f
# Parent  de826649dfa23399c1846e69f58f4e1eb84927fa
octave_fcn_handle::subsref: don't call next_subsref

diff -r de826649dfa2 -r 2f659a9c89f2 src/ChangeLog
--- a/src/ChangeLog	Thu Apr 03 10:24:18 2008 -0400
+++ b/src/ChangeLog	Thu Apr 03 13:13:48 2008 -0400
@@ -1,3 +1,8 @@
+2008-04-03  John W. Eaton  <jwe@octave.org>
+
+	* ov-fcn-handle.cc (octave_fcn_handle::subsref):
+	Don't call next_subsref here.
+
 2008-04-03  Jaroslav Hajek <highegg@gmail.com>
 
 	* DLD-FUNCTIONS/sort.cc (xabs, ascending_compare, operator <,
diff -r de826649dfa2 -r 2f659a9c89f2 src/ov-fcn-handle.cc
--- a/src/ov-fcn-handle.cc	Thu Apr 03 10:24:18 2008 -0400
+++ b/src/ov-fcn-handle.cc	Thu Apr 03 13:13:48 2008 -0400
@@ -139,12 +139,8 @@
       panic_impossible ();
     }
 
-  // FIXME -- perhaps there should be an
-  // octave_value_list::next_subsref member function?  See also
-  // octave_builtin::subsref.
-
-  if (idx.size () > 1)
-    retval = retval(0).next_subsref (nargout, type, idx);
+  // There's no need to call next_subsref here --
+  // octave_function::subsref will handle that for us.
 
   return retval;
 }