bug fix in setVertexNormal; edit docs

Author: peteroupc

glutil-eval.js

@@ -256,9 +256,8 @@ function CurveEval(){
 /**
 * Specifies a parametric curve function for generating vertex positions.
 * @param {object} evaluator An object that must contain a function
-* named "evaluate".  It takes the following parameters in this order:<ul>
-* <li><code>u</code> - Horizontal-axis coordinate, generally from 0 to 1.
-* <li><code>v</code> - Vertical-axis coordinate, generally from 0 to 1.
+* named "evaluate".  It takes the following parameter:<ul>
+* <li><code>u</code> - A curve coordinate, generally from 0 to 1.
 * </ul>
 * The evaluator function returns an array of the result of the evaluation.
 * @return {CurveEval} This object.
@@ -413,7 +412,7 @@ CurveEval.prototype.evalOne=function(mesh,u){
  * @param {number} [n] Number of subdivisions of the curve to be drawn.
  * May be omitted; default is 24.
  * @param {number} [u1] Starting point of the curve (within the range
- * given in the <code>vector</code>, normal</code>,
+ * given in the <code>vector</code>, <code>normal</code>,
  * <code>color</code>, and <code>texCoord</code> methods).
  *May be omitted; default is 0.
  * @param {number} [u2] Ending point of the curve (within the range

glutil-mesh.js

@@ -297,7 +297,8 @@ Mesh.prototype.normal3=function(x,y,z){
  /**
   * Transforms the positions and normals of all the vertices currently
   * in this mesh, using a 4x4 matrix.  The matrix won't affect
-  * vertices added afterwards.
+  * vertices added afterwards.  Future vertices should not be
+  * added after calling this method without calling mode() first.
   * @param {Array<number>} matrix A 4x4 matrix describing
   * the transformation.
   * @return {glutil.Mesh} This object.
@@ -520,8 +521,8 @@ Mesh.prototype.setVertex=function(index,x,y,z){
 }
 /**
  * Sets the normal associated with the vertex with the
- * given index.  Has no effect if the index exceeds the number
- * of vertices in the mesh.
+ * given index.  Has no effect if the index is less than 0 or
+ * equals the number of vertices in this mesh or greater.
  * @param {number} index Zero-based index of
  * the vertex to set.
  * The index ranges from 0 to less than
@@ -538,6 +539,7 @@ Mesh.prototype.setVertex=function(index,x,y,z){
  * @return {glutil.Mesh} This object.
  */
 Mesh.prototype.setVertexNormal=function(index,x,y,z){
+  if(index<0)return this;
   var count=0;
   if(typeof y=="undefined" && typeof z=="undefined"){
    y=x[1];
@@ -864,6 +866,11 @@ SubMesh.prototype.transform=function(matrix){
      v[i+normalOffset+2]=xform[2];
     }
   }
+  // TODO: Planned for 2.0.  Once implemented,
+  // Mesh#transform will say:  "Also, resets the primitive
+  // mode (see {@link glutil.Mesh#mode}) so that future vertices given
+  // will not build upon previous vertices."
+  //this.newPrimitive();
   return this;
 }
 

glutil.js

@@ -156,13 +156,13 @@ var GLUtil={
  * @return {Promise} A promise that is never rejected and resolves when
 * all of the promises are each resolved or rejected. The result
  * of the promise will be an object with
- * three keys:
- *  "successes" - contains a list of results from the
+ * three keys:<ul>
+ *  <li>"successes" - contains a list of results from the
  *  promises that succeeded, in the order in which those promises were listed.
- *  "failures" - contains a list of results from the
+ *  <li>"failures" - contains a list of results from the
  *  promises that failed, in the order in which those promises were listed.
- *  "results" - contains a list of boolean values for each
- * promise, in the order in which the promises were listed.
+ *  <li>"results" - contains a list of boolean values for each
+ * promise, in the order in which the promises were listed.</ul>
  * True means success, and false means failure.
  */
 "getPromiseResults":function(promises,