* src/LyXAction.C: mark goto-clear-bookmark as working without buffer

[lyx.git] / src / buffer_funcs.C

diff --git a/src/buffer_funcs.C b/src/buffer_funcs.C
index 8ed3e855a58923d276a9e7257736b3144b92e10c..52f9b6f5e325c6f70aca3f1daabeb7a74b1c3b8c 100644 (file)
--- a/src/buffer_funcs.C
+++ b/src/buffer_funcs.C
@@ -372,33 +372,35 @@ void setLabel(Buffer const & buf, ParIterator & it, LyXTextClass const & textcla
        // Compute the item depth of the paragraph
        par.itemdepth = getItemDepth(it);
 
-       // erase what was there before
-       par.params().labelString(docstring());
-
        if (layout->margintype == MARGIN_MANUAL) {
                if (par.params().labelWidthString().empty())
-                       par.setLabelWidthString(buf.translateLabel(layout->labelstring()));
+                       par.params().labelWidthString(par.translateIfPossible(layout->labelstring(), buf.params()));
        } else {
-               par.setLabelWidthString(docstring());
+               par.params().labelWidthString(docstring());
        }
 
+       // Optimisation: setLabel() can be called for each for each
+       // paragraph of the document. So we make the string static to
+       // avoid the repeated instanciation.
+       static docstring itemlabel;
+
        // is it a layout that has an automatic label?
        if (layout->labeltype == LABEL_COUNTER) {
                if (layout->toclevel <= buf.params().secnumdepth
                    && (layout->latextype != LATEX_ENVIRONMENT
                        || isFirstInSequence(it.pit(), it.plist()))) {
                        counters.step(layout->counter);
-                       docstring label = expandLabel(buf, layout,
-                                                     par.params().appendix());
-                       par.params().labelString(label);
-               }
+                       par.params().labelString(
+                               par.expandLabel(layout, buf.params()));
+               } else
+                       par.params().labelString(docstring());
+
        } else if (layout->labeltype == LABEL_ITEMIZE) {
                // At some point of time we should do something more
                // clever here, like:
                //   par.params().labelString(
                //    buf.params().user_defined_bullet(par.itemdepth).getText());
                // for now, use a simple hardcoded label
-               docstring itemlabel;
                switch (par.itemdepth) {
                case 0:
                        itemlabel = char_type(0x2022);
@@ -410,11 +412,11 @@ void setLabel(Buffer const & buf, ParIterator & it, LyXTextClass const & textcla
                        itemlabel = char_type(0x2217);
                        break;
                case 3:
-                       itemlabel += char_type(0x2219); // or 0x00b7
+                       itemlabel = char_type(0x2219); // or 0x00b7
                        break;
                }
-
                par.params().labelString(itemlabel);
+
        } else if (layout->labeltype == LABEL_ENUMERATE) {
                // FIXME
                // Yes I know this is a really, really! bad solution
@@ -463,46 +465,51 @@ void setLabel(Buffer const & buf, ParIterator & it, LyXTextClass const & textcla
                        break;
                }
 
-               par.params().labelString(counters.counterLabel(buf.B_(format)));
+               par.params().labelString(counters.counterLabel(
+                       par.translateIfPossible(from_ascii(format), buf.params())));
+
        } else if (layout->labeltype == LABEL_BIBLIO) {// ale970302
                counters.step(from_ascii("bibitem"));
                int number = counters.value(from_ascii("bibitem"));
                if (par.bibitem())
                        par.bibitem()->setCounter(number);
-               par.params().labelString(buf.translateLabel(layout->labelstring()));
-               // In biblio should't be following counters but...
+
+               par.params().labelString(
+                       par.translateIfPossible(layout->labelstring(), buf.params()));
+               // In biblio shouldn't be following counters but...
        } else if (layout->labeltype == LABEL_SENSITIVE) {
                // Search for the first float or wrap inset in the iterator
-               docstring type;
                size_t i = it.depth();
+               InsetBase * in;
                while (i > 0) {
                        --i;
-                       InsetBase * const in = &it[i].inset();
+                       in = &it[i].inset();
                        if (in->lyxCode() == InsetBase::FLOAT_CODE
-                           || in->lyxCode() == InsetBase::WRAP_CODE) {
-                               type = in->getInsetName();
+                           || in->lyxCode() == InsetBase::WRAP_CODE)
                                break;
-                       }
                }
+               docstring const & type = in->getInsetName();
 
-               docstring s;
                if (!type.empty()) {
                        Floating const & fl = textclass.floats().getType(to_ascii(type));
                        // FIXME UNICODE
                        counters.step(from_ascii(fl.type()));
 
                        // Doesn't work... yet.
-                       s = bformat(_("%1$s #:"), buf.B_(fl.name()));
+                       par.params().labelString(par.translateIfPossible(
+                               bformat(from_ascii("%1$s #:"), from_utf8(fl.name())),
+                               buf.params()));
                } else {
                        // par->SetLayout(0);
-                       s = buf.translateLabel(layout->labelstring());
+                       par.params().labelString(par.translateIfPossible(
+                               layout->labelstring(), buf.params()));
                }
 
-               par.params().labelString(s);
        } else if (layout->labeltype == LABEL_NO_LABEL)
                par.params().labelString(docstring());
        else
-               par.params().labelString(buf.translateLabel(layout->labelstring()));
+               par.params().labelString(
+                       par.translateIfPossible(layout->labelstring(), buf.params()));
 }
 
 } // anon namespace
@@ -609,30 +616,4 @@ void updateLabels(Buffer const & buf, bool childonly)
 }
 
 
-docstring expandLabel(Buffer const & buf,
-                     LyXLayout_ptr const & layout, bool appendix)
-{
-       LyXTextClass const & tclass = buf.params().getLyXTextClass();
-
-       docstring fmt = buf.translateLabel(appendix ?
-                       layout->labelstring_appendix() :
-                       layout->labelstring());
-
-       // handle 'inherited level parts' in 'fmt',
-       // i.e. the stuff between '@' in   '@Section@.\arabic{subsection}'
-       size_t const i = fmt.find('@', 0);
-       if (i != docstring::npos) {
-               size_t const j = fmt.find('@', i + 1);
-               if (j != docstring::npos) {
-                       docstring parent(fmt, i + 1, j - i - 1);
-                       // FIXME UNICODE
-                       docstring label = expandLabel(buf, tclass[to_utf8(parent)], appendix);
-                       fmt = docstring(fmt, 0, i) + label + docstring(fmt, j + 1, docstring::npos);
-               }
-       }
-
-       return tclass.counters().counterLabel(fmt);
-}
-
-
 } // namespace lyx