[vmchecker-dev] [code review] main Makefile

[Szekeres Adriana]

--- On Wed, 9/17/08, Lucian Adrian Grijincu <lucian.grijincu at gmail.com> wrote:

> From: Lucian Adrian Grijincu <lucian.grijincu at gmail.com>
> Subject: Re: [vmchecker-dev] [code review] main Makefile
> To: vmchecker-dev at lists.rosedu.org
> Date: Wednesday, September 17, 2008, 3:55 PM
> 2008/9/17 Alexandru Moşoi <brtzsnr at gmail.com>:
> > si code review-ul
> >
> > -Makefile:
> > cred ca poti sa faci regula aia fie un for. for dir in
> a b c; do ... ;
> > done. e mai clar si mai putin error prone.
> 
> era plănuit, dar mi-era lene azi dimineață
> 
> > -commander/Makefile
> > commander: commander.o
> > commander.o: commander.cpp
> > sunt doua reguli implicite. cred ca poti fi sterse.
> > dupa dist: all sunt doua taburi, unul e suficient
> > -vmexecutor/Makefile
> > acelasi lucru cu regulile implicite..
> 
> măcar într-unul din proiectele astea era pusă
> dependență și față de un
> header file și am impresia că nu merge fără două
> reguli implicite. O
> să mă mai uit.
> 
> > si inca un lucru te rog. code reviews inseamna doar
> review, nu mai
> > modifica sursa adrianei, caci poti introduce
> accidental un bug si
> > trebuie ca adriana sa invete ca liniile lungi sunt a
> bad habbit.
> 
> birocrația era prea mare dacă trebuia să-i zic Adrianei
> să-mi adauge
> nuș ce reguli în Makefile-urile din codul ei. Era puțin
> exagerat.
> pe vmexecutor lucrez (sau ar fi trebuit să lucrez) și eu.
> Recunosc că
> la un cod review ar trebui doar să semnalezi problemele,
> dar în timp
> ce browsam prin codul ei am făcut niște modificările
> minuscule:
> 
> -static int append_f(const char *infile, const char
> *outfile, const
> char* message)
> +static int
> +append_f(const char *infile, const char *outfile, const
> char* message)
> 
> și prostia asta de svn nu știe să facă commit doar pe
> anumite fișiere;
> ia un director odată - aici trebuia să sterg copia mea
> locală și să
> dau un svn revert sau ceva de genu ăsta să fac rost de
> fișierul
> original. Too much work for a simple "\n".
> 
> În ultimul commit stiu că "i got carried away"
> ... dar in condițiile
> în care și eu trebuie să lucrez pe codul ăsta nu-i mare
> chestie.
> 

Nu pe codul asta trebuia sa lucrezi :)) . La Commander te-ai trecut pe lista. 
In legatura cu liniile lungi, imi cer scuze. Am tot modificat cate ceva, sper sa se vada macar o mica diferenta. 

> -- 
> Lucian
> _______________________________________________
> vmchecker-dev mailing list
> vmchecker-dev at lists.rosedu.org
> http://lists.rosedu.org/cgi-bin/mailman/listinfo/vmchecker-dev