Re: svn commit: r1057088 - in /subversion/trunk/subversion: libsvn_fs_base/bdb/env.c libsvn_ra_svn/cyrus_auth.c libsvn_subr/io.c libsvn_subr/sqlite.c

On 01/12/2011 09:54 AM, Branko Čibej wrote:
> 's true that static (and in fact all global) storage is default-inited
> to 0 ... but there's no harm in putting the initializer there. IMO it's
> better to have it, if only for the sake of clarity.

You took the words right outta my ... Outbox.

I take a similar approach to the use of parentheses around conditionals.
Maybe you don't *need* them due to order of operations rules, but the
additional clarity is worth it to me.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand