Re: [PATCH] CodingStyle: Document the exception of not splitting user-visible strings, for grepping

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Josh Triplett <josh <at> joshtriplett.org> writes:
[...]
> -void fun(int a, int b, int c)
> -{
> -	if (condition)
> -		printk(KERN_WARNING "Warning this is a long printk with "
> -						"3 parameters a: %u b: %u "
> -						"c: %u \n", a, b, c);
> -	else
> -		next_statement;
> -}
> +Statements longer than 80 columns will be broken into sensible chunks, unless
> +exceeding 80 columns significantly increases readability and does not hide
> +information. Descendants are always substantially shorter than the parent and
> +are placed substantially to the right. The same applies to function headers
> +with a long argument list. However, never break user-visible strings such as
> +printk messages, because that breaks the ability to grep for them.
> +
> 
>  		Chapter 3: Placing Braces and Spaces
> 

Hi,

is there any particular reason why this patch has yet not made it to

http://www.kernel.org/doc/Documentation/CodingStyle

Thanks!
Gerlando


--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux