QuentinPerez / 42-toolkit

:seedling: Useful structs written in C
GNU General Public License v3.0
108 stars 26 forks source link

[NORME] Should we to respect the standard of 42? #4

Closed IohannRabeson closed 11 years ago

IohannRabeson commented 11 years ago

If yes, it would be necessary to change all conditional structures which are not surrounded by braces, even if it contains only one line.

Something as this:

if (prev_cell != NULL)
    prev_cell->v_next = cell;
else
    v_this->v_begin = cell;

Would become:

if (prev_cell != NULL)
{
    prev_cell->v_next = cell;
}
else
{
    v_this->v_begin = cell;
}

In my previous pull requests, I have make as you but since the doubt lives in me.

QuentinPerez commented 11 years ago

In the last version we can

IohannRabeson commented 11 years ago

Sweet! Can you send me a link to the updated standard? Or better, place a link to it into the coding guideline ;)

QuentinPerez commented 11 years ago

Look last commit