Attempt to pass through input that we can't make sense of

Fixes #301
beautifier · Oct 2, 2014 · c13efe3 · c13efe3
1 parent 52a591f
commit c13efe3
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/js/lib/beautify-html.js b/js/lib/beautify-html.js
@@ -811,6 +811,13 @@
                     }
                     multi_parser.current_mode = 'TAG';
                     break;
+                default:
+                    // We should not be getting here but we don't want to drop input on the floor
+                    // Just output the text and move on
+                    if (multi_parser.token_text !== '') {
+                        multi_parser.print_token(multi_parser.token_text);
+                    }
+                    break;
             }
             multi_parser.last_token = multi_parser.token_type;
             multi_parser.last_text = multi_parser.token_text;

diff --git a/js/test/beautify-tests.js b/js/test/beautify-tests.js
@@ -1839,7 +1839,12 @@ function run_beautifier_tests(test_obj, Urlencoded, js_beautify, html_beautify,
 
 
         opts.end_with_newline = false;
-        test_fragment('<head>\n' +
+        // error cases need love too
+        bth('<img title="Bad food!" src="foo.jpg" alt="Evil" ">');
+        bth("<!-- don't blow up if a comment is not complete");
+
+        test_fragment(
+            '<head>\n' +
             '    <script>\n' +
             '        mocha.setup("bdd");\n' +
             '    </script>\n' +