Angus Leeming wrote:

> Index: BufferView_pimpl.C
> Index: lyxfunc.C
> Index: lyxtext.h
> Revert the addition of spurious whitespace.

Sorry will do.
 
> Index: lyxtext.h
> +       /// absolute document pixel coordinates of this LyXText
> +       int x0_;
> +       int y0_;
> I thought these were supposed to be xo_, yo_...
> 
> -
> +//gets LyXText coordinates
>  void LyXText::setCursorFromCoordinates(LyXCursor & cur, int x, int y)
> Does this comment add anything? To me it just adds confusion.

well it does: it means that x,y are supposed to be coordinates relative to
that particular LyXText.

> I do like the ratio of '-' to '+' lines though ;-)

Nice isn't it?

Anyway this is yesterday's patch. ;-)

Alfredo


Reply via email to