doc comments fixes (#222)

* docs: remove @returns from @constructors, add @extends

* docs: fix property types

* various doc comments fixes

* more doc comments fixes, add missing member names, fix some types

* fix up property doc comments

Author: Spongman

src/audioVoice.js

@@ -7,7 +7,7 @@ define(function() {
    * should follow the API and implement the methods below in order to 
    * remain compatible with p5.PolySynth();
    *
-   * @class AudioVoice
+   * @class p5.AudioVoice
    * @constructor
    */
   p5.AudioVoice = function () {

src/compressor.js

@@ -33,7 +33,7 @@ define(function (require) {
      * The p5.Compressor is built with a <a href="https://www.w3.org/TR/webaudio/#the-dynamicscompressornode-interface" 
    *   target="_blank" title="W3 spec for Dynamics Compressor Node">Web Audio Dynamics Compressor Node
    *   </a>
-     * @property {WebAudioNode} compressor 
+     * @property {AudioNode} compressor 
      */
 
 
@@ -205,6 +205,8 @@ define(function (require) {
 
   /**
    * Return the current reduction value
+   *
+   * @method reduction
    * @return {Number} Value of the amount of gain reduction that is applied to the signal
    */
   p5.Compressor.prototype.reduction =function() {

src/effect.js

@@ -18,12 +18,12 @@ define(function (require) {
 	 *
 	 * @class  p5.Effect
 	 * @constructor
-	 *
+	 * 
 	 * @param {Object} [ac]   Reference to the audio context of the p5 object
-	 * @param {WebAudioNode} [input]  Gain Node effect wrapper
-	 * @param {WebAudioNode} [output] Gain Node effect wrapper
+	 * @param {AudioNode} [input]  Gain Node effect wrapper
+	 * @param {AudioNode} [output] Gain Node effect wrapper
 	 * @param {Object} [_drywet]   Tone.JS CrossFade node (defaults to value: 1)
-	 * @param {WebAudioNode} [wet]  Effects that extend this class should connect
+	 * @param {AudioNode} [wet]  Effects that extend this class should connect
 	 *                              to the wet signal to this gain node, so that dry and wet 
 	 *                              signals are mixed properly.
 	 */

src/eq.js

@@ -188,7 +188,7 @@ define(function (require) {
    * @method  _newBand
    * @param  {Number} freq
    * @param  {Number} res
-   * @return {Obect}      Abstracted Filter
+   * @return {Object}      Abstracted Filter
    */
   p5.EQ.prototype._newBand = function(freq, res) {
     return new EQFilter(freq, res);

src/filter.js

@@ -199,7 +199,8 @@ define(function (require) {
    * This is distinctly different from .amp() which is inherited from p5.Effect
    * .amp() controls the volume via the output gain node
    * p5.Filter.gain() controls the gain parameter of a Biquad Filter node.
-   * 
+   *
+   * @method gain
    * @param  {Number} gain 
    * @return {Number} Returns the current or updated gain value
    */
@@ -218,6 +219,8 @@ define(function (require) {
 
   /**
    * Toggle function. Switches between the specified type and allpass
+   *
+   * @method toggle
    * @return {boolean} [Toggle value]
    */
   p5.Filter.prototype.toggle = function() {
@@ -261,7 +264,7 @@ define(function (require) {
    *
    *  @class p5.LowPass
    *  @constructor
-   *  @extends {p5.Filter}
+   *  @extends p5.Filter
    */
   p5.LowPass = function() {
     p5.Filter.call(this, 'lowpass');
@@ -276,7 +279,7 @@ define(function (require) {
    *
    *  @class p5.HighPass
    *  @constructor
-   *  @extends {p5.Filter}
+   *  @extends p5.Filter
    */
   p5.HighPass = function() {
     p5.Filter.call(this, 'highpass');
@@ -289,9 +292,9 @@ define(function (require) {
    *  its method <code>setType('bandpass')</code>.
    *  See p5.Filter for methods.
    *
-   *  @class BandPass
+   *  @class p5.BandPass
    *  @constructor
-   *  @extends {p5.Filter}
+   *  @extends p5.Filter
    */
   p5.BandPass = function() {
     p5.Filter.call(this, 'bandpass');

src/helpers.js

@@ -24,6 +24,7 @@ define(function (require) {
    *  Returns the closest MIDI note value for
    *  a given frequency.
    *
+   *  @method freqToMidi
    *  @param  {Number} frequency A freqeuncy, for example, the "A"
    *                             above Middle C is 440Hz
    *  @return {Number}   MIDI note value

src/listener3d.js

@@ -95,20 +95,22 @@ define(function (require) {
     return this.listener.positionZ.value;
   };
 
-  /**
-   * Overrides the listener orient() method because Listener has slightly
-   * different params. In human terms, Forward vectors are the direction the 
-   * nose is pointing. Up vectors are the direction of the top of the head.
-   * 
-   * @param  {Number} xValF  Forward vector X direction
-   * @param  {Number} yValF  Forward vector Y direction
-   * @param  {Number} zValF  Forward vector Z direction
-   * @param  {Number} xValU  Up vector X direction
-   * @param  {Number} yValU  Up vector Y direction
-   * @param  {Number} zValU  Up vector Z direction
-   * @param  {Number} time  
-   * @return {Array}       All orienation params
-   */
+// cannot define method when class definition is commented
+//  /**
+//   * Overrides the listener orient() method because Listener has slightly
+//   * different params. In human terms, Forward vectors are the direction the 
+//   * nose is pointing. Up vectors are the direction of the top of the head.
+//   *
+//   * @method orient
+//   * @param  {Number} xValF  Forward vector X direction
+//   * @param  {Number} yValF  Forward vector Y direction
+//   * @param  {Number} zValF  Forward vector Z direction
+//   * @param  {Number} xValU  Up vector X direction
+//   * @param  {Number} yValU  Up vector Y direction
+//   * @param  {Number} zValU  Up vector Z direction
+//   * @param  {Number} time  
+//   * @return {Array}       All orienation params
+//   */
   p5.Listener3D.prototype.orient = function(xValF, yValF, zValF, 
                                               xValU, yValU, zValU, time) {
 

src/looper.js

@@ -9,6 +9,7 @@ define(function (require) {
    *  Set the global tempo, in beats per minute, for all
    *  p5.Parts. This method will impact all active p5.Parts.
    *
+   *  @method setBPM
    *  @param {Number} BPM      Beats Per Minute
    *  @param {Number} rampTime Seconds from now
    */
@@ -494,6 +495,7 @@ define(function (require) {
   /**
    *  Set the tempo for all parts in the score
    *
+   *  @method setBPM
    *  @param {Number} BPM      Beats Per Minute
    *  @param {Number} rampTime Seconds from now
    */

src/monosynth.js

@@ -99,7 +99,6 @@ define(function (require) {
      *                                 value in Note/Octave format ("C4", "Eb3"...etc")
      *                                 See <a href = "https://github.com/Tonejs/Tone.js/wiki/Instruments">
      *                                 Tone</a>. Defaults to 440 hz
-     *  @param  {Number} secondsFromNow time from now (in seconds)
      *  @param  {Number} [velocity] velocity of the note to play (ranging from 0 to 1)
      *  @param  {Number} [secondsFromNow]  time from now (in seconds) at which to play
      *  @method  triggerAttack
@@ -159,8 +158,14 @@ define(function (require) {
   /**
    * Getters and Setters
    * @property {Number} attack
+   */
+  /**
    * @property {Number} decay
+   */
+  /**
    * @property {Number} sustain
+   */
+  /**
    * @property {Number} release
    */
   Object.defineProperties(p5.MonoSynth.prototype, {

src/noise.js

@@ -157,6 +157,7 @@ define(function (require) {
    *  Set the amplitude of the noise between 0 and 1.0. Or,
    *  modulate amplitude with an audio signal such as an oscillator.
    *
+   *  @method amp
    *  @param  {Number|Object} volume amplitude between 0 and 1.0
    *                                     or modulating signal/oscillator
    *  @param {Number} [rampTime] create a fade that lasts rampTime