On Wed, 25 Apr 2007, Bo Peng wrote:

 I vote d.

You guys already won. see src/frontends/controllers/frontend_helpers.h/cpp. :-)

I have also changed file extension for controllers/*.C, and that will be all for today. I need to leave some time for you guys to check if everything is still working.

Ok.

/Christian

PS. I added the following to Devel/UnsortedNotes:

!!!! Bo's guidelines:

Guideline 1: Classname == file name

Because classnames are in uppercase, files are also named in this
way. src/insets/*, src/mathed/* and src/graphics/* follow this guideline
well so these are what I have renamed so far.

Guideline 2: Clean or merge helper files

If they cannot be merged, the name should be something like:
--> frontend_helper.{h,cpp}

This format will also be applied to _fwd and _pimpl and _funcs.

!!!! André's guidelines

If there's a single class Foo implemented in the file, it should be
Foo.h/Foo.cpp. That should be the common case.

If there are classes FooSomething, FooHelper, FooSimilar in the file,
it should be named Foo.h/Foo.cpp.

If there are free functions fooHelper, fooSimilar, fooOther in the file,
it should be named foo.{h,cpp}.

Name clashes should be avoided.

There should be no {L,LyX,Q,QL} prefix to class and file names.


--
Christian Ridderström, +46-8-768 39 44               http://www.md.kth.se/~chr

Reply via email to