X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FCoordCache.cpp;h=7e881780ebdb0b61b855664665a9acf3a8c28e84;hb=26ba2a65838731ce639a09539f617cb0f0be3b22;hp=0974dc6c20069c92da975b3aea9fc483cf64c3d9;hpb=f0fee538c2862ea4eaefd9ca12ab7f80f98d076a;p=lyx.git diff --git a/src/CoordCache.cpp b/src/CoordCache.cpp index 0974dc6c20..7e881780eb 100644 --- a/src/CoordCache.cpp +++ b/src/CoordCache.cpp @@ -2,7 +2,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. */ @@ -11,33 +11,20 @@ #include "CoordCache.h" -#include "debug.h" -#include "Text.h" -#include "support/docstring.h" +#include "support/debug.h" -#include "insets/Inset.h" - -#include +#include "support/lassert.h" namespace lyx { -Point::Point(int x, int y) - : x_(x), y_(y) -{ - BOOST_ASSERT(x > -1000000); - BOOST_ASSERT(x < 1000000); - BOOST_ASSERT(y > -1000000); - BOOST_ASSERT(y < 1000000); -} - // just a helper to be able to set a breakpoint void lyxbreaker(void const * data, const char * hint, int size) { - lyxerr << "break on pointer: " << data << " hint: " << hint - << " size: " << size << std::endl; - BOOST_ASSERT(false); + LYXERR0("break on pointer: " << data << " hint: " << hint + << " size: " << size); + LBUFERR(false); } @@ -50,12 +37,18 @@ void CoordCache::clear() void CoordCache::dump() const { - lyxerr << "InsetCache contains:" << std::endl; - for (CoordCacheBase::cache_type::const_iterator it = getInsets().getData().begin(); it != getInsets().getData().end(); ++it) { - Inset const * inset = it->first; - Point const p = it->second.pos; - lyxerr << "Inset " << inset << "(" << to_utf8(inset->name()) - << ") has point " << p.x_ << "," << p.y_ << std::endl; + if (getInsets().data_.empty()) { + LYXERR0("InsetCache is empty."); + return; + } + + LYXERR0("InsetCache contains:"); + for (auto const & ccd : getInsets().data_) { + // Warning: it is not guaranteed that inset is a valid pointer + // (therefore it has type 'void *') (see bug #7376). + void const * inset = ccd.first; + Point const p = ccd.second.pos; + LYXERR(Debug::PAINTING, "Inset " << inset << " has point " << p.x_ << "," << p.y_); } }