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.

Andre'

Reply via email to