Lionel Debroux (./102) :
Dis-donc, Kevin, [...]
Et le rapport de ce rant avec la Nspire? Je n'étais pas au courant de ton patch, c'est normal, il n'est une fois de plus pas committé. J'ai autre chose à faire aussi que d'aller courir après tes patches (surtout dans un log d'un chan IRC complètement indépendant de la LPG). J'ai discuté du protocole avec Brandon, je l'ai implémenté aussitôt que possible, sachant que je n'ai pas que ça à faire, et que je n'ai aucun moyen de tester le code.
Les 2 bogues devraient être corrigés:
http://svn.calcforge.org/viewvc/calcforgelp?view=revision&revision=4591http://svn.calcforge.org/viewvc/calcforgelp?view=revision&revision=4593Mais pour revenir au sujet, qui, je le rappelle, est la
Nspire:
et qu'elle n'est pas testée (pas plus qu'au moment de leur commit, tes nouvelles fonctions
Comment veux-tu que je teste? Je n'ai pas de Nspire! Je committe ce que j'ai, histoire qu'on ne s'amuse pas à réinventer la roue, à la communauté de tester. J'ai implémenté à la lettre le protocole documenté sur Hackspire qui a déjà servi pour Nspire8x, donc il y a des chances que ça marche du premier coup.
spéciales Nspire
Il n'y a qu'une fonction spéciale Nspire (
calcprotocols_calc_nspire_get_attributes). La fonction
calcprotocols_calc_copy_file est implémentée pour tous les modèles, elle copie à travers le PC pour les modèles autres que la Nspire tout simplement. (Pour la Nspire, la copie native est utilisée.)
non accessibles de l'UI
Ces fonctions ont été rajoutées exprès pour permettre un portage de Ndless, donc forcément je ne vois pas pourquoi elles devraient être obligatoirement présentes dans CalcForgeLP. Surtout
calcprotocols_calc_nspire_get_attributes n'a vraiment strictement rien à faire dans l'interface utilisateurs telle qu'elle se présente actuellement (le dirlist récupère déjà les informations de taille et de type, et pour la date, on ne sait même pas exactement ce que le fameux uint32_t veut dire concrètement).