The rule should be the opossite.
Every code that is not required must not exists. If some code
is here in advance for a future feature, it's the job of the
developer adding such feature to add comment to justify why
useless code is added to prevent it to be removed and to
explain to quality guys (guys that tracks dead code) why such
a code and why it must be kept.
When a code is useless it can be commented if we are not sure
it is used by dolibarr but if we are sure it is not (no
reference to a function for example), there is no reason to be
kept.
And no, the code of other is not the code of other: The code
of other is the code of nobody or everybody (We are an
opensource project).
Le 25/05/2012 22:00, Cyrille de Lambert a écrit :
Bof !!!
Le 25/05/2012 21:56, Régis Houssin a écrit :
Règle N°1 : le code des autres c'est le code des autres ! ;-)
j'aimerais bien qu'on demande et/ou qu'on mette en commentaire le code
au lieu de le supprimer. Car si un code est présent c'est qu'il devait
servir.
Je me retrouve avec un module bancale et je n'arrive pas à retrouver le
pourquoi du comment...
Bon week end ;-)
Cordialement,
--
<mime-attachment.jpg>
Cyrille de Lambert |
address@hidden
26 bis rue des olivettes
44300 NANTES |
Tél : +33 (0) 2 51 13
50 12
Mobile :+33 (0) 6 29 41 81 22
Fax : +33 (0) 2 51 13 52 88
http://www.auguria.net |
_______________________________________________
Dolibarr-dev mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
--
Eldy (Laurent Destailleur).
---------------------------------------------------------------
EMail: address@hidden
Web: http://www.destailleur.fr
Dolibarr (Project leader): http://www.dolibarr.org
To make a donation for Dolibarr project via Paypal: address@hidden
AWStats (Author) : http://awstats.sourceforge.net
To make a donation for AWStats project via Paypal: address@hidden
AWBot (Author) : http://awbot.sourceforge.net
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net