]> git.lyx.org Git - lyx.git/blobdiff - src/paragraph_funcs.C
cosmetic fix
[lyx.git] / src / paragraph_funcs.C
index c6b92bb839600d259497fcb09b4128220384366c..531afd3f27d07c9154a57bdce0821a6480c7cca7 100644 (file)
 #include "insets/insetlatexaccent.h"
 #include "insets/insettabular.h"
 #include "insets/insethfill.h"
-#include "insets/inseterror.h"
 #include "insets/insetnewline.h"
 
 extern string bibitemWidest(Buffer const *);
 
+using namespace lyx::support;
+
 using lyx::pos_type;
 //using lyx::layout_type;
 using std::endl;
@@ -51,7 +52,7 @@ void breakParagraph(BufferParams const & bparams,
 {
        // create a new paragraph, and insert into the list
        ParagraphList::iterator tmp = paragraphs.insert(boost::next(par),
-                                                       new Paragraph);
+                                                       Paragraph());
 
        // without doing that we get a crash when typing <Return> at the
        // end of a paragraph
@@ -71,7 +72,7 @@ void breakParagraph(BufferParams const & bparams,
                tmp->setLabelWidthString(par->params().labelWidthString());
        }
 
-       bool const isempty = (par->layout()->keepempty && par->empty());
+       bool const isempty = (par->allowEmpty() && par->empty());
 
        if (!isempty && (par->size() > pos || par->empty() || flag == 2)) {
                tmp->layout(par->layout());
@@ -150,7 +151,7 @@ void breakParagraphConservative(BufferParams const & bparams,
 {
        // create a new paragraph
        ParagraphList::iterator tmp = paragraphs.insert(boost::next(par),
-                                                       new Paragraph);
+                                                       Paragraph());
        tmp->makeSameLayout(*par);
 
        // When can pos > size()?
@@ -203,7 +204,7 @@ ParagraphList::iterator depthHook(ParagraphList::iterator pit,
                                  Paragraph::depth_type depth)
 {
        ParagraphList::iterator newpit = pit;
-       ParagraphList::iterator beg = plist.begin();
+       ParagraphList::iterator beg = const_cast<ParagraphList&>(plist).begin();
 
        if (newpit != beg)
                --newpit;
@@ -223,7 +224,7 @@ ParagraphList::iterator outerHook(ParagraphList::iterator pit,
                                  ParagraphList const & plist)
 {
        if (!pit->getDepth())
-               return plist.end();
+               return const_cast<ParagraphList&>(plist).end();
        return depthHook(pit, plist,
                         Paragraph::depth_type(pit->getDepth() - 1));
 }
@@ -243,12 +244,12 @@ int getEndLabel(ParagraphList::iterator p, ParagraphList const & plist)
 {
        ParagraphList::iterator pit = p;
        Paragraph::depth_type par_depth = p->getDepth();
-       while (pit != plist.end()) {
+       while (pit != const_cast<ParagraphList&>(plist).end()) {
                LyXLayout_ptr const & layout = pit->layout();
                int const endlabeltype = layout->endlabeltype;
 
                if (endlabeltype != END_LABEL_NO_LABEL) {
-                       if (boost::next(p) == plist.end())
+                       if (boost::next(p) == const_cast<ParagraphList&>(plist).end())
                                return endlabeltype;
 
                        Paragraph::depth_type const next_depth = boost::next(p)->getDepth();
@@ -261,7 +262,7 @@ int getEndLabel(ParagraphList::iterator p, ParagraphList const & plist)
                if (par_depth == 0)
                        break;
                pit = outerHook(pit, plist);
-               if (pit != plist.end())
+               if (pit != const_cast<ParagraphList&>(plist).end())
                        par_depth = pit->getDepth();
        }
        return END_LABEL_NO_LABEL;
@@ -296,7 +297,7 @@ TeXDeeper(Buffer const * buf,
        lyxerr[Debug::LATEX] << "TeXDeeper...     " << &*pit << endl;
        ParagraphList::iterator par = pit;
 
-       while (par != paragraphs.end() &&
+       while (par != const_cast<ParagraphList&>(paragraphs).end() &&
                     par->params().depth() == pit->params().depth()) {
                if (par->layout()->isEnvironment()) {
                        par = TeXEnvironment(buf, paragraphs, par,
@@ -328,7 +329,7 @@ TeXEnvironment(Buffer const * buf,
        Language const * language = pit->getParLanguage(bparams);
        Language const * doc_language = bparams.language;
        Language const * previous_language =
-               (pit != paragraphs.begin())
+               (pit != const_cast<ParagraphList&>(paragraphs).begin())
                ? boost::prior(pit)->getParLanguage(bparams)
                : doc_language;
        if (language->babel() != previous_language->babel()) {
@@ -379,7 +380,7 @@ TeXEnvironment(Buffer const * buf,
        do {
                par = TeXOnePar(buf, paragraphs, par, os, texrow, runparams);
 
-               if (par != paragraphs.end()&& par->params().depth() > pit->params().depth()) {
+               if (par != const_cast<ParagraphList&>(paragraphs).end() && par->params().depth() > pit->params().depth()) {
                            if (par->layout()->isParagraph()) {
 
                            // Thinko!
@@ -402,7 +403,7 @@ TeXEnvironment(Buffer const * buf,
                        par = TeXDeeper(buf, paragraphs, par, os, texrow,
                                        runparams);
                }
-       } while (par != paragraphs.end()
+       } while (par != const_cast<ParagraphList&>(paragraphs).end()
                 && par->layout() == pit->layout()
                 && par->params().depth() == pit->params().depth()
                 && par->params().leftIndent() == pit->params().leftIndent());
@@ -425,11 +426,11 @@ TeXEnvironment(Buffer const * buf,
 InsetOptArg * optArgInset(Paragraph const & par)
 {
        // Find the entry.
-       InsetList::iterator it = par.insetlist.begin();
-       InsetList::iterator end = par.insetlist.end();
+       InsetList::const_iterator it = par.insetlist.begin();
+       InsetList::const_iterator end = par.insetlist.end();
        for (; it != end; ++it) {
-               Inset * ins = it.getInset();
-               if (ins->lyxCode() == Inset::OPTARG_CODE) {
+               InsetOld * ins = it->inset;
+               if (ins->lyxCode() == InsetOld::OPTARG_CODE) {
                        return static_cast<InsetOptArg *>(ins);
                }
        }
@@ -449,7 +450,7 @@ TeXOnePar(Buffer const * buf,
 << "'" << endl;
        BufferParams const & bparams = buf->params;
 
-       Inset const * in = pit->inInset();
+       InsetOld const * in = pit->inInset();
        bool further_blank_line = false;
        LyXLayout_ptr style;
 
@@ -466,7 +467,7 @@ TeXOnePar(Buffer const * buf,
                }
 
                if (!pit->params().spacing().isDefault()
-                       && (pit == paragraphs.begin() || !boost::prior(pit)->hasSameLayout(*pit))) {
+                       && (pit == const_cast<ParagraphList&>(paragraphs).begin() || !boost::prior(pit)->hasSameLayout(*pit))) {
                        os << pit->params().spacing().writeEnvirBegin() << '\n';
                        texrow.newline();
                }
@@ -504,14 +505,14 @@ TeXOnePar(Buffer const * buf,
        Language const * language = pit->getParLanguage(bparams);
        Language const * doc_language = bparams.language;
        Language const * previous_language =
-               (pit != paragraphs.begin())
+               (pit != const_cast<ParagraphList&>(paragraphs).begin())
                ? boost::prior(pit)->getParLanguage(bparams)
                : doc_language;
 
        if (language->babel() != previous_language->babel()
            // check if we already put language command in TeXEnvironment()
            && !(style->isEnvironment()
-                && (pit == paragraphs.begin() ||
+                && (pit == const_cast<ParagraphList&>(paragraphs).begin() ||
                     (boost::prior(pit)->layout() != pit->layout() &&
                      boost::prior(pit)->getDepth() <= pit->getDepth())
                     || boost::prior(pit)->getDepth() < pit->getDepth())))
@@ -591,7 +592,7 @@ TeXOnePar(Buffer const * buf,
        bool is_command = style->isCommand();
 
        if (style->resfont.size() != font.size()
-           && boost::next(pit) != paragraphs.end()
+           && boost::next(pit) != const_cast<ParagraphList&>(paragraphs).end()
            && !is_command) {
                if (!need_par)
                        os << '{';
@@ -604,23 +605,27 @@ TeXOnePar(Buffer const * buf,
        switch (style->latextype) {
        case LATEX_ITEM_ENVIRONMENT:
        case LATEX_LIST_ENVIRONMENT:
-               if (boost::next(pit) != paragraphs.end()
+               if (boost::next(pit) != const_cast<ParagraphList&>(paragraphs).end()
                    && (pit->params().depth() < boost::next(pit)->params().depth())) {
                        os << '\n';
                        texrow.newline();
                }
                break;
-       case LATEX_ENVIRONMENT:
+       case LATEX_ENVIRONMENT: {
                // if its the last paragraph of the current environment
                // skip it otherwise fall through
-               if (boost::next(pit) != paragraphs.end()
-                   && (boost::next(pit)->layout() != pit->layout()
-                       || boost::next(pit)->params().depth() != pit->params().depth()))
+               ParagraphList::iterator next = boost::next(pit);
+
+               if (next != const_cast<ParagraphList&>(paragraphs).end()
+                   && (next->layout() != pit->layout()
+                       || next->params().depth() != pit->params().depth()))
                        break;
+       }
+
                // fall through possible
        default:
                // we don't need it for the last paragraph!!!
-               if (boost::next(pit) != paragraphs.end()) {
+               if (boost::next(pit) != const_cast<ParagraphList&>(paragraphs).end()) {
                        os << '\n';
                        texrow.newline();
                }
@@ -649,14 +654,14 @@ TeXOnePar(Buffer const * buf,
                }
 
                if (!pit->params().spacing().isDefault()
-                       && (boost::next(pit) == paragraphs.end()|| !boost::next(pit)->hasSameLayout(*pit))) {
+                       && (boost::next(pit) == const_cast<ParagraphList&>(paragraphs).end()|| !boost::next(pit)->hasSameLayout(*pit))) {
                        os << pit->params().spacing().writeEnvirEnd() << '\n';
                        texrow.newline();
                }
        }
 
        // we don't need it for the last paragraph!!!
-       if (boost::next(pit) != paragraphs.end()) {
+       if (boost::next(pit) != const_cast<ParagraphList&>(paragraphs).end()) {
                os << '\n';
                texrow.newline();
        } else {
@@ -699,12 +704,12 @@ void latexParagraphs(Buffer const * buf,
        bool was_title = false;
        bool already_title = false;
        LyXTextClass const & tclass = buf->params.getLyXTextClass();
-       ParagraphList::iterator par = paragraphs.begin();
-       ParagraphList::iterator endpar = paragraphs.end();
+       ParagraphList::iterator par = const_cast<ParagraphList&>(paragraphs).begin();
+       ParagraphList::iterator endpar = const_cast<ParagraphList&>(paragraphs).end();
 
        // if only_body
        while (par != endpar) {
-               Inset * in = par->inInset();
+               InsetOld * in = par->inInset();
                // well we have to check if we are in an inset with unlimited
                // length (all in one row) if that is true then we don't allow
                // any special options in the paragraph and also we don't allow
@@ -857,7 +862,7 @@ int readParToken(Buffer & buf, Paragraph & par, LyXLex & lex, string const & tok
                        istringstream is(ss.str());
                        LyXLex tmplex(0, 0);
                        tmplex.setStream(is);
-                       Inset * inset = new InsetCaption;
+                       InsetOld * inset = new InsetCaption;
                        inset->Read(this, tmplex);
                        par.insertInset(pos, inset, font);
                        ++pos;
@@ -883,7 +888,7 @@ int readParToken(Buffer & buf, Paragraph & par, LyXLex & lex, string const & tok
                // But insets should read it, it is a part of
                // the inset isn't it? Lgb.
        } else if (token == "\\begin_inset") {
-               Inset * i = readInset(lex, buf);
+               InsetOld * i = readInset(lex, buf);
                par.insertInset(par.size(), i, font, change);
        } else if (token == "\\family") {
                lex.next();
@@ -933,10 +938,9 @@ int readParToken(Buffer & buf, Paragraph & par, LyXLex & lex, string const & tok
                lex.next();
                font.setLyXColor(lex.getString());
        } else if (token == "\\InsetSpace" || token == "\\SpecialChar") {
-               LyXLayout_ptr const & layout = par.layout();
 
                // Insets don't make sense in a free-spacing context! ---Kayvan
-               if (layout->free_spacing || par.isFreeSpacing()) {
+               if (par.isFreeSpacing()) {
                        if (token == "\\InsetSpace")
                                par.insertChar(par.size(), ' ', font, change);
                        else if (lex.isOK()) {
@@ -951,7 +955,7 @@ int readParToken(Buffer & buf, Paragraph & par, LyXLex & lex, string const & tok
                                }
                        }
                } else {
-                       Inset * inset = 0;
+                       InsetOld * inset = 0;
                        if (token == "\\SpecialChar" )
                                inset = new InsetSpecialChar;
                        else
@@ -960,17 +964,17 @@ int readParToken(Buffer & buf, Paragraph & par, LyXLex & lex, string const & tok
                        par.insertInset(par.size(), inset, font, change);
                }
        } else if (token == "\\i") {
-               Inset * inset = new InsetLatexAccent;
+               InsetOld * inset = new InsetLatexAccent;
                inset->read(&buf, lex);
                par.insertInset(par.size(), inset, font, change);
        } else if (token == "\\backslash") {
                par.insertChar(par.size(), '\\', font, change);
        } else if (token == "\\newline") {
-               Inset * inset = new InsetNewline;
+               InsetOld * inset = new InsetNewline;
                inset->read(&buf, lex);
                par.insertInset(par.size(), inset, font, change);
        } else if (token == "\\LyXTable") {
-               Inset * inset = new InsetTabular(buf);
+               InsetOld * inset = new InsetTabular(buf);
                inset->read(&buf, lex);
                par.insertInset(par.size(), inset, font, change);
        } else if (token == "\\bibitem") {
@@ -1004,9 +1008,9 @@ int readParToken(Buffer & buf, Paragraph & par, LyXLex & lex, string const & tok
                lex.eatLine();
                string const s = bformat(_("Unknown token: %1$s %2$s\n"),
                        token, lex.getString());
-               // we can do this here this way because we're actually reading
-               // the buffer and don't care about LyXText right now.
-               par.insertInset(par.size(), new InsetError(s), font);
+
+               buf.error(ErrorItem(_("Unknown token"), s,
+                                   par.id(), 0, par.size()));
                return 1;
        }
        return 0;
@@ -1057,10 +1061,10 @@ LyXFont const outerFont(ParagraphList::iterator pit,
        LyXFont tmpfont(LyXFont::ALL_INHERIT);
 
        // Resolve against environment font information
-       while (pit != plist.end() &&
+       while (pit != const_cast<ParagraphList&>(plist).end() &&
               par_depth && !tmpfont.resolved()) {
                pit = outerHook(pit, plist);
-               if (pit != plist.end()) {
+               if (pit != const_cast<ParagraphList&>(plist).end()) {
                        tmpfont.realize(pit->layout()->font);
                        par_depth = pit->getDepth();
                }