<div dir="ltr"><div><div><div>Salut Mihai,<br><br></div>Multumesc mult pentru feedback! <br>Voi remedia chestiunile indicate si o sa revin cu rezultatul folosirii <br>extern const int.<br><br></div>Numai bine,<br></div>Alex Mihai<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 10 Mar 2019 at 20:53, Mihai Barbulescu <<a href="mailto:b12mihai@gmail.com">b12mihai@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Salut Alex,<br>
<br>
Am verificat ce e pe <a href="http://gitlab.cs.pub.ro" rel="noreferrer" target="_blank">gitlab.cs.pub.ro</a>, din punctul meu de vedere e OK<br>
ce vad. Data viitoare daca cereti verificare pe vmchecker la<br>
teste/submisii sa precizati va rog frumos username-ul de LDAP (e.g. in<br>
cazul tau era<br>
alexandru.mihai1708)<br>
<br>
Identific o alta problema la priorityQueue.h -> nu l-ai protejat la<br>
multiple inclusion in mai multe fisiere. Fie o faci cu #pragma once<br>
fie o faci cu ceva de genul:<br>
<br>
#ifndef MYSUPERUBERHEADER_FILENAME_H<br>
#define MYSUPERUBERHEADER_FILENAME_H<br>
<br>
/* semnaturi si macrouri */<br>
<br>
#endif<br>
<br>
unde MYSUPERUBERHEADER_FILENAME_H te asiguri ca il alegi ca macro<br>
suficient de bine cat sa nu fie definit pe undeva.<br>
<br>
Mai sunt curios o chestie si n-am eu timp s-o verific dar poate o faci<br>
tu: in loc de<br>
<br>
#define ERRORNOMEM -12<br>
#define ERRORMEMBOUNDS -13<br>
<br>
sa definesti in .h doua extern const int, dar la valori pozitive, si<br>
in .c-ul lui priorityqueue unde le folosesti le setezi si valorile. Si<br>
sa le folosesti p-alea ca return values - oare se mai plange<br>
<a href="http://checkpatch.pl" rel="noreferrer" target="_blank">checkpatch.pl</a>?<br>
<br>
De asemenea vezi ca nu imi plac denumirile astea prea "generice" de<br>
functii: insert, pop ... Vezi indicatiile generale de teme [1]:  -0.1:<br>
denumire neadecvată a funcțiilor sau variabilelor (do_stuff, my_var) ;<br>
<br>
[1] <a href="https://ocw.cs.pub.ro/courses/so/teme/general" rel="noreferrer" target="_blank">https://ocw.cs.pub.ro/courses/so/teme/general</a><br>
<br>
On Sun, 10 Mar 2019 at 18:40, Alex Cosmin Mihai<br>
<<a href="mailto:alexcosmin.mihai@gmail.com" target="_blank">alexcosmin.mihai@gmail.com</a>> wrote:<br>
><br>
> Grozav!<br>
><br>
> Multumesc frumos!<br>
><br>
> Numai bine,<br>
> Alex Mihai<br>
><br>
> On Sun, 10 Mar 2019 at 14:59, Mihai Barbulescu <<a href="mailto:b12mihai@gmail.com" target="_blank">b12mihai@gmail.com</a>> wrote:<br>
>><br>
>> 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>
<br>
<br>
<br>
-- <br>
Cu stimă,<br>
Mihai Bărbulescu<br>
</blockquote></div>