Andre Poenitz wrote:
> On Thu, Apr 26, 2007 at 03:33:45PM +0200, Stephan Witt wrote:
>> 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}.
>>>
>> May I ask to fix src/frontends/controllers/ControlTabular.cpp also?
>> After moving and renaming LyxTabular the tree is broken...
>> Thank you.
> 
> Done.

Thank you.
But I had to "revert" your LCursor => Cursor rename to make it compile.

I'll wait a little bit to check again later...

Stephan

Reply via email to