axis-c-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Damitha Kumarage <damith...@gmail.com>
Subject Re: Correct Axis2/C coding conventions.
Date Tue, 03 Jun 2008 15:05:18 GMT
Why can't we make it 120. Once we pass 80 limit anyway we have broken 
the old convention.

thanks
Damitha

Manjula Peiris wrote:

>+1 for 100.
>
>-Manjula.
>
>On Mon, 2008-05-05 at 13:54 +0530, Milinda Pathirage wrote:
>  
>
>>Here's my +1 for 100 as the number of columns for line.  Most of the
>>time we need 100 columns or more with the current naming conventions. 
>>
>>Thanks
>>Milinda
>>
>>On Mon, May 5, 2008 at 11:55 AM, Ruwan Janapriya <janapriya@wso2.com>
>>wrote:
>>        Yeah, agreed. 100 is better than 80, given that we have long
>>        variable/function names.
>>        
>>        so +1 for that.
>>        
>>        regards,
>>        
>>        Janapriya.
>>        
>>        
>>        
>>        On Mon, May 5, 2008 at 11:43 AM, Damitha Kumarage
>>        <damitha23@gmail.com> wrote:
>>                Hi,
>>                I suggest that number of columns per line should be
>>                100 instead of 80. Specifically as Axis2/C happen to
>>                have long variable/function names because namespace
>>                prefixes 80 columns width
>>                is hardly enough to write readable code.
>>                
>>                thoughts?
>>                thanks
>>                Damitha
>>                
>>                
>>                Ruwan Janapriya wrote:
>>                
>>                        Hi All,
>>                        
>>                        Following is extracted from Axis2/C coding
>>                        convention document [1].
>>                        
>>                        " 4. Function Parameters and Return Value
>>                        Conventions
>>                        
>>                             Each function should be passed a pointer
>>                        to an instance of the axutil_env_t struct as
>>                        the *first *parameter. If the function is
>>                        tightly bound to a struct, the *second
>>                        *parameter is a pointer to an instance of that
>>                        struct. "
>>                        
>>                        But almost all the functions implemented in
>>                        Axis2/C, axutil_env_t is passed as the *second
>>                        *parameter if there is a tightly bound struct
>>                        available(that struct would be the *first
>>                        *parameter).
>>                        
>>                        Further to above, AFAIK, in every "if", "else"
>>                        statements, Axis2/C uses curly brackets even
>>                        when there is one line of code present under
>>                        it. BUT in the above document has code
>>                        snippets, which does not adhere to that.
>>                        
>>                        don't we have to correct the document?
>>                        
>>                        regards,
>>                        
>>                        Janapriya
>>                        
>>                        
>>                        [1]
>>                        http://ws.apache.org/axis2/c/coding_conventions.html
>>                
>>                
>>                
>>                
>>                
>>                ---------------------------------------------------------------------
>>                To unsubscribe, e-mail:
>>                axis-c-dev-unsubscribe@ws.apache.org
>>                For additional commands, e-mail:
>>                axis-c-dev-help@ws.apache.org
>>                
>>                
>>        
>>        
>>
>>
>>
>>-- 
>>http://inf-dimensions.blogspot.com "Infinite Dimensions"
>>http://wsaxc.blogspot.com "Web Services With Axis2/C"
>>    
>>
>
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: axis-c-dev-unsubscribe@ws.apache.org
>For additional commands, e-mail: axis-c-dev-help@ws.apache.org
>
>
>  
>


---------------------------------------------------------------------
To unsubscribe, e-mail: axis-c-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-c-dev-help@ws.apache.org


Mime
View raw message