bug 108:   not legal in XML

Author: [email protected]

CHANGES.html

@@ -120,6 +120,11 @@ <h2>19 July 2010</h2>
          >102</a> courtesy jacobly.</li>
       <li>A variety of markup formatting fixes courtesy smain and thezbyg.</li>
       <li>Fixed copy and paste in IE[678].
+      <li>Changed output to use <code>&amp;#160;</code> instead of
+        <code>&amp;nbsp;</code> so that the output works when embedded in XML.
+        Bug
+        <a href="http://code.google.com/p/google-code-prettify/issues/detail?id=108"
+         >108</a>.</li>
     </ul>
   </body>
 </html>

src/prettify.js

@@ -1096,10 +1096,11 @@ window['_pr_isIE6'] = function () {
            // on IE.
            // Doing this on other browsers breaks lots of stuff since \r\n is
            // treated as two newlines on Firefox.
-           ? (isIE678 === 6 ? ' \r\n' : ' \r')
+           ? (isIE678 === 6 ? ' \r\n' : ' \r')
            // IE collapses multiple adjacent <br>s into 1 line break.
-           // Prefix every newline with '&nbsp;' to prevent such behavior.
-           : '&nbsp;<br />')
+           // Prefix every newline with '&#160;' to prevent such behavior.
+           // &nbsp; is the same as &#160; but works in XML as well as HTML.
+           : '&#160;<br />')
         : '<br />');
 
     // Look for a class like linenums or linenums:<n> where <n> is the 1-indexed
@@ -1152,7 +1153,7 @@ window['_pr_isIE6'] = function () {
             tabExpander(sourceText.substring(outputIdx, sourceIdx)))
             .replace(lastWasSpace
                      ? startOrSpaceRe
-                     : adjacentSpaceRe, '$1&nbsp;');
+                     : adjacentSpaceRe, '$1&#160;');
         // Keep track of whether we need to escape space at the beginning of the
         // next chunk.
         lastWasSpace = trailingSpaceRe.test(htmlChunk);