[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/
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Octave-bug-tracker] [bug #41818] Fixed up "find" function with templates and fourth parameter,
Lachlan Andrew <=