X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FCursorSlice.cpp;h=0e46c0781fc760ac8e8783fcd099011db555151f;hb=4db3e641ed6765e005343010cb90ee8af26f8f99;hp=09cf6241346ec097d9cfdd152d691a936a1f0f5c;hpb=0014ca7eee593b05959b99a0d9afb263f61ecf01;p=lyx.git

diff --git a/src/CursorSlice.cpp b/src/CursorSlice.cpp
index 09cf624134..0e46c0781f 100644
--- a/src/CursorSlice.cpp
+++ b/src/CursorSlice.cpp
@@ -3,10 +3,10 @@
  * This file is part of LyX, the document processor.
  * Licence details can be found in the file COPYING.
  *
- * \author Lars Gullik Bjønnes
+ * \author Lars Gullik Bjønnes
  * \author Matthias Ettrich
- * \author André Pönitz
- * \author Jürgen Vigna
+ * \author André Pönitz
+ * \author Jürgen Vigna
  *
  * Full author contact details are available in file CREDITS.
  */
@@ -23,9 +23,11 @@
 #include "insets/Inset.h"
 
 #include "mathed/InsetMath.h"
-#include "mathed/MathData.h"
+#include "mathed/MathMacro.h"
 
-#include <boost/assert.hpp>
+#include "support/ExceptionMessage.h"
+#include "support/gettext.h"
+#include "support/lassert.h"
 
 #include <ostream>
 
@@ -42,7 +44,7 @@ CursorSlice::CursorSlice()
 CursorSlice::CursorSlice(Inset & p)
 	: inset_(&p), idx_(0), pit_(0), pos_(0)
 {
-	BOOST_ASSERT(inset_);
+	LBUFERR(inset_);
 }
 
 
@@ -60,9 +62,11 @@ Paragraph & CursorSlice::paragraph() const
 
 pos_type CursorSlice::lastpos() const
 {
-	BOOST_ASSERT(inset_);
-	return inset_->asInsetMath() ? cell().size() 
-		: (text()->empty() ? 0 : paragraph().size());
+	LBUFERR(inset_);
+	InsetMath const * math = inset_->asInsetMath();
+	bool paramless_macro = math && math->asMacro() && !math->asMacro()->nargs();
+	return math ? (paramless_macro ? 0 : cell().size()) 
+		    : (text()->empty() ? 0 : paragraph().size());
 }
 
 
@@ -76,14 +80,18 @@ pit_type CursorSlice::lastpit() const
 
 CursorSlice::row_type CursorSlice::row() const
 {
-	BOOST_ASSERT(asInsetMath());
+	// LASSERT: This should only ever be called from an InsetMath.
+	// Should we crash in release mode, though, or try to continue?
+	LASSERT(asInsetMath(), /**/);
 	return asInsetMath()->row(idx_);
 }
 
 
 CursorSlice::col_type CursorSlice::col() const
 {
-	BOOST_ASSERT(asInsetMath());
+	// LASSERT: This should only ever be called from an InsetMath.
+	// Should we crash in release mode, though, or try to continue?
+	LASSERT(asInsetMath(), /**/);
 	return asInsetMath()->col(idx_);
 }
 
@@ -108,7 +116,7 @@ void CursorSlice::forwardPos()
 	// otherwise move on one cell
 	//lyxerr << "... next idx" << endl;
 
-	BOOST_ASSERT(idx_ < nargs());
+	LASSERT(idx_ < nargs(), return);
 
 	++idx_;
 	pit_ = 0;
@@ -118,7 +126,7 @@ void CursorSlice::forwardPos()
 
 void CursorSlice::forwardIdx()
 {
-	BOOST_ASSERT(idx_ < nargs());
+	LASSERT(idx_ < nargs(), return);
 
 	++idx_;
 	pit_ = 0;
@@ -146,7 +154,7 @@ void CursorSlice::backwardPos()
 		return;
 	}
 
-	BOOST_ASSERT(false);
+	LATTEST(false);
 }
 
 
@@ -185,7 +193,8 @@ bool operator<(CursorSlice const & p, CursorSlice const & q)
 	if (p.inset_ != q.inset_) {
 		LYXERR0("can't compare cursor and anchor in different insets\n"
 		       << "p: " << p << '\n' << "q: " << q);
-		BOOST_ASSERT(false);
+		// It should be safe to continue, just registering the error.
+		LASSERT(false, return false);
 	}
 	if (p.idx_ != q.idx_)
 		return p.idx_ < q.idx_;