[Octave-bug-tracker] [bug #47464] audioformats: a new function to get information on supported audio formats

[Mike Miller]

Update of bug #47464 (project octave):

                  Status:                    None => In Progress            

    _______________________________________________________

Follow-up Comment #4:

Very preliminary review, not yet applied/built/tested:

* Style needs a lot of work, this is very important when working in Octave.
Use appropriate spacing, use appropriate brace indentation, use underscores
instead of CamelCase, don't comment out unused code.
* I don't think there's any value to having audioformats defined in an m-file.
You've shown with this latest patch that most of the work is easily done in
C++, so why not just do it all in C++?

I'd say focus more on fixing these basic issues before worrying about things
like tests, error checking, and passing a file format argument.

    _______________________________________________________

Reply to this item at:

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

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