Rockbox mail archive

• Next message: Claudio Sabia: "1 Bug, 1 misbehaviour and a feature"
• Previous message: Markus Braun: "[PATCH] scrollbar for tree and menu on recorders"
• In reply to: perterm: "Coding conventions"
• Next in thread: perterm: "Re: Coding conventions"
• Reply: perterm: "Re: Coding conventions"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ] [ attachment ]

Directly from the Contributing file:
http://rockbox.haxx.se/docs/contributing.html

"When changing code, follow the code style of the file you are editing.
 When writing new files, you may use the brace placement style of your
 choice."

If your format has been changed, you can check with the Code Police ;)

(aka: Zagor :) )

Hope that helps..

/Adi

On Thu, 29 Aug 2002, perterm wrote:

Robert Hak wrote:
> avoid putting the if clause's on the same
> line:
>
> if(true) i++;
>
> should be
>
> if(true)
> i++;

Does this convention cover how braces should be used, too?
I usually would write

if (true) {
     i++;
}

But I notice that code I supplied was altered -> someone doesn't seem to
like single lines in braces.

Phil

   ===+===+===+===+===+===+===+===+===+===+===+===
                    Robert E. Hak
            http://phobos.ramapo.edu/~rhak

       Rockbox: The Archos Firmware Replacement!
         FAQ: http://rockbox.haxx.se/docs/FAQ
   ===+===+===+===+===+===+===+===+===+===+===+===

"God is obsolete."
        - !!!SRINI : http://www.unamerican.com

• Next message: Claudio Sabia: "1 Bug, 1 misbehaviour and a feature"
• Previous message: Markus Braun: "[PATCH] scrollbar for tree and menu on recorders"
• In reply to: perterm: "Coding conventions"
• Next in thread: perterm: "Re: Coding conventions"
• Reply: perterm: "Re: Coding conventions"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ] [ attachment ]