--- books/www/corrections.html	2002/11/01 19:38:59	1.19
+++ books/www/corrections.html	2002/12/05 13:59:34	1.38
@@ -1,18 +1,39 @@
 <p><b>Corrections</b>
 
-<p>This page contains a list of corrections made to the original text of
+<p>This page contains a list of corrections to be made in the next printing of
 the book.  If you have suggestions for changes or find errors in the text,
-feel free to post a note below or send a message to the <a
+feel free open a <a href="bugs.html">bug</a> with a report of the problem, post a message in the User Notes 
+section on the chapter pages, or send a message to the <a
 href="mailto:reviewers@mozdev.org">reviewers@mozdev.org</a> mailing list.
 
+
+<a name="preface">
+<p><b>Preface</b>
+
 <ul>
 
 <li>In Preface (p xiv) 'http://www.oreilly.com/mozilla/' has been changed
 to 'http://www.oreilly.com/catalog/mozilla/'<br><br>
 
+</ul>
+
+
+<a name="ch1">
+<p><b>Chapter 1</b>
+
+<ul>
+
 <li>In Chapter 1 (p 7) 'applications web built using XPFE' has been
 changed to 'applications built using XPFE'<br><br>
 
+</ul>
+
+
+<a name="ch2">
+<p><b>Chapter 2</b>
+
+<ul>
+
 <li>In Chapter 2 (p 12) '&lt;window
 xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"  
 /&gt;' has been changed to '&lt;window
@@ -26,16 +47,170 @@ changed to 'selecting File > Open File f
 <li>In Chapter 2 (p 20) 'CenterWindoOnScreen( ) function' has been changed
 to 'CenterWindowOnScreen( ) function'<br><br>
 
+</ul>
+
+
+<a name="ch3">
+<p><b>Chapter 3</b>
+
+<ul>
+
 <li>In Chapter 3 (p 40) '&lt;toolbarbutton id="printBtn" label="Open"
 oncommand="doPrint( );" /&gt' has been changed to '&lt;toolbarbutton
 id="printBtn" label="Print" oncommand="doPrint( );" &gt'<br><br>
 
+</ul>
+
+<a name="ch5">
+<p><b>Chapter 5</b>
+
+<ul>
+  <li>(p 113) 
+    <xmp>
+      Changed footnote link to
+        http://developer.netscape.com/docs/manuals/index.html?content=javascript.html
+    </xmp>
+  </li>
+  <li>(p 120) 
+    <xmp>
+      dump(l+'\n');
+    has been changed to
+      dump(len+'\n');
+    </xmp>
+  </li>
+  <li>(p 137) 
+    <xmp>
+      var connection = Components.classes ["@mozilla.org/network/ldap-connection;1"; // create an object from the nsILDAPConnection interface;
+    has been changed to
+      var connection = Components.classes ["@mozilla.org/network/ldap-connection;1"]; // create an object from the nsILDAPConnection interface;
+    </xmp>
+  </li>
+  <li>(p 140) 
+    <xmp>
+       Changed load(`chrome/jslib/jslib.js');
+       To load('chrome/jslib/jslib.js');
+    </xmp>
+  </li>
+</ul>
+
+<a name="ch08">
+<p><b>Chapter 8</b>
+
+<ul>
+<li>(p 211) Root Interfaces<br/>
+  Changed 2nd sentence QueryInterace to QueryInterface.
+</li>
+<li>(p 222) Example 8-4
+  <code>mName: "a default value"</code> should be <code>mName: "a default value",</code>
+</li>
+<li>(p 222) Example 8-4
+            <xmp>            
+              this.myCID,
+              "My JS Component",
+              this.myProgID, fileSpec,
+              location, true, true,
+              type);
+            </xmp>            
+            Should be
+            <xmp>            
+              compMgr.registerComponentWithType(
+              this.myCID,
+              "My JS Component",
+              this.myProgID, fileSpec,
+              location, true, true,
+              type);
+            </xmp>            
+</li>
+<li>(p 222, p 223) Example 8-4
+            <xmp>            
+              Removed double lines
+              dump(" ***** Registering: Simple JS component! ****\n");
+              and
+              if (outer != null)
+            </xmp>            
+</li>
+<li>(p 239) Example 8-15 has been changed to <br/>
+
+<xmp>
+DEPTH         = ../../..
+topsrcdir     = ../../..
+srcdir        = .
+VPATH         = .
+
+include $(DEPTH)/config/autoconf.mk
+
+MODULE        = xpcom
+XPIDL_MODULE  = simple
+LIBRARY_NAME  = simple
+IS_COMPONENT  = 1
+MODULE_NAME   = nsSimpleModule
+REQUIRES      = string \
+                xpcom \
+                $(NULL)
+
+
+CPPSRCS   =              \
+    nsSimple.cpp         \
+    nsSimpleModule.cpp   \
+    $(NULL)
+
+XPIDLSRCS = nsISimple.idl
+
+include $(topsrcdir)/config/config.mk
+
+LIBS    +=        \
+    $(XPCOM_LIBS) \
+    $(NSPR_LIBS)  \
+    $(NULL)
+
+include $(topsrcdir)/config/rules.mk
+
+EXTRA_DSO_LDOPTS += $(MOZ_COMPONENT_LIBS)
+
+install:: $(TARGETS)
+</xmp>
+</li>
+
+</ul>
+
+
+<a name="ch10">
+<p><b>Chapter 10</b>
+
+<ul>
 <li>In Chapter 10 (p 297) 'http://www.jslib.mozdev.org' has been changed
-to 'http://jslib.mozdev.org'<br><br>
+to 'http://jslib.mozdev.org'<br><br></li>
+</ul>
+
+
+<a name="ch12">
+<p><b>Chapter 12</b>
+
+<ul>
 
 <li>In Chapter 12 (p 339) A new 'Certificate Authorities and Digital
 Signatures' section is being written with drafts available on <a
 href="http://certs.mozdev.org">http://certs.mozdev.org</a><br><br>
+</li>
+
+<li>(p 336) Use of backtics instead of single quote.
+<xmp>
+ print `  <checkbox label="%s" />' % form['op1'].value
+ print `  <checkbox label="%s" />' % form['op2'].value
+   print `  <checkbox label="%s" />' % form['op3'].value
+Should be:
+ print '  <checkbox label="%s" />' % form['op1'].value
+ print '  <checkbox label="%s" />' % form['op2'].value
+   print '  <checkbox label="%s" />' % form['op3'].value
+</xmp>
+</li>
+</ul>
+
+
+<a name="appa">
+<p><b>Appendix A</b>
+
+<ul>
 
 <li>In Appendix A (p 365) 'http://www.mozilla.org/source.tml' has been
 changed to 'http://www.mozilla.org/source.html'<br><br>
@@ -44,6 +219,5 @@ changed to 'http://www.mozilla.org/sourc
 
 <br><br>
 
-<!-- ?php require(ANNOTATE); ? -->
-
-<?php $post_to_list=NO; require(ANNOTATE); ?>
+<!-- ?php require(NOTES); ? -->
+<?php $post_to_list=NO; $author='reviewers@mozdev.org'; $target_page='corrections'; require(NOTES); ?>