[Octave-patch-tracker] [patch #9976] Missing function: freqspace

[Kai Torben Ohlhus]

Update of patch #9976 (project octave):

                  Status:                    None => Need Info              
                 Summary: Missing function:  freqspace => Missing function:
freqspace

    _______________________________________________________

Follow-up Comment #1:

Thank you for your contribution, they are always welcome.

Are you interested in adapting your code to the Octave style guide?

https://wiki.octave.org/Octave_style_guide

As many maintainers are busy, it might take some time until someone catches up
your great work.

First of all, your function does what it says.  Some observations before I
would consider your function to be ready for inclusion into Octave core. 
Maybe a look at the code of meshgrid might give a good orientation what is
left to do

https://hg.savannah.gnu.org/hgweb/octave/file/d28016d16e9a/scripts/plot/util/meshgrid.m

Style fixes:

- [ ] No use of `end`, use `endif`, etc.
- [ ] No use of `~=`, use `!=`
- [ ] No more than 79 chars per line
- [ ] Leave a space between variables and operators
- [ ] Adapt docstring to latest standards

General necessities:

- [ ] No tests (BIST)
- [ ] No Mercurial patch (NEWS entry, module.mk entries, etc.)

General questions:

- [ ] Do you want to be publicly listed as Octave contributor?
- [ ] Matlab offers only support for two 2D outputs, your function seems to be
N-dimensional.  Are there real use cases justifying the more complicated
code?


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/patch/?9976>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/