[vmchecker-dev] [code review] vm_ipc_sem_ctl

[Alexandru Moşoi]

lgtm

afara de partea de C++, acum e ok :).

2008/9/6 Alexandru Moşoi <brtzsnr at gmail.com>:
> adopta un coding style :)
>
> fara using namespace std; using std::cerr si etc. anyway... foloseste
> C nu C++ (aici folosesti doar iostream si nu-si are sensul).
> pune toate defineurile la inceputul sursei si lasa mai mult spatiu :)
> cred ca trebuie sa adaugi mai multe comentarii
>
> linia 167: UP == action ? 1 : -1 (spatii, inversare)
> linia 208: } while (0) (spatii, fara ;)
>
> nu mai amesteca spatii cu taburi
>
> liniile 130..: sterge codul comentat
> 182, 183: uneste liniile pentru vizibilitate
> 33-35 -> muta comentariu la inceputul surse si fa-l putin mai normal
>
> FTOCKPREFIX - de unde ai scos valoarea lui - documenteaza? stiu ca ai
> nevoie de el pentru cheie, dar nu poti sa folosesti un fisier din
> VMCHECKER_ROOT/..??
>
> ce este FIRST_ERROR_CODE??? nu-l folosesti nicaieri. scapa de el
>
> inlocuieste comentarii de genul '//asdfadsf' la '// asdfasdf' (spatiu)
>
>
> On Sat, Sep 6, 2008 at 4:46 PM, Lucian Adrian Grijincu
> <lucian.grijincu at gmail.com> wrote:
>> ...
>>
>> --
>> Lucian
>> _______________________________________________
>> vmchecker-dev mailing list
>> vmchecker-dev at lists.rosedu.org
>> http://lists.rosedu.org/cgi-bin/mailman/listinfo/vmchecker-dev
>>
>
>
>
> --
> Alexandru Moşoi
> http://alexandru.mosoi.googlepages.com
>
> Please avoid sending me Word or PowerPoint attachments.
> See http://www.gnu.org/philosophy/no-word-attachments.html
>



-- 
Alexandru Moşoi
http://alexandru.mosoi.googlepages.com

Please avoid sending me Word or PowerPoint attachments.
See http://www.gnu.org/philosophy/no-word-attachments.html