[so] [Tema2] Contestatie

[Dan Ungureanu]

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