|
| From: | Markus Mützel |
| Subject: | [Octave-bug-tracker] [bug #57471] streamtube.m should use surface object rather than patch object |
| Date: | Sun, 16 Feb 2020 04:47:55 -0500 (EST) |
| User-agent: | Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:74.0) Gecko/20100101 Firefox/74.0 |
Follow-up Comment #9, bug #57471 (project octave):
I also like that idea. And your patch looks good to me.
Could you please split the patch into two parts? One that renames the current
"streamtube" to "ostreamtube" and a second one that adds the Matlab compatible
"streamtube".
Also, I'm not sure if "vertice" is an English word. Could you please replace
it by "vertex" in the doc string?
Maybe it would be nice, too, if the doc strings referred to each other with
@seealso.
Thanks again for your very valuable contributions.
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/bugs/?57471>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/
| [Prev in Thread] | Current Thread | [Next in Thread] |