[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-bug-tracker] [bug #47464] audioformats: a new function to get in
From: |
Mike Miller |
Subject: |
[Octave-bug-tracker] [bug #47464] audioformats: a new function to get information on supported audio formats |
Date: |
Wed, 30 Mar 2016 19:42:06 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Firefox/45.0 |
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/