From 0333d83d2e9a540455c59e004886ea5ffe6c846b Mon Sep 17 00:00:00 2001 From: Jean-Francois Dockes Date: Wed, 16 May 2012 10:13:53 +0200 Subject: [PATCH] html: small additional cleanup after previous processing modification --- src/internfile/myhtmlparse.cpp | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/src/internfile/myhtmlparse.cpp b/src/internfile/myhtmlparse.cpp index 8c963029..d34d9687 100644 --- a/src/internfile/myhtmlparse.cpp +++ b/src/internfile/myhtmlparse.cpp @@ -311,6 +311,7 @@ MyHtmlParser::opening_tag(const string &tag) // even text before the body is displayed by Opera and // Firefox. We used to reset the dump each time we saw a // body tag, but I can't see any reason to do so. + if (tag == "blockquote" || tag == "br") { dump += '\n'; pending_space = true; @@ -471,14 +472,11 @@ MyHtmlParser::closing_tag(const string &tag) if (tag == "address") pending_space = true; break; case 'b': - if (tag == "body") { - LOGDEB1(("Myhtmlparse: body close tag found\n")); - // We used to signal and end of doc here by returning - // false but the browsers just ignore body and html - // closing tags if there is further text, so it seems right - // to do the same - break; - } + // body: We used to signal and end of doc here by returning + // false but the browsers just ignore body and html + // closing tags if there is further text, so it seems right + // to do the same + if (tag == "blockquote" || tag == "br") pending_space = true; break; case 'c':