|
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] |