[Script] Demande d'avis sur l'Anti pub V 2.1

[CrazyCat]

Tu as des & #92; dans ton code qui devraient être remplacés par des \

Les "set" dans le namespace eval devraient comporter des noms de variable pleinement qualifiés ( set ::antispams::why(advert) .... )

Les 2 foreach servant à arrêter les timers/utimers dans la proc de désinstallation ne te servent à rien

Tiens, je n'avais pas fait attention à cette erreur (due au forum) de & #92;, tellement habitué à la voir
Pour les "set", vu qu'ils sont dans le eval, quel intérêt de qualifier ?

Et concernant les kills de timers, je pense que ce n'est pas génant dans le sens où la désinstallation intervient rarement et qu'on ne sait pas trop comment évoluera le tcl. Là, ça ne sert à rien, mais un jour qui sait...

salut,


Les lignes
217 et 200 ont la même fonction " *viens* / *viens sur #* "
216 et 219 aussi " *http* / *http://* "

La ligne 229 " *irc* " me semble trop restrictive exemple " Salut, je suis nouveau ,qu'est-ce irc ? " ça match

De plus mettre un "antispam" sur le #salon, je trouve ça un peu fort, lors d'une discution, un utilisateur, peut etre amené dire a un autre "Viens sur #aide, un helper vas t'expliquer" et pour le coup sera "matché" .

Cordialement.

Je ne pense pas que le choix des mots-clés soit à discuter mais plutôt à prendre comme exemple. C'est vraiment la partie qui devrait devenir la plus configurable.

Ah, ben voila une critique: il faudrait mettre une liste des mots clés facile à éditer, et que l'init du tcl crée les binds à partir de cette liste.