Andre Poenitz <[EMAIL PROTECTED]> writes:

| See attached.
>
| -- 
| Those who desire to give up Freedom in order to gain Security, will not have,
| nor do they deserve, either one.     (T. Jefferson or B. Franklin or both...)
>
| Index: lyxcursor.C
| ===================================================================
| RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/lyxcursor.C,v
| retrieving revision 1.26
| diff -u -p -r1.26 lyxcursor.C
| --- lyxcursor.C       9 Oct 2003 10:52:07 -0000       1.26
| +++ lyxcursor.C       16 Oct 2003 16:01:48 -0000
| @@ -17,7 +17,7 @@
|  
|  
|  LyXCursor::LyXCursor()
| -     : par_(-1), pos_(0), boundary_(false), x_(0), x_fix_(0), y_(0)
| +     : par_(0), pos_(0), boundary_(false), x_(0), x_fix_(0), y_(0)
|  {}

Is this -1 -> 0 part of the same "rowlist speedup" or is it from
another change?

| -RowList::iterator LyXText::getRowNearY(int y,
| -     ParagraphList::iterator & pit) const
| +RowList::iterator LyXText::getRowNearY(int y, ParagraphList::iterator & pit)
| +     const

can we please break the line in a different place?
(after the return type f.ex.)

I think it looks good.


-- 
        Lgb

Reply via email to