--- books/www/corrections.html	2002/12/04 13:23:05	1.26
+++ books/www/corrections.html	2002/12/17 13:13:58	1.63
@@ -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
@@ -6,14 +11,16 @@ feel free open a <a href="bugs.html">bug
 section on the chapter pages, or send a message to the <a
 href="mailto:reviewers@mozdev.org">reviewers@mozdev.org</a> mailing list.
 
+<p><b>Note:</b> All errors in the text and examples are shown in <b class="broken">red</b> and all corrections are shown in <b
+class="fixed">green</b>.
 
 <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>
+<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 +30,8 @@ to 'http://www.oreilly.com/catalog/mozil
 
 <ul>
 
-<li>In Chapter 1 (p 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 +41,21 @@ changed to 'applications built using XPF
 
 <ul>
 
-<li>In Chapter 2 (p 12) '&lt;window
+<li>On page 10: Added a new sentence to the end of the 2nd paragraph of the xFly examples sidebar: '<b class="fixed">An installable version
+of the complete xFly application can be found at http://xfly.mozdev.org.</b>'<br><br>
+
+<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>In Chapter 2 (p 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>In Chapter 2 (p 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>In Chapter 2 (p 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 +65,9 @@ to 'CenterWindowOnScreen( ) function'<br
 
 <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>
+<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,24 +75,96 @@ id="printBtn" label="Print" oncommand="d
 <p><b>Chapter 5</b>
 
 <ul>
-  <li>In Chapter 5(p 120) 
-    <xmp>
-      dump(l+'\n');
-    </xmp>
-    has been changed to
-    <xmp>
-      dump(len+'\n');
-    </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>
+
+<li>On page 120: '<b class="broken">&lt;/script.</b>' has been changed to '<b class="fixed">&lt;/script&gt;</b>'<br><br>
+</li>
+
+<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>
+
+<li>On page 135: Example 5-11 has been changed to
+<xmp class="fixed">
+<html>
+  <head>
+    <title>Sound Service Play Example</title>
+    <script>  
+      function play() { 
+        netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect"); 
+        var sample = Components.classes["@mozilla.org/sound;1"].createInstance(); 
+        sample = sample.QueryInterface(Components.interfaces.nsISound); 
+        const SND_NETWORK_STD_CID = "@mozilla.org/network/standard-url;1";
+        const SND_I_URL           = "nsIURL";
+        const SND_URL             = new  C.Constructor(SND_NETWORK_STD_CID, SND_I_URL);
+        var url                   = new SND_URL();
+        url.spec                  = 'http://jslib.mozdev.org/test.wav';
+        sample.play(url);
+      } 
+    </script>  
+  </head>  
+  <body>  
+    <form name="form">  
+      <input type="button" value="Play Sound" onclick="play();">  
+    </form>  
+  </body>  
+</html>
+</xmp>
+</li>
+
+<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>
+
+<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>
+</li>
+
 </ul>
 
 <a name="ch08">
 <p><b>Chapter 8</b>
 
 <ul>
-<li>In Chapter 8 (p 239) Example 8-15 has been changed to <br/>
 
-<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>            
+
+has been changed to
+
+            <xmp class="fixed">            
+              compMgr.registerComponentWithType(
+              this.myCID,
+              "My JS Component",
+              this.myProgID, fileSpec,
+              location, true, true,
+              type);
+            </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        = .
@@ -120,7 +202,6 @@ EXTRA_DSO_LDOPTS += $(MOZ_COMPONENT_LIBS
 
 install:: $(TARGETS)
 </xmp>
-</li>
 
 </ul>
 
@@ -129,8 +210,8 @@ install:: $(TARGETS)
 <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></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>
 
 
@@ -139,9 +220,26 @@ to 'http://jslib.mozdev.org'<br><br></li
 
 <ul>
 
-<li>In Chapter 12 (p 339) A new 'Certificate Authorities and Digital
+<li>On page 336: In Example 12-6
+
+<xmp class="broken">
+ print `  <checkbox label="%s" />' % form['op1'].value
+ print `  <checkbox label="%s" />' % form['op2'].value
+   print `  <checkbox label="%s" />' % form['op3'].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>
 
@@ -151,8 +249,8 @@ href="http://certs.mozdev.org">http://ce
 
 <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>
+<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>