--- books/www/corrections.html	2002/12/10 16:22:55	1.39
+++ books/www/corrections.html	2002/12/11 16:40:13	1.54
@@ -1,3 +1,8 @@
+<style>
+  .broken { color: red; }
+  .fixed { color: green; }
+</style>
+
 <p><b>Corrections</b>
 
 <p>This page contains a list of corrections to be made in the next printing of
@@ -12,8 +17,8 @@ href="mailto:reviewers@mozdev.org">revie
 
 <ul>
 
-<li>On page xiv: 'http://www.oreilly.com/mozilla/' has been changed
-to 'http://www.oreilly.com/catalog/mozilla/'<br><br>
+<li>On page xiv: '<b class="broken">http://www.oreilly.com/mozilla/</b>' has been changed
+to '<b class="fixed">http://www.oreilly.com/catalog/mozilla/</b>'<br><br>
 
 </ul>
 
@@ -23,8 +28,8 @@ to 'http://www.oreilly.com/catalog/mozil
 
 <ul>
 
-<li>On page 7: 'applications web built using XPFE' has been
-changed to 'applications built using XPFE'<br><br>
+<li>On page 7: '<b class="broken">applications web built using XPFE</b>' has been
+changed to '<b class="fixed">applications built using XPFE</b>'<br><br>
 
 </ul>
 
@@ -34,18 +39,18 @@ changed to 'applications built using XPF
 
 <ul>
 
-<li>On page 12: '&lt;window
+<li>On page 12: '<b class="broken">&lt;window
 xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"  
-/&gt;' has been changed to '&lt;window
-xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"&gt;'<br><br>
+/&gt;</b>' has been changed to '<b class="fixed">&lt;window
+xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"&gt;</b>'<br><br>
 
-<li>On page 14: Removed an extra '>' from example code<br><br>
+<li>On page 14: Removed an extra '<b class="broken">></b>' from example code<br><br>
 
-<li>On page 17: 'selecting File > Open File > from' has been
-changed to 'selecting File > Open File from'<br><br>
+<li>On page 17: '<b class="broken">selecting File > Open File > from</b>' has been
+changed to '<b class="fixed">selecting File > Open File from</b>'<br><br>
 
-<li>On page 20: 'CenterWindoOnScreen( ) function' has been changed
-to 'CenterWindowOnScreen( ) function'<br><br>
+<li>On page 20: '<b class="broken">CenterWindoOnScreen( ) function</b>' has been changed
+to '<b class="fixed">CenterWindowOnScreen( ) function</b>'<br><br>
 
 </ul>
 
@@ -55,9 +60,9 @@ to 'CenterWindowOnScreen( ) function'<br
 
 <ul>
 
-<li>On page 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>
+<li>On page 40: '<b class="broken">&lt;toolbarbutton id="printBtn" label="Open"
+oncommand="doPrint( );" /&gt</b>' has been changed to '<b class="fixed">&lt;toolbarbutton
+id="printBtn" label="Print" oncommand="doPrint( );" &gt</b>'<br><br>
 
 </ul>
 
@@ -65,54 +70,47 @@ id="printBtn" label="Print" oncommand="d
 <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>
+
+<li>On page 113: Javascript 1.5 reference link in footnote has been changed from 
+'<b class="broken">http://developer.netscape.com/docs/manuals/js/core/jsref15/contents.html</b>' to
+'<b class="fixed">http://developer.netscape.com/docs/manuals/index.html?content=javascript.html</b>'<br><br>
+
+<li>On page 120: '<b class="broken">&lt;/script.</b>' has been changed to '<b class="fixed">&lt;/script&gt;</b>'<br><br>
+
+<li>On page 120: '<b class="broken">dump(l+'\n');</b>' has been changed to '<b class="fixed">dump(len+'\n');</b>'<br><br>
+
+<li>On page 137: '<b class="broken">var connection = Components.classes ["@mozilla.org/network/ldap-connection;1";</b>' has been changed to 
+'<b class="fixed">var connection = Components.classes ["@mozilla.org/network/ldap-connection;1"];</b>'<br><br>
+
+<li>On page 140: '<b class="broken">js> load(`chrome/jslib/jslib.js');</b>' has been changed to '<b class="fixed">js>
+load('chrome/jslib/jslib.js');</b>'<br><br>
+
 </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>            
+
+<li>On page 211: In 2nd sentence of paragraph '<b class="broken">QueryInterace</b>' has been changed to '<b 
+class="fixed">QueryInterface</b>'<br><br>
+
+<li>On page 222: In Example 8-4 '<b class="broken">mName: "a default value"</b>' has been changed to '<b class="fixed">mName: "a default 
+value",</b>'<br><br>
+
+<li>On page 222: In Example 8-4
+
+            <xmp class="broken">
               this.myCID,
               "My JS Component",
               this.myProgID, fileSpec,
               location, true, true,
               type);
             </xmp>            
-            Should be
-            <xmp>            
+
+has been changed to
+
+            <xmp class="fixed">            
               compMgr.registerComponentWithType(
               this.myCID,
               "My JS Component",
@@ -120,18 +118,15 @@ id="printBtn" label="Print" oncommand="d
               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>
+<li>On page 222: In Example 8-4 one of the duplicate '<b class="broken">dump(" ***** Registering: Simple JS component! ****\n");</b>' lines
+has been removed<br><br>
+
+<li>On page 223: In Example 8-4 one of the duplicate '<b class="broken">if (outer != null)</b>' lines has been removed<br><br>
+
+<li>On page 239: Example 8-15 has been changed to <br/>
+
+<xmp class="fixed">
 DEPTH         = ../../..
 topsrcdir     = ../../..
 srcdir        = .
@@ -169,7 +164,6 @@ EXTRA_DSO_LDOPTS += $(MOZ_COMPONENT_LIBS
 
 install:: $(TARGETS)
 </xmp>
-</li>
 
 </ul>
 
@@ -178,8 +172,8 @@ install:: $(TARGETS)
 <p><b>Chapter 10</b>
 
 <ul>
-<li>On page 297: 'http://www.jslib.mozdev.org' has been changed
-to 'http://jslib.mozdev.org'<br><br></li>
+<li>On page 297: '<b class="broken">http://www.jslib.mozdev.org</b>' has been changed
+to '<b class="fixed">http://jslib.mozdev.org</b>'<br><br></li>
 </ul>
 
 
@@ -188,22 +182,27 @@ to 'http://jslib.mozdev.org'<br><br></li
 
 <ul>
 
-<li>On page 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>On page 336: In Example 12-6
 
-<li>(p 336) Use of backtics instead of single quote.
-<xmp>
+<xmp class="broken">
  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
+</xmp>
+
+has been changed to (note the difference in indentation and in using a single quote versus a backtick)
+
+<xmp class="fixed>
+   print '  <checkbox label="%s" />' % form['op1'].value
+   print '  <checkbox label="%s" />' % form['op2'].value
    print '  <checkbox label="%s" />' % form['op3'].value
 </xmp>
+
+<li>On page 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>
+
 </ul>
 
 
@@ -212,8 +211,8 @@ Should be:
 
 <ul>
 
-<li>On page 365: 'http://www.mozilla.org/source.tml' has been
-changed to 'http://www.mozilla.org/source.html'<br><br>
+<li>On page 365: '<b class="broken">http://www.mozilla.org/source.tml</b>' has been changed to '<b
+class="fixed">http://www.mozilla.org/source.html</b>'<br><br>
 
 </ul>