Salut Dan, Pe Linux la cd in caz de eroare nu afisezi nimic. M-am uitat din nou pe Linux si as putea sa iti acord cele -0.2 puncte pierdute. Desi cd e foarte greu de evaluat in cazul temei tale. Ar fi trebuit poate, din pacate, -0.1 pentru ne-existenta unui mesaj de eroare/imposibilitatea verificarii de functionalitate a redirect-ului cd-ului. Desi asta e doar functionalitate de Bash, nu-s sigur in celelalte shell-uri. Pt ca e prea tipic am decis sa nu iti scad nimic
Pt variabilele hardcoded cam avea dreptate pe windows. Solutia cu macro era cea mai buna, cea propusa de tine, intrucat depunctarea nu se regasea pe Linux am scos-o si de pe Windows Ai 10 pe ambele teme si karma de la mine 2017-05-01 19:07 GMT+03:00 Dan Ungureanu via so <so@cursuri.cs.pub.ro>: > Salut, > > Doresc recorectarea temei 2 intrucat depunctarile mentionate pe > VmChecker nu se aplica in cadrul temei mele: > >> -0.1: mesajele de eroare nu sunt foarte relevante/ajutatoare la localizarea >> problemei > > Majoritatea acestor mesaje de eroare vin sunt emise de macro-ul DIE(). > Acesta afiseaza fisierul si linia care sunt arhisuficiente pentru > localizarea problemei. Cu toate acestea, am incercat sa adaug > comentarii descriptive pentru fiecare aparuta (i.e. "Invalid command." > in cazul in care era specificata o comanda invalida, "Failed waiting > for child.", "Could not reallocate memory.", etc.) > >> -0.1: nu se face redirectare pentru comanda internă cd > > Se face. Unul din teste chiar verifica asta si mai mult, se vede si in > cod (functia "execute_simple", sectiunea descrisa in comentarii ca > "Built-ins", mai exact liniile 191 si 193 ale fisierului cmd.c in > cazul variantei pe Linux, asemanator si pe Windows). > >> -0.1: folosire de valori hard-coded, în loc să se definească macrodefiniții >> (e.g. nr de threaduri) > > Singura valoare hard-coded din intreaga tema este "2" care reprezinta > numarul de capaturi ale unui pipe si numarul de thread-uri necesare > pentru a executa un pipe. Nu cred ca ar fi ajutat daca as fi folosit > doua variabile pentru capetele pipe-ului sau daca as fi avut ceva de > genul int pipefds[MAX_PIPE_ENDS], unde MAX_PIPE_ENDS ar fi 2. > > O zi buna, > Dan Ungureanu > _______________________________________________ > http://ocw.cs.pub.ro/courses/so/info/lista-discutii -- Cu stimă, Mihai Bărbulescu _______________________________________________ http://ocw.cs.pub.ro/courses/so/info/lista-discutii
