On 5-4-2011 17:51, Enrico Forestieri wrote:
On Tue, Apr 05, 2011 at 05:45:38PM +0200, Stephan Witt wrote:
Am 05.04.2011 um 17:40 schrieb for...@lyx.org:

Author: forenr
Date: Tue Apr  5 17:40:00 2011
New Revision: 38257
URL: http://www.lyx.org/trac/changeset/38257

Log:
Generalize check to generic non-editable insets.

Modified:
   lyx-devel/trunk/src/frontends/qt4/GuiErrorList.cpp

Modified: lyx-devel/trunk/src/frontends/qt4/GuiErrorList.cpp
==============================================================================
--- lyx-devel/trunk/src/frontends/qt4/GuiErrorList.cpp  Tue Apr  5 16:54:38 
2011        (r38256)
+++ lyx-devel/trunk/src/frontends/qt4/GuiErrorList.cpp  Tue Apr  5 17:40:00 
2011        (r38257)
@@ -172,8 +172,8 @@
                return false;
        }

-       // Don't try to highlight the content of info insets
-       while (dit.inset().lyxCode() == INFO_CODE)
+       // Don't try to highlight the content of non-editable insets
+       while (!dit.inset().editable())
                dit.backwardPos();
Isn't it a must to check for reaching the begin of document?
Should not be necessary... (famous last words)

What enrico means to say is that we always encounter a cursor position in the main InsetText before reaching the beginning of the document (i.e. slice 0, pit 0, pos 0).

Vincent

Reply via email to