On Sat, Dec 14, 2013 at 06:33:43AM -0800, Dylan Baker wrote:
> On Friday, December 13, 2013 07:07:13 PM Brian Dolbec wrote:
> > -	confdefaults={ "storedir":"/var/tmp/catalyst",\
> > -		"sharedir":"/usr/share/catalyst","distdir":"/usr/portage/distfiles",\
> > -		"portdir":"/usr/portage","options":"",\
> > -		"snapshot_cache":"/var/tmp/catalyst/snapshot_cache",\
> > -		"hash_function":"crc32"}
> > +	confdefaults={
> > +		"hash_function": "crc32",
> > +		"portdir": "/usr/portage",
> > +		"repo_name": "portage",
> > +		"sharedir": "/usr/lib/catalyst",
> > +		"snapshot_name": "portage-",
> > +		"snapshot_cache": "/var/tmp/catalyst/snapshot_cache",
> > +		"storedir": "/var/tmp/catalyst",
> > +		}
> 
> pleas keep the formatting style; no comma on the last element and
> closing brace on the same line

The “one entry per line, comma after last element, separate brace
lines” format is both easier to read and easier to diff.  All the keys
are lined up in an alphabetized column for you, and if you add
something after storedir entry, you don't have to touch the storedir
line to remove a closing brace and add a comma.

Cheers,
Trevor

-- 
This email may be signed or encrypted with GnuPG (http://www.gnupg.org).
For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy