[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: |
Tue, 19 Apr 2016 23:08:26 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Firefox/45.0 |
Follow-up Comment #7, bug #47464 (project octave):
The 3rd revision is much clearer and better.
Some minor coding style issues, please pay attention to whitespace, and have
your editor delete trailing whitespace if that's an option, you have several
blank lines that contain indented whitespace.
* Consistently use space after function name and before opening parenthesis,
but not when indexing an array.
* Always use space around operators, e.g. "<", "<=", "=", "==", "!=", and "<<"
in this patch.
The function should be named "audioformats". It also needs some actual usage
documentation in its texinfo header.
This is a pretty good submission. If you'd like to try to work on some of
these remaining issues, please do, but I can also take it as is and clean it
up myself.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/bugs/?47464>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/