changeset 11730:2f659a9c89f2 release-3-0-x

octave_fcn_handle::subsref: don't call next_subsref
author John W. Eaton <jwe@octave.org>
date Thu, 03 Apr 2008 13:13:48 -0400
parents de826649dfa2
children cf2fc7338195
files src/ChangeLog src/ov-fcn-handle.cc
diffstat 2 files changed, 7 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -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 <,
--- a/src/ov-fcn-handle.cc
+++ b/src/ov-fcn-handle.cc
@@ -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;
 }