It should be that, using const as much as possible for pointers and
mandatory for references would be nice.


On Wed, May 28, 2014 at 7:17 PM, Lioncash <notifications@github.com> wrote:

> In Contributing.md:
>
> > +- When declaring includes in a source file, make sure they follow the given pattern:
> > +  - Standard library headers
> > +  - System-specific headers (these should also likely be in an `#ifdef` block unless the source file itself is system-specific).
> > +  - Dolphin source file headers
> > +- Each of the above header sections should also be in alphabetical order
> > +- Project source file headers should be included in a way that is relative to the `[Dolphin Root]/Source/Core` directory.
> > +- This project uses `#pragma once` as header guards.
> > +
> > +### Loops
> > +- If an infinite loop is required, do not use `for (;;)`, use `while (true)`.
> > +- Empty-bodied loops should use braces after their header, not a semicolon.
> > +  - Yes: `while (condition) {}`
> > +  - No: `while (condition);`
> > +
> > +### Functions
> > +- If a function parameter is not intended to have their assigned value or data changed, please mark that parameter as `const`. Local variables within functions that do not change value should also be marked accordingly as well.
>
> True, I guess this can just be trimmed down so that it applies to only
> functions parameters that are references or pointers. Is that okay? If
> people still don't agree on at least this, then we can just throw it out
> altogether.
>
> cc'ing @delroth <https://github.com/delroth>, @comex<https://github.com/comex>,
> @galop1n <https://github.com/galop1n>
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dolphin-emu/dolphin/pull/410/files#r13143111>
> .
>



-- 
Pierre "delroth" Bourdon <delroth@gmail.com>
Software Engineer @ Zürich, Switzerland
http://code.delroth.net/