Remove unneccessary uses of dynamic_cast from the code.

[lyx.git] / src / mathed / MacroTable.cpp

diff --git a/src/mathed/MacroTable.cpp b/src/mathed/MacroTable.cpp
index 406e901c49d0f3503f9606e379540b81c866e9e4..20a713a513e15e1d3af377282e4cfe63d359cfe5 100644 (file)
--- a/src/mathed/MacroTable.cpp
+++ b/src/mathed/MacroTable.cpp
@@ -3,7 +3,7 @@
  * This file is part of LyX, the document processor.
  * Licence details can be found in the file COPYING.
  *
- * \author André Pönitz
+ * \author André Pönitz
  *
  * Full author contact details are available in file CREDITS.
  */
@@ -19,13 +19,12 @@
 #include "InsetMathNest.h"
 
 #include "Buffer.h"
+#include "DocIterator.h"
 #include "InsetList.h"
 #include "Text.h"
 
 #include "support/debug.h"
-#include "DocIterator.h"
-
-#include <boost/assert.hpp>
+#include "support/lassert.h"
 
 #include <sstream>
 
@@ -40,42 +39,45 @@ namespace lyx {
 //
 /////////////////////////////////////////////////////////////////////
 
-MacroData::MacroData()
-       : queried_(true), numargs_(0), optionals_(0), lockCount_(0),
+MacroData::MacroData(Buffer * buf)
+       : buffer_(buf), queried_(true), numargs_(0), optionals_(0), lockCount_(0),
          redefinition_(false), type_(MacroTypeNewcommand)
 {}
 
        
        
-MacroData::MacroData(Buffer const & buf, DocIterator const & pos)
-       : buffer_(&buf), pos_(pos), queried_(false), numargs_(0),
+MacroData::MacroData(Buffer * buf, DocIterator const & pos)
+       : buffer_(buf), pos_(pos), queried_(false), numargs_(0),
          optionals_(0), lockCount_(0), redefinition_(false),
          type_(MacroTypeNewcommand)
 {
 }
        
        
-MacroData::MacroData(MathMacroTemplate const & macro)
-       : queried_(false), numargs_(0), optionals_(0), lockCount_(0),
+MacroData::MacroData(Buffer * buf, MathMacroTemplate const & macro)
+       : buffer_(buf), queried_(false), numargs_(0), optionals_(0), lockCount_(0),
          redefinition_(false), type_(MacroTypeNewcommand)
 {
        queryData(macro);
-}      
+}
 
 
 void MacroData::expand(vector<MathData> const & args, MathData & to) const
 {
        updateData();
 
-       InsetMathSqrt inset; // Hack. Any inset with a cell would do.
+       // Hack. Any inset with a cell would do.
+       static InsetMathSqrt inset(0);
+       inset.setBuffer(const_cast<Buffer &>(*buffer_));
+
        // FIXME UNICODE
        asArray(display_.empty() ? definition_ : display_, inset.cell(0));
        //lyxerr << "MathData::expand: args: " << args << endl;
        //lyxerr << "MathData::expand: ar: " << inset.cell(0) << endl;
-       for (DocIterator it = doc_iterator_begin(inset); it; it.forwardChar()) {
+       for (DocIterator it = doc_iterator_begin(buffer_, &inset); it; it.forwardChar()) {
                if (!it.nextInset())
                        continue;
-               if (it.nextInset()->lyxCode() != MATHMACROARG_CODE)
+               if (it.nextInset()->lyxCode() != MATH_MACROARG_CODE)
                        continue;
                //it.cell().erase(it.pos());
                //it.cell().insert(it.pos(), it.nextInset()->asInsetMath()
@@ -107,7 +109,7 @@ vector<docstring> const &  MacroData::defaults() const
 void MacroData::unlock() const
 {
        --lockCount_;
-       BOOST_ASSERT(lockCount_ >= 0);
+       LASSERT(lockCount_ >= 0, /**/);
 }
 
 
@@ -123,7 +125,8 @@ void MacroData::queryData(MathMacroTemplate const & macro) const
        redefinition_ = macro.redefinition();
        type_ = macro.type();
        optionals_ = macro.numOptionals();
-       macro.getDefaults(defaults_);   
+       
+       macro.getDefaults(defaults_);
 }
 
 
@@ -132,7 +135,7 @@ void MacroData::updateData() const
        if (queried_)
                return;
 
-       BOOST_ASSERT(buffer_ != 0);
+       LASSERT(buffer_ != 0, /**/);
        
        // Try to fix position DocIterator. Should not do anything in theory.
        pos_.fixIfBroken();
@@ -149,7 +152,7 @@ void MacroData::updateData() const
 }
        
 
-void MacroData::write(odocstream & os, bool overwriteRedefinition) const
+int MacroData::write(odocstream & os, bool overwriteRedefinition) const
 {
        updateData();
 
@@ -157,14 +160,14 @@ void MacroData::write(odocstream & os, bool overwriteRedefinition) const
        Inset * inset = pos_.nextInset();
        if (inset == 0 || inset->lyxCode() != MATHMACRO_CODE) {
                lyxerr << "BUG: No macro template found by MacroData" << endl;
-               return;
+               return 0;
        }
                
        // output template
-       MathMacroTemplate const & tmpl 
-       = static_cast<MathMacroTemplate const &>(*inset);
-       WriteStream wi(os, false, true);
-       tmpl.write(wi, overwriteRedefinition);
+       MathMacroTemplate const & tmpl =
+               static_cast<MathMacroTemplate const &>(*inset);
+       WriteStream wi(os, false, true, WriteStream::wsDefault);
+       return tmpl.write(wi, overwriteRedefinition);
 }
 
 
@@ -195,16 +198,23 @@ void MacroTable::insert(docstring const & name, MacroData const & data)
 }
 
 
-void MacroTable::insert(docstring const & def, string const & requires)
+void MacroTable::insert(Buffer * buf, docstring const & def, string const & requires)
 {
        //lyxerr << "MacroTable::insert, def: " << to_utf8(def) << endl;
-       MathMacroTemplate mac(def);
-       MacroData data(mac);
+       MathMacroTemplate mac(buf, def);
+       MacroData data(buf, mac);
        data.requires() = requires;
        insert(mac.name(), data);
 }
 
 
+void MacroTable::getMacroNames(std::set<docstring> & names) const
+{
+       for (const_iterator it = begin(); it != end(); ++it)
+               names.insert(it->first);
+}
+
+
 void MacroTable::dump()
 {
        lyxerr << "\n------------------------------------------" << endl;
@@ -223,7 +233,7 @@ void MacroTable::dump()
 //
 /////////////////////////////////////////////////////////////////////
 
-MacroContext::MacroContext(Buffer const & buf, DocIterator const & pos)
+MacroContext::MacroContext(Buffer const * buf, DocIterator const & pos)
        : buf_(buf), pos_(pos)
 {
 }
@@ -231,7 +241,7 @@ MacroContext::MacroContext(Buffer const & buf, DocIterator const & pos)
 
 MacroData const * MacroContext::get(docstring const & name) const
 {
-       return buf_.getMacro(name, pos_);
+       return buf_->getMacro(name, pos_);
 }
 
 } // namespace lyx