<div><div dir="auto">Grozav!</div></div><div dir="auto"><br></div><div dir="auto">Multumesc frumos!</div><div dir="auto"><br></div><div dir="auto">Numai bine,</div><div dir="auto">Alex Mihai</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 10 Mar 2019 at 14:59, Mihai Barbulescu <<a href="mailto:b12mihai@gmail.com">b12mihai@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Salut Alex,<br>
<br>
Din ce descrii tu aici sunt doua probleme:<br>
1. Daca folosesti valorile "stas" din errno.h: EINVAL, ENOMEM etc.<br>
atunci <a href="http://checkpatch.pl" rel="noreferrer" target="_blank">checkpatch.pl</a> o sa se planga ca vrea sa fie valoarea intoarsa<br>
negativa (adica -ENOMEM in loc de ENOMEM)<br>
2. Testul cu "faulty" malloc se asteapta sa introci codul de eroare 12<br>
__valoare pozitiva__ (deci din multimea N*) sub ce forma vrei tu: ori<br>
exit(12) ori return 12 - oricare din abordari e buna si acceptat si<br>
modul in care imi returnezi tu valoarea 12 pozitiva nu prea imi pasa,<br>
atat timp cat nu o folosesti ca valoarea hardcodata ci printr-un macro<br>
sau un const int.<br>
<br>
On Sun, 10 Mar 2019 at 13:40, Alex Cosmin Mihai via so<br>
<<a href="mailto:so@cursuri.cs.pub.ro" target="_blank">so@cursuri.cs.pub.ro</a>> wrote:<br>
> Checker-ul de coding style imi spunea sa returnez coduri de eroare negative de tipul "-ERRORNOMEM", iar in enunt spune ca valoarea returnata in cazul unei erori de malloc(), de exemplu, trebuie sa fie 12. Din aceasta cauza am schimbat codul sa returnez -ERRORNOMEM, dar am schimbat si definitia ERRORNOMEM in -12. Este corecta aceasta abordare?<br>
><br>
<br>
Pare a fi ok.<br>
<br>
> De asemenea, nu am folosit deloc functia DIE din laboratoare si nici nu am afisat nimic la STDOUT, nici STDERR in afara de output-ul comenzii top, iar erorile le-am propragat ca int-uri valori de return ale functiilor pana in functia main, unde in cazul in care o astfel de valoare este diferita de 0 o folosesc ca parametru pentru exit(). Este aceasta abordare corecta?<br>
<br>
Da e foarte buna abordarea asta.<br>
<br>
><br>
> As fi recunoscator daca cineva din echipa ar putea arunca un ochi peste codul meu care este incarcat si pe vmchecker si pe GitLab si sa-mi dea un ok / not ok.<br>
<br>
N-am apucat sa ma uit pe surse (poate s-o fac diseara cand am niste ragaz)<br>
<br>
-- <br>
Cu stimă,<br>
Mihai Bărbulescu<br>
</blockquote></div></div>