<br><br><div class="gmail_quote">2012/2/22 Toni Hermoso Pulido <span dir="ltr"><<a href="mailto:toniher@softcatala.cat">toniher@softcatala.cat</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
El 20 de febrer de 2012 12:13, Toni Hermoso Pulido<br>
<<a href="mailto:toniher@softcatala.cat">toniher@softcatala.cat</a>> ha escrit:<br>
<br>
><br>
> 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.<br>
><br>
><br>
<br>
Hola,<br>
ja he pujat el codi a:<br>
<a href="https://github.com/Softcatala/CatalanitzadorMozilla" target="_blank">https://github.com/Softcatala/CatalanitzadorMozilla</a><br><br></blockquote><div><br></div><div>Ep,</div><div><br></div><div>- Quan diem "<div id="Nomes">No em molestes més</div>" podem dir alguna cosa com "No m'ho tornis a preguntar".</div>
<div>- 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.</div>
<div>- Ara mateix tenim codi duplicat per la versió en català i valencià. Això hauria de ser un únic codi font per simplificar el manteniment. </div><div>- Podem comprovar si algú té anglès de Canada (en-ca) no hi ha cap problema?</div>
<div>- Tenim proves unitàries de la lògica per poder fer test de regressió? Es poden fer amb la plataforma Mozilla?</div><div>- Funcions detchanlang com caldria refactoritzar-les perquè no fossin tant llargues. És un principi molt comú de code clean.</div>
<div><br>Aquest codi:</div><div><br></div><div><div>// Check first entry</div><div>      if (clist == 0) {</div><div><br></div><div><span class="Apple-tab-span" style="white-space:pre">   </span>// If first entry is already default lang and second entry is second lang-> Done    </div>
<div><span class="Apple-tab-span" style="white-space:pre">      </span>if ((listlangs[clist] == defaultlang) && (listlangs[clist+1] == secondlang) ) {</div><div><span class="Apple-tab-span" style="white-space:pre">      </span>  insdef++;</div>
<div><span class="Apple-tab-span" style="white-space:pre">      </span>  break;</div><div><span class="Apple-tab-span" style="white-space:pre">     </span>}</div></div><div><br></div><div><br></div><div>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ó?</div>
<div><br></div><div>Espero sigui útil</div><div><br>Atentament,</div><div><br>Jordi,</div><div><br></div><div> </div></div>