[Octave-bug-tracker] [bug #46536] Bad indexing could produce more informative error messages

[John W. Eaton]

Follow-up Comment #9, bug #46536 (project octave):

I generally don't like the idea of returning const reference.  Currently that
would work because the object being returned is a tree_index_expression data
member, so it will be live for as long as expr is (in the context we are
discussing).  But what if tree_index_expression::arg_names were changed so
that instead of returning a data member, it created a temporary value to
return?  Then I don't think returning a const reference to a temporary would
help you.  In that case I don't think there would be any guarantee that the
reference would be valid after the arg_names method returns.  So please leave
it returning a std::list<> object and just make a temporary that has the
lifetime you need.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?46536>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/