[Ns-developers] new ns-2 software contributions

John Heidemann johnh at ISI.EDU
Fri Mar 25 13:09:07 PST 2005


On Fri, 25 Mar 2005 08:51:15 +0100, Mathieu Lacage wrote: 
>...
>I would like to add:
>
>- conformance to a clear established coding style

This would be good, but would require reformatting some code
since there are a few distinct styles in the current code base.

>- conformance to a clear established set of architectural guidelines
>(i.e., use of STL or use of BSD-style lists, etc.)
>
>- detailed ChangeLog entry for _every_ checkin. Scripts can be used to
>generate the bulk of these entries. See
>http://developer.gnome.org/tools/scripts/prepare-ChangeLog.pl.

Our policy has been to use CHANGES.html for this purpose:

	CHANGES.html

	NS and nam both have a CHANGES.html file. Please record
	``medium-sized'' (or larger) changes in this file.
	
	What is a medium-sized change? A judgement call. Simple bug fixes
	aren't (the Makefile had an extra .o in OBJS and was breaking, I fixed
	it). Larger bug fixes are (the tahoe TCP module produced wrong results
	in a particular case). New features certainly are (multicast code is
	now present). If in doubt, ask someone.

Also, we had pre-commit requirements:

	Before Committing Changes to CVS
	
	Before commiting changes to ns, you should always run the
	validation test suites (run "./validate" in the top-level ns
	directory). Since ns is a large and complex piece of software
	it's critical to locate bugs early.
	
	This applies to even "obviously correct" one-line changes. All
	too frequently there's a minor error that slips through and
	someone else has to fix it.


   -John Heidemann


More information about the Ns-developers mailing list