X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FCoordCache.cpp;h=dedd1355b677e3af008c314059238855ed9569a4;hb=4db3e641ed6765e005343010cb90ee8af26f8f99;hp=56997467e8e05365b2e87b47da7977925023921e;hpb=f1cba8ff64b369792fd49f5ddf90e8126ab476ac;p=lyx.git

diff --git a/src/CoordCache.cpp b/src/CoordCache.cpp
index 56997467e8..dedd1355b6 100644
--- a/src/CoordCache.cpp
+++ b/src/CoordCache.cpp
@@ -26,18 +26,19 @@ namespace lyx {
 Point::Point(int x, int y)
 	: x_(x), y_(y)
 {
-	LASSERT(x > -1000000, /**/);
-	LASSERT(x <  1000000, /**/);
-	LASSERT(y > -1000000, /**/);
-	LASSERT(y <  1000000, /**/);
+	LASSERT(x > -1000000, x = -1000000);
+	LASSERT(x <  1000000, x =  1000000);
+	LASSERT(y > -1000000, y = -1000000);
+	LASSERT(y <  1000000, y =  1000000);
 }
 
+
 // just a helper to be able to set a breakpoint
 void lyxbreaker(void const * data, const char * hint, int size)
 {
 	LYXERR0("break on pointer: " << data << " hint: " << hint
 		<< " size: " << size);
-	LASSERT(false, /**/);
+	LASSERT(false, return);
 }
 
 
@@ -51,11 +52,14 @@ void CoordCache::clear()
 void CoordCache::dump() const
 {
 	LYXERR0("InsetCache contains:");
-	for (CoordCacheBase<Inset>::cache_type::const_iterator it = getInsets().getData().begin(); it != getInsets().getData().end(); ++it) {
-		Inset const * inset = it->first;
+	CoordCacheBase<Inset>::cache_type::const_iterator it =
+		getInsets().data_.begin();
+	for (; it != getInsets().data_.end(); ++it) {
+		// Warning: it is not guaranteed that inset is a valid pointer
+		// (therefore it has type 'void *') (see bug #7376).	
+		void const * inset = it->first;
 		Point const p = it->second.pos;
-		LYXERR0("Inset " << inset << "(" << to_utf8(inset->name())
-			<< ") has point " << p.x_ << "," << p.y_);
+		LYXERR0("Inset " << inset << " has point " << p.x_ << "," << p.y_);
 	}
 }