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,
--
Cyrille de Lambertaddress@hidden
26 bis rue des olivettes
44300 NANTESTé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