On Tue, 5 Sep 2006 10:08:14 +0200 Pavel Machek wrote:
> Hi!
>
> > > > So, just this morning I have seen questions and opinions about
> > > > the following that could (or could not) use more documentation
> > > > or codification and I'm sure that we could easily find more,
> > > > but do we want to codify Everything??
> > > >
> > > >
> > > > 1. Kconfig help text should be indented (it's not indented in the
> > > > GFS2 patches)
>
> Chapter 10 speaks about that alread?
OK, good.
I had only looked in Documentation/kbuild/kconfig-language.txt.
> > > > 2. if (!condition1) /* no space between ! and condition1
> */
> > > > 3. don't use C99-style // comments
> > > > 4. unsigned int bitfield :<nr_bits>;
>
> Ok.
>
> > > Looks reasonable to me. Will you do the patch or should I ?
> >
> > Please (you) go ahead with it.
>
> Does it look okay?
>
> Signed-off-by: Pavel Machek <pavel@xxxxxxx>
>
> diff --git a/Documentation/CodingStyle b/Documentation/CodingStyle
> index 6d2412e..d8e51a6 100644
> --- a/Documentation/CodingStyle
> +++ b/Documentation/CodingStyle
> @@ -46,6 +46,16 @@ used for indentation, and the above exam
>
> Get a decent editor and don't leave whitespace at the end of lines.
>
> +Bitfield variables should be indented like this:
> +
> + unsigned int foo :1;
> +
> +Avoid extra spaces around ! operator, and do not place spaces around (s.
I would spell out "parentheses".
> + if (!buffer)
> +
> +is okay, if ( ! buffer ) is just ugly.
> +
>
> Chapter 2: Breaking long lines and strings
>
> @@ -280,7 +290,7 @@ int fun(int a)
> int result = 0;
> char *buffer = kmalloc(SIZE);
>
> - if (buffer == NULL)
> + if (!buffer)
> return -ENOMEM;
>
> if (condition1) {
> @@ -316,6 +326,9 @@ When commenting the kernel API functions
> See the files Documentation/kernel-doc-nano-HOWTO.txt and scripts/kernel-doc
> for details.
>
> +Please do not use C99-style comments, and do not use comments to
^
I would insert: "// "
> +comment out unused code.
> +
Is there an acceptable way to leave source code in a file but
render it unused? Like #if 0/#endif or #if BOGUS_SYMBOL/#endif ?
> Chapter 9: You've made a mess of it
>
> That's OK, we all do. You've probably been told by your long-time Unix
---
~Randy
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/alsa-devel
