yop voila, j'ai commite ton patch...
je pense qu'on peut commiter dans le main maintenant... a++ tof Olivier MATZ a écrit : >Salut tof, > >J'ai commencé à regarder. Est-ce que ça dérange si je refais >l'indentation de certains fichiers ? Je pense qu'il faut faire attention >à l'indentation, car d'elle dépend la lisibilité du code. Surtout que >les éditeurs corrects le font très bien sans prise de tête. > >Pour ma part, j'ai une préference pour une configuration avec : > - une tabulation = 8 espaces (en taille), et > - une indentation = une tabulation. > >Dans l'idéal, tout le monde a la même configuration. En pratique, je >pense qu'il faut au moins que l'indentation soit cohérente dans un fichier. > >Donc, l'idée c'est que j'ai fait mes modifs, et je t'envoie un patch que >tu peux appliquer dans ta branche: > >$ cd aversive_b_tof >$ patch -p 1 < aversive_indentation_fix.diff > >Si tu es ok avec, tu ls commites dans ta branche. Je ferai alors mon >merge depuis cette version de ta branche. > >Modifs incluse dans le patch (à part l'indentation): > - */brushless.c : légère modification des #if pour ne pas perturber >l'indentation automatique > > >A faire aussi dans ta branche : > - supprimer >modules/devices/brushless_motors/brushless_3phase_digital_hall/test/utils_config.h > - supprimer >modules/devices/brushless_motors/brushless_3phase_digital_hall_double/test/utils_config.h > - supprimer modules/devices/control_system/filters/biquad/test/autoconf.h > - supprimer >modules/devices/control_system/filters/quadramp_derivate/test/autoconf.h > - supprimer >modules/devices/control_system/filters/quadramp_derivate/test/utils_config.h > > > >Remarques : > - Je suppose qu'il n'est pas possible de faire marcher >brushless_3phase_digital_hall et brushless_3phase_digital_hall_double en >même temps. Ceci au moins pour la raison suivante: les fichiers >s'appellent brushless.c tous les 2 dans 2 modules différents. C'est >surement un bug du framework. Cela dit, on doit pouvoir l'interdire dans >le config.in > - la suppression des locks dans quadramp m'interpelle un peu quand >même. Tu abordes le sujet dans ton mail "asserv : plus de coherence" du >11/11/06. Le pb c'est que sans les locks, je pense que les paramètres >peuvent être modifiés pendant qu'ils sont lus. > > >Voila, j'espère que je n'ai rien oublié. > >Olivier > > _______________________________________________ Avr-list mailing list Avr-list@droids-corp.org CVSWEB : http://cvsweb.droids-corp.org/cgi-bin/viewcvs.cgi/aversive WIKI : http://wiki.droids-corp.org/index.php/Aversive DOXYGEN : http://zer0.droids-corp.org/doxygen_aversive/html/ BUGZILLA : http://bugzilla.droids-corp.org COMMIT LOGS : http://zer0.droids-corp.org/aversive_commitlog
