--- books/www/corrections.html	2002/12/10 16:22:55	1.39
+++ books/www/corrections.html	2003/01/03 21:03:22	1.67
@@ -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>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 +30,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 +41,23 @@ changed to 'applications built using XPF
 
 <ul>
 
-<li>On page 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>On page 14: Removed an extra '>' from example code<br><br>
+<li>On page 14: In first code sample from Windows features section, '<b
+class="broken">xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"&gt;</b>' has been changed to '<b
+class="fixed">xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"</b>'<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 +67,15 @@ 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 34: '<b class="broken">We've packed a lot information in this chapter and it be a useful reference</b>' has been changed
+to '<b class="fixed">We've packed a lot of information in this chapter and it will be a useful reference</b>'<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>
+
+<li>On page 51: '<b class="broken">&lt;decription&gt;The mozdev.org site [...] your own development project&lt;/decription&gt;</b>' has been 
+changed to '<b class="fixed">&lt;description&gt;The mozdev.org site [...] your own development project&lt;/description&gt;</b>'<br><br>
 
 </ul>
 
@@ -65,54 +83,80 @@ 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>
+
+<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>(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 +164,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 +210,6 @@ EXTRA_DSO_LDOPTS += $(MOZ_COMPONENT_LIBS
 
 install:: $(TARGETS)
 </xmp>
-</li>
 
 </ul>
 
@@ -178,8 +218,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 +228,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 +257,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>