2012/2/22 Toni Hermoso Pulido <toniher@softcatala.cat>
El 20 de febrer de 2012 12:13, Toni Hermoso Pulido <toniher@softcatala.cat> ha escrit:
P.D.: Abans de treure cap nova versió, miro de tenir el codi en un
repositori, perquè tu i altres companys pugueu donar una millor opinió i també remenar-hi.
Hola, ja he pujat el codi a: https://github.com/Softcatala/CatalanitzadorMozilla
Ep, - Quan diem "<div id="Nomes">No em molestes més</div>" podem dir alguna cosa com "No m'ho tornis a preguntar". - Estem llicenciant el paquet sota tres llicències "MPL 1.1/GPL 2.0/LGPL 2.1" però el que codi que incloem de JQuery només té dos. Estem segurs podem rellicenciar sota LGPL 2.1 quan JQuery no té aquesta llicència? Caldria mirar-ho. - Ara mateix tenim codi duplicat per la versió en català i valencià. Això hauria de ser un únic codi font per simplificar el manteniment. - Podem comprovar si algú té anglès de Canada (en-ca) no hi ha cap problema? - Tenim proves unitàries de la lògica per poder fer test de regressió? Es poden fer amb la plataforma Mozilla? - Funcions detchanlang com caldria refactoritzar-les perquè no fossin tant llargues. És un principi molt comú de code clean. Aquest codi: // Check first entry if (clist == 0) { // If first entry is already default lang and second entry is second lang-> Done if ((listlangs[clist] == defaultlang) && (listlangs[clist+1] == secondlang) ) { insdef++; break; } Vols dir que no peta? Si clist és 0 accedir a un element d'un array com ara ((listlangs[clist] i listlangs[clist+1]) no produeix una excepció? Espero sigui útil Atentament, Jordi,