On Thursday 26 April 2007 8:29:43 am Andre Poenitz wrote:
> So... definitely a step in the right direction.
>
> I've a few issues left:
>
> 1. I'd like to get rid of the 'LyX'/'L' prefix whenever possible.
>    I mean, it is clear that 'LyXLayout' in src/ has something todo with
>    LyX, isn't it. Of course this mean
>
> 2. rowpainter.cpp is all about a class RowPainter, which is used only
>    locally, so it does not show up in the .h. I'd prefer 'RowPainter'
>    as file names here.
>
> 3. I'd rather rename classes kb_keymap into KeyMap, and the files
>    in KeyMap.{h,cpp}.

  Those arguments seems sensible to me. :-)

> Andre'

-- 
José Abílio

Reply via email to