On 25.06.2008, at 09:25, Craig D wrote: >> Is there any interest in general code quality cleanup? > > This seems like such a religious issue but a style guideline would be > useful for community code, especially for us ancients for whom IDE > means > emacs.
A style guide? Like this one? <http://www.fltk.org/cmp.php#CODING_STANDARDS> As you may guys may have read in earlier posts, we are currently discussing how to get fltk 1 and fltk 2 back together again and bundle forces to streamline the process. Since fltk1 and fltk2 actually did branch from the same code base quite a while back, if would be useful to keep the code as is for a while to make diff's easier. Once we are back to stable, I won't mind a code and API cleanup at all. > And > of course the similarly awesomely named FL_X (declared in Fl_x.cxx and > x.H, naturally, and called 'x' in the source... not to be confused > with the 'x()' method!) has an 'i' but it's a method this time. FLTK's API was based on the Forms Library in an effort to replace that free library when it was going commercial. FLTK2 got some far reaching changes in the API, but some old shoes remained in the closet. > Trying to understand Fl_Tile::handle was the hardest I've ever worked > trying to read *anything* including handwritten balinese astrological > charts Yes, that one is a monster. FLTK in its nature as an Open SOurce project has been put together from many sources and by many authors, everyone of them having different styles, preferences, and - um - skills (when I look at code that I wrote ten years ago, I shudder. My coding today is not that much better, but at least somewhat more readable. I envy those who can keep total coding discipline: congratulations!). Which also explains the tabulator chaos. We did agree on two spaces per indent at least. If we now manage to agree on how amny spaces are in a tab, we will be fine ;-). Matthias ---- http://robowerk.com/ _______________________________________________ fltk mailing list [email protected] http://lists.easysw.com/mailman/listinfo/fltk
