changeset 7689:a9d25da4ed9c

octave_fcn_handle::subsref: don't call next_subsref
author John W. Eaton <jwe@octave.org>
date Thu, 03 Apr 2008 13:13:01 -0400
parents 5502b815a556
children 97e535ec65db
files src/ChangeLog src/ov-fcn-handle.cc
diffstat 2 files changed, 7 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/src/ChangeLog	Thu Apr 03 14:19:59 2008 +0200
+++ b/src/ChangeLog	Thu Apr 03 13:13:01 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-01  John W. Eaton  <jwe@octave.org>
 
 	* pt-id.h (do_lookup (bool&, bool)): Delete.
--- a/src/ov-fcn-handle.cc	Thu Apr 03 14:19:59 2008 +0200
+++ b/src/ov-fcn-handle.cc	Thu Apr 03 13:13:01 2008 -0400
@@ -104,12 +104,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;
 }