home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #20 / NN_1992_20.iso / spool / comp / graphics / 9529 < prev    next >
Encoding:
Internet Message Format  |  1992-09-08  |  5.5 KB
  1. Path: sparky!uunet!psinntp!eye!erich
  2. From: erich@eye.com (Eric Haines)
  3. Newsgroups: comp.graphics
  4. Subject: Graphics Gems errata
  5. Message-ID: <1992Sep8.105358.10215@eye.com>
  6. Date: Tue, 8 Sep 92 14:53:57 GMT
  7. Sender: erich@eye.com (Eric Haines)
  8. Organization: 3D/EYE, Inc.  Ithaca, NY
  9. Lines: 166
  10.  
  11. Here's something I compiled last night while watching mindless TV.  It should
  12. soon be available on wuarchive.wustl.edu in the graphics books errata section
  13. (/graphics/graphics/books).
  14.  
  15. Eric
  16.  
  17.  
  18. Errata to _Graphics Gems_, first edition, edited by Andrew Glassner
  19. (glassner.pa@xerox.com), Academic Press 1990.  Code available online in
  20. princeton.edu:pub/Graphics/GraphicsGems/Gems.
  21.  
  22. compiled by Eric Haines (erich@eye.com) from author and reader contributions
  23.  
  24. date:  9/12/92
  25.  
  26. -----
  27.  
  28. Errors in the text:
  29.  
  30. p. 3, bottom:  The equation "N . P + c = 0" is better expressed as
  31.     "N . P - c = 0" in order to match Figure 1a.
  32.  
  33. p. 5, V2 Perpendicular:  change "N <- (-Vx, Vy)" to "N <- (-Vy, Vx)"
  34.  
  35. p. 5, V2 Reflect:  change "N <- (-Vy, -Vx)" to "N <- (-Vx, -Vy)"
  36.  
  37. p. 105, last sentence of first paragraph:  "ajacent" to "adjacent".
  38.  
  39. p. 365, last line:  "Kajia" to "Kajiya".
  40.  
  41. p. 395, first paragraph:  change "discussed by Haines (1989)" to "discussed by
  42.     Haines in Glassner (1989)".
  43.  
  44. p. 448, last sentence of second paragraph:  change "and now nearly as simple"
  45.     to "and not nearly as simple".
  46.  
  47. p. 463, second to last line:  change "then alpha <- alpha + pi/2" to "then
  48.     alpha <- pi - alpha".
  49.  
  50. p. 495, equation 5:  this should have an equal sign (=) before the
  51.     plus-or-minus (+/-).
  52.  
  53. p. 499, middle of page:  change "and i,j,K" to "and i,j,k".
  54.  
  55. p. 516, last paragraph:  a reader notes an additional reference which
  56.     predates Berger and Salmon & Slater, namely "The Viewing Transformation,"
  57.     Technical Memo. no. 84, Alvy Ray Smith, Computer Graphics Project,
  58.     Lucasfilm, June 24, 1983 (rev. May 4, 1984).
  59.  
  60. p. 809:  the author of "Approximation of Sweep Surfaces by Tensor Product
  61.     B-Splines" is M. (not J.) Bloomenthal.  The author is correctly
  62.     attributed in the text (page 569).
  63.  
  64. -----
  65.  
  66. The following are errors in the code listings (corrected in the online code at
  67. princeton.edu:pub/Graphics/GraphicsGems/Gems).
  68.  
  69. Serious errors (ones your compiler cannot or may not catch):
  70.  
  71. p. 632:  procedure declarations for routines in the "2D and 2D Vector C
  72.      Library" (next pages) are missing from "GraphicsGems.h", e.g.
  73.         
  74.         double V2SquaredLength() ;
  75.         double V2Length() ;
  76.         Vector2 *V2Negate() ;
  77.         ...
  78.  
  79. p. 640:  V3MulPointByMatrix() does not work.  A separate local Point3 (e.g.
  80.     "Point3 q ;") should be used in place of "p" for assignment and then
  81.     passed back.
  82.  
  83. p. 649, top:  add "#include <math.h>"
  84.  
  85. p. 663, line 45:  change first "+" to "="; should read
  86.               "VnextLeft = (Vleft=VnextLeft) + 1;"
  87.  
  88. p. 714, line 20:  the last "1" in "if (i + 1 < l * 1)" should be an "l"
  89.  
  90. p. 748, line 22:  change "negetive" to "negative"
  91.  
  92. p. 756, line 15:  add space around "=" in "unsigned int *fi=&f;", as some
  93.               compilers think "=&" means "&="
  94.  
  95. p. 766, line 25:  change "det = det4x4( out );" to "det = det4x4( in );"
  96.         throughout:  change "matrix4" to "Matrix4"
  97.  
  98.  
  99. Syntax errors (ones your compiler will catch):
  100.  
  101. p. 650, line 1:  bad end-of comment; delete "/"
  102.  
  103. p. 651, throughout:  Can't use "const" as a variable name, as it is a reserved
  104.              word in ANSI C.  Use "liconst" instead.
  105.  
  106. p. 660, line 11:  header missing end of comment "*/"
  107.  
  108. p. 662, line 13:  change "SYBYRES" to "SUBYRES"
  109.         line 16:  bad space after "MODRES"
  110.         line 42:  change "XRmax" to "xRmax"
  111. p. 665, line 15:  missing semicolon after "int area"
  112.         line 27:  change "O" to "0" in "if (partialArea>O)"
  113. p. 666, line 13:  change "O" to "0" in "rightMask = O;"
  114.  
  115. p. 671, line 35:  missing "{" at end of "while (y<ly && y<ry)"
  116. p. 676, end:  change "void pixelproc();" to "static void pixelproc();"
  117.  
  118. p. 696, line 8:  add '#include "GraphicsGems.h"'
  119. p. 700, line 5:  add '#include "GraphicsGems.h"'
  120. p. 706, end:  add '#include "GraphicsGems.h"'
  121. p. 709, line 23:  missing semicolon at end of "up = (up) ? FALSE : TRUE"
  122.  
  123. p. 713, line 26:  change ":" to ";" in "char **argv:"
  124.  
  125. p. 715, top:  pseudo-code at head of file (to advance from one element to the
  126.               next) should be commented out.
  127.         line 14:  missing declaration "int randmasks[32];"
  128.  
  129. p. 727, line 11:  remove ")" in "static double bigC,..." line
  130. p. 728, line 26:  change "con_const" to "cone_const" in
  131.               "bigC = m1sq + con_const * q1"
  132.     last line:  add a "}" to end albers_project procedure
  133.  
  134. p. 730:  missing inclusion of GraphicsGems.h.
  135.  
  136. p. 736, line 20:  change "O" to "0" in "for (i=O;"
  137.  
  138. p. 757, line 4:  change "{" to "[" in "sqrttab{"
  139.  
  140. p. 774, line 2:  missing ";" at end of "long *argx, *argy"
  141.  
  142. p. 775:  P, Q, and M need to be declared as externs
  143.  
  144. p. 780, line 18:  bad start of comment for "/ re-parameterization"
  145.  
  146. p. 785, line 1:  bad start-of-comment
  147.  
  148. -----
  149.  
  150. The following are typographical errors in the comments:
  151.  
  152. p. 687, line 3:  "plottted" to "plotted"
  153.  
  154. p. 701, line 26:  change "interscetion" to "intersection"
  155.  
  156. p. 728, line 10:  "latitute" to "latitude"
  157. p. 729, line 8:  "degress" to "degrees"
  158.  
  159. p. 724, line 38:  "seperated" to "separated"
  160. p. 725, lines 7-9:  "componant" to "component"
  161.  
  162. p. 730, line 17:  "minium" to "minimum"
  163.  
  164. p. 752, line 2:  "positve" to "positive"
  165.  
  166. p. 760, line 5:  "interger" to "integer"
  167. p. 761, line 4:  "preceed" to "precede"
  168.  
  169. p. 766, throughout (5 times):  "determinent" to "determinant"
  170.  
  171. p. 781, lines 7,23:  "demoninator" to "denominator"
  172.  
  173. -----
  174.  
  175. Addenda:  There is additional code for Kelvin Thompson's "Rendering
  176. Anti-Aliased Lines" gem in the online distribution of the code.
  177.