octave-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-bug-tracker] [bug #41818] Fixed up "find" function with template


From: Lachlan Andrew
Subject: [Octave-bug-tracker] [bug #41818] Fixed up "find" function with templates and fourth parameter
Date: Mon, 08 Feb 2016 14:06:56 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:43.0) Gecko/20100101 Firefox/43.0

Follow-up Comment #2, bug #41818 (project octave):

David, do you plan to respond to Rik's request?

Also, what is the main benefit of your patch?  Is it smaller code, faster
execution, easier maintenance or just the elegance of having a common
template?

You note that min() isn't in the code; the code uses std::min in some places.

It would be nice for each function or template to have a brief comment
explaining why it exists.  Most are so simple that the reader can see *what*
and *how*, but it is useful to describe *why* -- what does the function
enable?

    _______________________________________________________

Reply to this item at:

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

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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]