[so] [Tema 2] Nelamurire depunctari

[Emma Mirică]

Buna Mihai,

Iata raspunsurile mele inline.

2015-04-24 20:04 GMT+03:00 Mihai Volmer via so <so at cursuri.cs.pub.ro>:

> Salut,
>
> Am observat ca au fost puse rezultatele la tema 2 si am cateva intrebari
> in legatura cu urmatoarele depunctari pentru tema de pe Linux:
>
> -0.1 tab-uri amestecate cu spații pentru indentare; folosiți fie numai
> spații, fie numai tab-uri
> Eu am scris tema in vim si am tab-ul setat sa imi introduca 4 spatii in
> loc de tab, deci nu aveam cum sa amestec tab-uri cu spatii. Singura sursa
> de tab-uri ar putea fi scheletul de cod (pe care nu l-am modificat).
>

In tot codul apar si taburi si spatii la indentare. Problema este ca ai
luat scheletul de cod care era scris cu taburi (fara expand si are
dimensiunea de 8 spatii), iar ce ai adaugat tu este scris cu spatii, iar
tabul este setat la 4 spatii. Chiar daca primiti schelet de cod trebuie sa
mentineti coding style-ul sau sa aranjati tot codul dupa coding style cu
care va simtiti mai confortabil.
Depunctare aceasta se mentine.


>
> -0.1 indentare inconsecventă (uneori există, alteori nu, un TAB, mai mult
> TAB-uri)
> Aici nu am ce explicatie sa dau (in afara de optiunea auto-indent din vim
> pe care am folosit-o excesiv). As aprecia daca cineva ar putea sa
> reverifice aceasta depunctare.
>

Din nou apare o indentare inconsecventa. Se vede in parse_command si nu
numai. Probabil din aceeasi cauza, ca nu ai indentat tot codul, inclusiv
cel primit. Din nou depunctarea se mentine. Ca sa vedeti mai usor toate
aceste probleme folositi in vim "set list listchars=". De asemenea, sugerez
folosirea comenzii indent.


>
> -0.2 cod comentat
> Am avut o linie de cod comentata in toata tema.
>

Niciodata nu se accepta patch-uri cu cod comentat. Ai cod comentat in
utils-lin.c:315 si utils-lin.c:355. Depunctarea ramane.


>
> -0.1 folosire alocare dinamică pentru o dimensiune statică (cunoscută la
> compilare) (alocare vector pentru pipe, pentru pids)
> Am alocat astfel:
> int (*pipes)[2] = malloc(2 * sizeof(int) * length);
> pid_t *pids = malloc((length + 1) * sizeof(pid_t));
> Variabila length este calculata chiar inainte de aceste alocari in functie
> de numarul de pipe-uri din comanda, deci nu este cunoscuta la compilare.
>

Aici este greseala mea. Voi elimina aceasta depunctare.


>
> De asemenea, as avea aceleasi intrebari in legatura cu depunctarile pentru
> tab-uri / indentare inconsecventa / trailing whitespaces si pentru tema de
> pe Windows.
>

Acelasi lucru si pe Windows: ai taburi amestecate cu spatii, ai trailing
whitespaces, ai cod comentat (pe care nu l-am depunctat), indentarea este
inconsecventa. Depunctarile raman.


Multumesc,
Emma
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://cursuri.cs.pub.ro/pipermail/so/attachments/20150425/3b45a15f/attachment-0001.html>