[asterisk-dev] murf: trunk r130145 - /trunk/main/pbx.c

Russell Bryant russell at digium.com
Fri Jul 11 15:48:21 CDT 2008


SVN commits to the Digium repositories wrote:
> Author: murf
> Date: Fri Jul 11 13:24:31 2008
> New Revision: 130145

Please try to honor the formatting guidelines when adding new code.  I 
have included a few examples below of things that occur throughout your 
changes.


> +		for(c2=contexts;c2;c2=c2->next) {

missing space after for, and lack of proper spacing within statements

> +		if (c1)
> +		{

Invalid brace placement

> +				while( (e2 = ast_hashtab_next(t1)) ) {

missing space after while



-- 
Russell Bryant
Senior Software Engineer
Open Source Team Lead
Digium, Inc.



More information about the asterisk-dev mailing list