[so] [Tema1] test 00

[Emma Mirică]

2015-03-09 21:10 GMT+02:00 Gabriel Samoila via so <so at cursuri.cs.pub.ro>:

> Buna seara,
>
> Care sunt conditiile pe care trebuie sa le treaca sursele noastre pentru a
> trece testul 00) Sources check si ce conditioneaza din punctaj acest test?
>
> Multumesc,
> Gabriel Samoila
>

Buna Gabriel,

Acest test nu are punctaj. Este o rulare a checkpatch.pl, un script care
verifica coding style-ul folosit in kernelul Linux. Pentru a face acest
test sa treaca, va trebui sa elimini warning-urile si erorile pe care le
vezi aici:

./list.c:81: WARNING:BRACES: braces {} are not necessary for single
statement blocks
./main.c:92: WARNING:LINE_SPACING: Missing a blank line after declarations
./main.c:178: WARNING:LONG_LINE: line over 80 characters
./list.h:6: WARNING:NEW_TYPEDEFS: do not add new typedefs
./main.c:17: ERROR:SPACING: space required before the open parenthesis '('
./main.c:10: WARNING:SUSPECT_CODE_INDENT: suspect code indent for
conditional statements (8, 8)
./main.c:29: ERROR:TRAILING_WHITESPACE: trailing whitespace

Este doar o sugestie pentru coding style. Unele se aplica si pentru
aceasta materie (vezi linii peste 80 de caractere), altele nu neaparat
(nu trebuie neaparat sa folositi taburi pentru indentare, dar trebuie
sa fiti consecventi).

Daca vei rezolva warning-urile pe care le-am afisat mai sus (iti spune
exact in ce fisier, la ce linie este problema), acest test va fi
passed.

Emma
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://cursuri.cs.pub.ro/pipermail/so/attachments/20150309/1a38a1fc/attachment.html>