<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2015-04-25 10:48 GMT+03:00 Mihai Volmer <span dir="ltr"><<a href="mailto:mihaivolmer@gmail.com" target="_blank">mihaivolmer@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class=""><div class="h5">2015-04-25 0:36 GMT+03:00 Emma Mirică <<a href="mailto:emma.mirica@gmail.com">emma.mirica@gmail.com</a>>:<br>
><br>
> Buna Mihai,<br>
><br>
> Iata raspunsurile mele inline.<br>
><br>
> 2015-04-24 20:04 GMT+03:00 Mihai Volmer via so <<a href="mailto:so@cursuri.cs.pub.ro">so@cursuri.cs.pub.ro</a>>:<br>
>><br>
>> Salut,<br>
>><br>
>> Am observat ca au fost puse rezultatele la tema 2 si am cateva intrebari in legatura cu urmatoarele depunctari pentru tema de pe Linux:<br>
>><br>
>> -0.1 tab-uri amestecate cu spații pentru indentare; folosiți fie numai spații, fie numai tab-uri<br>
>> 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).<br>
><br>
><br>
> 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.<br>
> Depunctare aceasta se mentine.<br>
><br>
>><br>
>><br>
>> -0.1 indentare inconsecventă (uneori există, alteori nu, un TAB, mai mult TAB-uri)<br>
>> 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.<br>
><br>
><br>
> 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.<br>
><br>
>><br>
>><br>
>> -0.2 cod comentat<br>
>> Am avut o linie de cod comentata in toata tema.<br>
><br>
><br>
> Niciodata nu se accepta patch-uri cu cod comentat. Ai cod comentat in utils-lin.c:315 si utils-lin.c:355. Depunctarea ramane.<br>
><br>
>><br>
>><br>
>> -0.1 folosire alocare dinamică pentru o dimensiune statică (cunoscută la compilare) (alocare vector pentru pipe, pentru pids)<br>
>> Am alocat astfel:<br>
>> int (*pipes)[2] = malloc(2 * sizeof(int) * length);<br>
>> pid_t *pids = malloc((length + 1) * sizeof(pid_t));<br>
>> Variabila length este calculata chiar inainte de aceste alocari in functie de numarul de pipe-uri din comanda, deci nu este cunoscuta la compilare.<br>
><br>
><br>
> Aici este greseala mea. Voi elimina aceasta depunctare.<br>
><br>
>><br>
>><br>
>> De asemenea, as avea aceleasi intrebari in legatura cu depunctarile pentru tab-uri / indentare inconsecventa / trailing whitespaces si pentru tema de pe Windows.<br>
><br>
><br>
> 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.<br>
><br>
><br>
> Multumesc,<br>
> Emma<br>
><br>
<br>
</div></div>Buna Emma,<br>
<br>
Multumesc pentru raspunsul rapid.<br>
<br>
In legatura cu tab-urile: nu am acordat mare atentie tab-urilor din<br>
scheletul de cod deoarece am considerat ca toate aspectele legate de<br>
coding style sunt legate de codul pe care l-am scris eu, nu si de<br>
schelet.<br>
In momentul in care am scris tema nu am observat ca scheletul de cod<br>
are tab-uri in loc de spatii (o sa fiu mai atent data viitoare).<br>
In legatura cu indentarea: aceasta problema apare strict din cauza<br>
amestecarii tab-urilor cu spatii. Chiar si asa, eu, avand tab-ul setat<br>
la 4 spatii, nu am observat probleme de indentare. Consider ca nu este<br>
justificat sa fiu depunctat deoarece nu am acelasi tabwidth cu cel al<br>
persoanei care corecteaza tema.<br></blockquote><div><br></div><div>Buna Mihai,<br><br></div><div>Nu este problema ca nu ai acelasi tabwidth cu cel care corecteaza tema, ci nu aveai acelasi tabwidth cu cel din codul existent.<br></div><div>Datorita acestor probleme, am adaugat un exemplu de vimrc la [0]. Puteti lua acel vimrc si sa-l puneti in ~/.vimrc. Oricum, ma bucur sa aflu ca folositi vim.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
In legatura cu liniile de cod comentate: este vreo diferenta de<br>
depunctare intre blocuri intregi de cod comentat lasate din lene si 2<br>
linii de cod comentat lasate din neatentie? Inainte sa trimit tema am<br>
inspectat codul si am sters toate liniile comentate, dar pe aceastea<br>
nu le-am observat. Nu consider ca este chiar atat de grav incat sa fiu<br>
depunctat dublu.<br>
<br></blockquote><div><br></div><div>Ok. Voi aplica o depunctare de -0.1 pentru cod comentat.<br><br></div><div>Emma<br></div></div><br>[0] <a href="http://ocw.cs.pub.ro/courses/so/teme/general?&#recomandari">http://ocw.cs.pub.ro/courses/so/teme/general?&#recomandari</a><br></div></div>