The speed patch: redraw only rows that have changed

[lyx.git] / src / insets / insetbibitem.C

diff --git a/src/insets/insetbibitem.C b/src/insets/insetbibitem.C
index 670c4d200e79ef73ebf5a306a673e20fcee07660..c26cb17a6e530884b2cbf37228256a377c0f4e71 100644 (file)
--- a/src/insets/insetbibitem.C
+++ b/src/insets/insetbibitem.C
@@ -5,76 +5,73 @@
  *
  * \author Alejandro Aguilar Sierra
  *
- * Full author contact details are available in file CREDITS
+ * Full author contact details are available in file CREDITS.
  */
+
 #include <config.h>
 
 #include "insetbibitem.h"
+
 #include "buffer.h"
 #include "BufferView.h"
+#include "dispatchresult.h"
 #include "funcrequest.h"
+#include "lyxfont.h"
 #include "lyxlex.h"
+#include "paragraph.h"
+#include "ParagraphList_fwd.h"
 
 #include "frontends/font_metrics.h"
 
 #include "support/lstrings.h"
+#include "support/std_ostream.h"
+#include "support/convert.h"
 
+using lyx::support::prefixIs;
 
 using std::max;
-
+using std::string;
+using std::auto_ptr;
+using std::ostream;
 
 int InsetBibitem::key_counter = 0;
-const string key_prefix = "key-";
+string const key_prefix = "key-";
+
 
 InsetBibitem::InsetBibitem(InsetCommandParams const & p)
-       : InsetCommand(p), counter(1)
+       : InsetCommand(p, "bibitem"), counter(1)
 {
        if (getContents().empty())
-               setContents(key_prefix + tostr(++key_counter));
+               setContents(key_prefix + convert<string>(++key_counter));
 }
 
 
-InsetBibitem::~InsetBibitem()
+auto_ptr<InsetBase> InsetBibitem::doClone() const
 {
-       InsetCommandMailer mailer("bibitem", *this);
-       mailer.hideDialog();
-}
-
-
-Inset * InsetBibitem::clone(Buffer const &, bool) const
-{
-       InsetBibitem * b = new InsetBibitem(params());
+       auto_ptr<InsetBibitem> b(new InsetBibitem(params()));
        b->setCounter(counter);
-       return b;
+       return auto_ptr<InsetBase>(b);
 }
 
 
-dispatch_result InsetBibitem::localDispatch(FuncRequest const & cmd)
+void InsetBibitem::doDispatch(LCursor & cur, FuncRequest & cmd)
 {
-       Inset::RESULT result = UNDISPATCHED;
-
        switch (cmd.action) {
+
        case LFUN_INSET_MODIFY: {
                InsetCommandParams p;
-               InsetCommandMailer::string2params(cmd.argument, p);
-               if (p.getCmdName().empty())
-                       break;
-
-               if (view() && p.getContents() != params().getContents()) {
-                       view()->ChangeCitationsIfUnique(params().getContents(),
-                                                       p.getContents());
-               }
-
-               setParams(p);
-               cmd.view()->updateInset(this, true);
-               result = DISPATCHED;
+               InsetCommandMailer::string2params("bibitem", cmd.argument, p);
+               if (!p.getCmdName().empty())
+                       setParams(p);
+               else
+                       cur.noUpdate();
+               break;
        }
-       break;
+
        default:
-               result = InsetCommand::localDispatch(cmd);
+               InsetCommand::doDispatch(cur, cmd);
+               break;
        }
-
-       return result;
 }
 
 
@@ -88,7 +85,7 @@ void InsetBibitem::setCounter(int c)
 // as a LyX 2.x command, and lyxlex is not enough smart to understand
 // real LaTeX commands. Yes, that could be fixed, but would be a waste
 // of time cause LyX3 won't use lyxlex anyway.  (ale)
-void InsetBibitem::write(Buffer const *, std::ostream & os) const
+void InsetBibitem::write(Buffer const &, std::ostream & os) const
 {
        os << "\n\\bibitem ";
        if (!getOptions().empty())
@@ -99,16 +96,15 @@ void InsetBibitem::write(Buffer const *, std::ostream & os) const
 
 // This is necessary here because this is written without begin_inset
 // This should be changed!!! (Jug)
-void InsetBibitem::read(Buffer const *, LyXLex & lex)
+void InsetBibitem::read(Buffer const &, LyXLex & lex)
 {
-       if (lex.eatLine()) {
+       if (lex.eatLine())
                scanCommand(lex.getString());
-       } else {
+       else
                lex.printError("InsetCommand: Parse error: `$$Token'");
-       }
 
        if (prefixIs(getContents(), key_prefix)) {
-               int key = strToInt(getContents().substr(key_prefix.length()));
+               int const key = convert<int>(getContents().substr(key_prefix.length()));
                key_counter = max(key_counter, key);
        }
 }
@@ -116,59 +112,35 @@ void InsetBibitem::read(Buffer const *, LyXLex & lex)
 
 string const InsetBibitem::getBibLabel() const
 {
-       return getOptions().empty() ? tostr(counter) : getOptions();
+       return getOptions().empty() ? convert<string>(counter) : getOptions();
 }
 
 
-string const InsetBibitem::getScreenLabel(Buffer const *) const
+string const InsetBibitem::getScreenLabel(Buffer const &) const
 {
        return getContents() + " [" + getBibLabel() + ']';
 }
 
-
-void InsetBibitem::edit(BufferView * bv, int, int, mouse_button::state)
+int InsetBibitem::plaintext(Buffer const &, ostream & os,
+                           OutputParams const &) const
 {
-       InsetCommandMailer mailer("bibitem", *this);
-       mailer.showDialog(bv);
-}
-
-
-void InsetBibitem::edit(BufferView * bv, bool)
-{
-       edit(bv, 0, 0, mouse_button::none);
-}
-
-
-// ale070405 This function maybe shouldn't be here. We'll fix this at 0.13.
-int bibitemMaxWidth(BufferView * bv, LyXFont const & font)
-{
-       int w = 0;
-       // Ha, now we are mainly at 1.2.0 and it is still here (Jug)
-       // Does look like a hack? It is! (but will change at 0.13)
-       ParagraphList::iterator it = bv->buffer()->paragraphs.begin();
-       ParagraphList::iterator end = bv->buffer()->paragraphs.end();
-       for (; it != end; ++it) {
-               if (it->bibitem()) {
-                       int const wx = it->bibitem()->width(bv, font);
-                       if (wx > w)
-                               w = wx;
-               }
-       }
-       return w;
+       os << '[' << getCounter() << "] ";
+       return 0;
 }
 
 
 // ale070405
-string const bibitemWidest(Buffer const * buffer)
+string const bibitemWidest(Buffer const & buffer)
 {
        int w = 0;
        // Does look like a hack? It is! (but will change at 0.13)
 
-       InsetBibitem * bitem = 0;
+       InsetBibitem const * bitem = 0;
        LyXFont font;
 
-       ParagraphList::iterator it = buffer->paragraphs.begin();
-       ParagraphList::iterator end = buffer->paragraphs.end();
+       ParagraphList::const_iterator it = buffer.paragraphs().begin();
+       ParagraphList::const_iterator end = buffer.paragraphs().end();
+
        for (; it != end; ++it) {
                if (it->bibitem()) {
                        int const wx =