]> git.lyx.org Git - lyx.git/blobdiff - src/output_xhtml.cpp
The logic of the endParagraph() routine is wrong. We should first
[lyx.git] / src / output_xhtml.cpp
index 4cc928b4b8618bc7d9ad83f0335567f677713b51..11d2de2734c90c309ebd4e368919999dbe3e070a 100644 (file)
@@ -145,7 +145,8 @@ docstring cleanAttr(docstring const & str)
 
 bool isFontTag(string const & s)
 {
-       return s == "em" || s == "strong"; // others?
+       // others?
+       return s == "em" || s == "strong" || s == "i" || s == "b";
 }
 
 
@@ -197,17 +198,50 @@ XHTMLStream::XHTMLStream(odocstream & os)
 {}
 
 
-void XHTMLStream::writeError(std::string const & s)
+#if 0
+void XHTMLStream::dumpTagStack(string const & msg) const
+{
+       writeError(msg + ": Tag Stack");
+       TagStack::const_reverse_iterator it = tag_stack_.rbegin();
+       TagStack::const_reverse_iterator en = tag_stack_.rend();
+       for (; it != en; ++it) {
+               writeError(it->tag_);
+       }
+       writeError("Pending Tags");
+       it = pending_tags_.rbegin();
+       en = pending_tags_.rend();
+       for (; it != en; ++it) {
+               writeError(it->tag_);
+       }
+       writeError("End Tag Stack");
+}
+#endif
+
+
+void XHTMLStream::writeError(std::string const & s) const
 {
        LYXERR0(s);
-       os_ << from_utf8("<!-- Output Error: " + s + " -->");
+       os_ << from_utf8("<!-- Output Error: " + s + " -->\n");
+}
+
+
+namespace {
+       // an illegal tag for internal use
+       static string const parsep_tag = "&LyX_parsep_tag&";
 }
 
 
 bool XHTMLStream::closeFontTags()
 {
+       if (isTagPending(parsep_tag))
+               // we haven't had any content
+               return true;
+
+       // this may be a useless check, since we ought at least to have
+       // the parsep_tag. but it can't hurt too much to be careful.
        if (tag_stack_.empty())
                return true;
+
        // first, we close any open font tags we can close
        html::StartTag curtag = tag_stack_.back();
        while (html::isFontTag(curtag.tag_)) {
@@ -220,19 +254,67 @@ bool XHTMLStream::closeFontTags()
                        return true;
                curtag = tag_stack_.back();
        }
-       // so we've hit a non-font tag. let's see if any of the
-       // remaining tags are font tags.
-       TagStack::const_iterator it = tag_stack_.begin();
-       TagStack::const_iterator en = tag_stack_.end();
-       bool noFontTags = true;
+       
+       if (curtag.tag_ == parsep_tag)
+               return true;
+
+       // so we've hit a non-font tag.
+       writeError("Tags still open in closeFontTags(). Probably not a problem,\n"
+                  "but you might want to check these tags:");
+       TagStack::const_reverse_iterator it = tag_stack_.rbegin();
+       TagStack::const_reverse_iterator const en = tag_stack_.rend();
        for (; it != en; ++it) {
-               if (html::isFontTag(it->tag_)) {
-                       writeError("Font tag `" + it->tag_ + "' still open in closeFontTags().\n"
-                               "This is likely not a problem, but you might want to check.");
-                       noFontTags = false;
+               string const tagname = it->tag_;
+               if (tagname == parsep_tag)
+                       break;
+               writeError(it->tag_);
+       }
+       return false;
+}
+
+
+void XHTMLStream::startParagraph(bool keep_empty)
+{
+       pending_tags_.push_back(html::StartTag(parsep_tag));
+       if (keep_empty)
+               clearTagDeque();
+}
+
+
+void XHTMLStream::endParagraph()
+{
+       if (isTagPending(parsep_tag)) {
+               // this case is normal. it just means we didn't have content,
+               // so the parsep_tag never got moved onto the tag stack.
+               while (!pending_tags_.empty()) {
+                       // clear all pending tags up to and including the parsep tag.
+                       // note that we work from the back, because we want to get rid
+                       // of everything that hasn't been used.
+                       html::StartTag const cur_tag = pending_tags_.back();
+                       string const & tag = cur_tag.tag_;
+                       pending_tags_.pop_back();
+                       if (tag == parsep_tag)
+                               break;
                }
+               return;
+       }
+
+       if (!isTagOpen(parsep_tag)) {
+               writeError("No paragraph separation tag found in endParagraph().");
+               return;
+       }
+
+       // this case is also normal, if the parsep tag is the last one 
+       // on the stack. otherwise, it's an error.
+       while (!tag_stack_.empty()) {
+               html::StartTag const cur_tag = tag_stack_.back();
+               string const & tag = cur_tag.tag_;
+               tag_stack_.pop_back();
+               if (tag == parsep_tag)
+                       break;
+               writeError("Tag `" + tag + "' still open at end of paragraph. Closing.");
+               os_ << cur_tag.asEndTag();
        }
-       return noFontTags;
 }
 
 
@@ -240,8 +322,9 @@ void XHTMLStream::clearTagDeque()
 {
        while (!pending_tags_.empty()) {
                html::StartTag const & tag = pending_tags_.front();
-               // tabs?
-               os_ << tag.asTag();
+               if (tag.tag_ != parsep_tag)
+                       // tabs?
+                       os_ << tag.asTag();
                tag_stack_.push_back(tag);
                pending_tags_.pop_front();
        }
@@ -332,7 +415,7 @@ XHTMLStream & XHTMLStream::operator<<(html::CR const &)
 }
 
 
-bool XHTMLStream::isTagOpen(string const & stag)
+bool XHTMLStream::isTagOpen(string const & stag) const
 {
        TagStack::const_iterator sit = tag_stack_.begin();
        TagStack::const_iterator const sen = tag_stack_.end();
@@ -343,6 +426,17 @@ bool XHTMLStream::isTagOpen(string const & stag)
 }
 
 
+bool XHTMLStream::isTagPending(string const & stag) const
+{
+       TagStack::const_iterator sit = pending_tags_.begin();
+       TagStack::const_iterator const sen = pending_tags_.end();
+       for (; sit != sen; ++sit)
+               if (sit->tag_ == stag)
+                       return true;
+       return false;
+}
+
+
 // this is complicated, because we want to make sure that
 // everything is properly nested. the code ought to make 
 // sure of that, but we won't assert (yet) if we run into
@@ -475,7 +569,8 @@ XHTMLStream & XHTMLStream::operator<<(html::EndTag const & etag)
        html::StartTag curtag = tag_stack_.back();
        while (curtag.tag_ != etag.tag_) {
                writeError(curtag.tag_);
-               os_ << curtag.asEndTag();
+               if (curtag.tag_ != parsep_tag)
+                       os_ << curtag.asEndTag();
                tag_stack_.pop_back();
                curtag = tag_stack_.back();
        }
@@ -658,11 +753,11 @@ ParagraphList::const_iterator makeBibliography(Buffer const & buf,
        // FIXME XHTML
        // Use TextClass::htmlTOCLayout() to figure out how we should look.
        xs << html::StartTag("h2", "class='bibliography'")
-          << pbegin->layout().labelstring(false);
-          << html::EndTag("h2");
-       xs << html::CR();
-       xs << html::StartTag("div", "class='bibliography'");
-       xs << html::CR();
+          << pbegin->layout().labelstring(false)
+          << html::EndTag("h2")
+          << html::CR()
+          << html::StartTag("div", "class='bibliography'")
+          << html::CR();
        makeParagraphs(buf, xs, runparams, text, pbegin, pend);
        xs << html::EndTag("div");
        return pend;
@@ -857,11 +952,20 @@ void xhtmlParagraphs(Text const & text,
                       OutputParams const & runparams)
 {
        ParagraphList const & paragraphs = text.paragraphs();
-       ParagraphList::const_iterator par = paragraphs.begin();
-       ParagraphList::const_iterator pend = paragraphs.end();
+       if (runparams.par_begin == runparams.par_end) {
+               runparams.par_begin = 0;
+               runparams.par_end = paragraphs.size();
+       }
+       pit_type bpit = runparams.par_begin;
+       pit_type const epit = runparams.par_end;
+       LASSERT(bpit < epit, /* */);
 
        OutputParams ourparams = runparams;
-       while (par != pend) {
+       ParagraphList::const_iterator const pend =
+               (epit == (int) paragraphs.size()) ?
+                       paragraphs.end() : paragraphs.constIterator(epit);
+       while (bpit < epit) {
+               ParagraphList::const_iterator par = paragraphs.constIterator(bpit);
                if (par->params().startOfAppendix()) {
                        // FIXME: only the counter corresponding to toplevel
                        // sectioning should be reset
@@ -870,7 +974,7 @@ void xhtmlParagraphs(Text const & text,
                        cnts.appendix(true);
                }
                Layout const & style = par->layout();
-               ParagraphList::const_iterator lastpar = par;
+               ParagraphList::const_iterator const lastpar = par;
                ParagraphList::const_iterator send;
 
                switch (style.latextype) {
@@ -907,10 +1011,7 @@ void xhtmlParagraphs(Text const & text,
                        par = makeParagraphs(buf, xs, ourparams, text, par, send);
                        break;
                }
-               // FIXME??
-               // makeEnvironment may process more than one paragraphs and bypass pend
-               if (distance(lastpar, par) >= distance(lastpar, pend))
-                       break;
+               bpit += distance(lastpar, par);
        }
 }