<div dir="ltr"><div>Buna Mihai,<br><br></div>Iata raspunsurile mele inline.<br><div class="gmail_extra"><br><div class="gmail_quote">2015-04-24 20:04 GMT+03:00 Mihai Volmer via so <span dir="ltr"><<a href="mailto:so@cursuri.cs.pub.ro" target="_blank">so@cursuri.cs.pub.ro</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div>Salut,<br><br></div>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></div></div></div></div></div></blockquote><div><br></div><div>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></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><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></div></div></div></div></div></blockquote><div><br></div><div>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></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><br>-0.2 cod comentat<br>Am avut o linie de cod comentata in toata tema.<br></div></div></div></div></div></blockquote><div><br></div><div>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></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><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></div>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></div></div></div></div></blockquote><div><br></div><div>Aici este greseala mea. Voi elimina aceasta depunctare.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div></div><div><br></div>De asemenea, as avea aceleasi intrebari in legatura cu depunctarile pentru tab-uri / indentare inconsecventa / trailing whitespaces si pentru tema de pe Windows.<br></div></div></div></blockquote><div><br></div><div>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></div><div>Multumesc,<br></div><div>Emma<br></div></div><br></div></div>