"Hyrum K. Wright" <hyrum_wright@mail.utexas.edu> writes:
> After spending a while in various parts of our code base, I've noticed
> an inconsistency between our declared coding style in HACKING, and
> what we actually do. The following patch to HACKING makes it
> congruent with our current practices. Comments?
I think we actually go both ways, depending on how long the argument
list is. Could you make hacking.html reflect that?
> [[[
> * www/hacking.html
> (coding-style): Emphasize our one-argument-per-line convention for
> function declarations.
> ]]]
> Index: hacking.html
> ===================================================================
> --- hacking.html (revision 25767)
> +++ hacking.html (working copy)
> @@ -380,7 +380,8 @@
>
> <pre>
> char * /* func type on own line */
> - argblarg(char *arg1, int arg2) /* func name on own line */
> + argblarg(char *arg1, /* func name on own line */
> + int arg2) /* each arg on own line */
> { /* first brace on own line */
> if ((some_very_long_condition && arg2) /* indent 2 cols */
> || remaining_condition) /* new line before operator */
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Jul 17 18:35:01 2007