On 2009-09-22 15:52+0100 Andrew Ross wrote: > I have gone ahead and committed a uncrustified version of plpage.c. This > does not seem to have affected the doxygen comments. The changes in this > case were relatively minor suggesting that we have a good "style". The > core code is on the whole far more uniform. > > If people are happy I will progress with the rest of src/ and bindings/c++.
When evaluating these style changes, I highly recommend using, e.g., http://plplot.svn.sourceforge.net/viewvc/plplot/trunk/src ==> plpage.c ==> Diff to previous. Those coloured diffs are great for clearly showing the changes, and I like what I see. Of course, others should judge for themselves, but the principal style issues for me are consistency and (now that my eyes aren't quite as good as they used to be) visual clarity via appropriate whitespace. I am happy those fundamental style goals are finally within our grasp at long last due to the power of uncrustify. Alan __________________________ Alan W. Irwin Astronomical research affiliation with Department of Physics and Astronomy, University of Victoria (astrowww.phys.uvic.ca). Programming affiliations with the FreeEOS equation-of-state implementation for stellar interiors (freeeos.sf.net); PLplot scientific plotting software package (plplot.org); the libLASi project (unifont.org/lasi); the Loads of Linux Links project (loll.sf.net); and the Linux Brochure Project (lbproject.sf.net). __________________________ Linux-powered Science __________________________ ------------------------------------------------------------------------------ Come build with us! The BlackBerry® Developer Conference in SF, CA is the only developer event you need to attend this year. Jumpstart your developing skills, take BlackBerry mobile applications to market and stay ahead of the curve. Join us from November 9-12, 2009. Register now! http://p.sf.net/sfu/devconf _______________________________________________ Plplot-devel mailing list Plplot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/plplot-devel
