On Sat, Mar 11, 2006 at 11:51:39AM +0100, Jean-Marc Lasgouttes wrote:
> >>>>> "Martin" == Martin Vermeer <[EMAIL PROTECTED]> writes:
> 
> Martin> A large part of the backspace method was factored out into
> Martin> backspacePos0. Some 50+ lines, that changes indent and show up
> Martin> as insertions/deletions in the patch. Out of total 203- and
> Martin> 256+.
> 
> If it is the same code, then OK. I guess having a ChangeLog would have
> spared you many questions...

Here is a doxygen patch to improve the documentation for this.

I'm sure you like this one. Goes in soon unless I hear screams.

- Martin

Index: lyxtext.h
===================================================================
--- lyxtext.h   (revision 13339)
+++ lyxtext.h   (working copy)
@@ -220,11 +220,14 @@
        void cursorTop(LCursor & cur);
        ///
        void cursorBottom(LCursor & cur);
-       ///
+       /// Delete character at cursor. Honour change tracking
        bool Delete(LCursor & cur);
-       ///
+       /** At cursor position 0, merge paragraph with the one before it.
+        * Ignore CT (this is used in \c acceptChange, \c rejectChange for
+        * physical deletion of paragraph break)
+        */
        bool backspacePos0(LCursor & cur);
-       ///
+       /// Delete character before cursor. Honour CT
        bool backspace(LCursor & cur);
        ///
        bool selectWordWhenUnderCursor(LCursor & cur, lyx::word_location);

Attachment: pgpAfugG84T3c.pgp
Description: PGP signature

Reply via email to