Hola:<br><br>He probado el schema con la version 291 y compila y funciona bien, además hemos realizado unos cambios en el codigo porque había un problema al cambia de un modo de uso a otro, ya que en uno el pantilt tenia más reocrrido que en otro.<br>
<br>Adjunto el parche con el código corregido.<br><br>Un saludo.<br><br>Darío Rodríguez.<br><br><div class="gmail_quote">El 26 de febrero de 2009 11:28, Roberto Calvo <span dir="ltr"><<a href="mailto:rocapal@gsyc.es">rocapal@gsyc.es</a>></span> escribió:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
Este tiene mucha mejor pinta :-)<br>
<br>
Ya he hecho el commit del parche y corresponde a la revisión 290 del<br>
subversion.<br>
<br>
Cuando tengas un rato, verifica que desde el svn va todo bien y entonces<br>
si cerramos el ticket :-)<br>
<br>
Gracias Dario!!<br>
<br>
El mar, 24-02-2009 a las 21:36 +0100, Darío Rodríguez escribió:<br>
<div><div></div><div class="Wj3C7c">> Hola:<br>
><br>
> Haber creo que ya he pillado esto de los parches (o eso espero :) ) he<br>
> modificado el código sobre el que estaba ya en el SVN que modifico<br>
> Jose Maria y he sacado el parche que adjunto.<br>
><br>
> He cambiado lo necesario en la funcion iteration, que son las<br>
> condiciones de unos IF y he quitado una comprobación que había antes<br>
> de asignar el valor de la longitud porque creo que no es necesaria ya<br>
> que tenemos un IF para comprobarlo.<br>
><br>
> Bueno haber que tal esta vez :)<br>
><br>
> Un saludo.<br>
><br>
> Darío Rodríguez.<br>
><br>
> El 23 de febrero de 2009 16:14, Roberto Calvo <<a href="mailto:rocapal@gsyc.es">rocapal@gsyc.es</a>><br>
> escribió:<br>
><br>
> Pues casi :-)<br>
><br>
> Te cuento, tu parche está bien pero he visto un problema.<br>
> Mira:<br>
><br>
> -runFn buttons_run=NULL;<br>
> -stopFn buttons_stop=NULL;<br>
> -runFn acc_run=NULL;<br>
> -stopFn acc_stop=NULL;<br>
> +runFn buttons_resume;<br>
> +stopFn buttons_suspend;<br>
> +runFn acc_resume;<br>
> +stopFn acc_suspend;<br>
><br>
><br>
> Creo que el parche está generado después de que Jose Maria<br>
> actualizara<br>
> cosas del api. Con este parche estamos quitando los "run" y<br>
> "stop" que<br>
> son nombres del nuevo API por los "resumen" , "suspend" que<br>
> son del<br>
> antiguo api.<br>
><br>
> Creo que se podría solucionar si haces un merge con tu codigo<br>
> y luego el<br>
> svn diff. Lo podemos ver si andas por aquí.<br>
><br>
> Y no te preocupes porque tardemos 4 intentos en tener un<br>
> parche para<br>
> hacer el commit. Mejor eso, a que hubieramos sobre-escrito<br>
> cambios de<br>
> Jose Maria del cambio de api's :-)<br>
><br>
> un saludete!<br>
><br>
> El lun, 23-02-2009 a las 12:17 +0100, Darío Rodríguez<br>
> escribió:<br>
><br>
> > Espero que este sea el bueno :)<br>
> ><br>
> > He puesto los cambios y ha compilado sin ningún problema.<br>
> ><br>
> > Si ves algo raro o que se deba cambiar me lo dices para<br>
> dejarlo<br>
> > cerrado cuanto antes y que entre en la 4.3<br>
> ><br>
> > Un saludo.<br>
> ><br>
> > Darío Rodríguez.<br>
> ><br>
> ><br>
> > El 22 de febrero de 2009 18:23, Roberto Calvo<br>
> <<a href="mailto:rocapal@gsyc.es">rocapal@gsyc.es</a>><br>
> > escribió:<br>
> > El dom, 22-02-2009 a las 12:04 +0100, Darío<br>
> Rodríguez<br>
> > escribió:<br>
> > > Hola Roberto:<br>
> > ><br>
> > > Lo primero lo siento por cerrar el ticket tan<br>
> pronto, la<br>
> > emoción :)<br>
> ><br>
> ><br>
> > Nada! Ni te preocupes! Entre todos estamos creando<br>
> la mejor<br>
> > manera de<br>
> > crear software libre de calidad :-)<br>
> ><br>
> > ><br>
> > > Bueno, lo de los includes paso porque teniamos<br>
> problemas de<br>
> > > compilación y no sabiamos muy bien por donde<br>
> corregirlos y<br>
> > pensamos<br>
> > > que el error podía ser que algunos de ellos<br>
> estaban puestos<br>
> > con <> y<br>
> > > debían ir con "", con la diferenciación que has<br>
> hecho tú<br>
> > comprendo que<br>
> > > deben ir con <> por que no van a estar en el mismo<br>
> > directorio.<br>
> ><br>
> ><br>
> > Ok. Prueba a ponerlos con <> y si te falla al<br>
> compilar lo<br>
> > mandas a la<br>
> > lista e intentamos ver porqué pasa.<br>
> ><br>
> > ><br>
> > > Sobre la segunda duda fue un poco por el mismo<br>
> motivo<br>
> > teniamos un<br>
> > > error a la hora de compilar directamente sobre la<br>
> estructura<br>
> > del SVN y<br>
> > > no entedía porque y probe a quitar el NULL y se me<br>
> olvido<br>
> > ponerlo de<br>
> > > nuevo. Las buenas son así:<br>
> > ><br>
> > > runFn motors_run=NULL;<br>
> > > stopFn motors_stop=NULL;<br>
> > ><br>
> > > Como las demas inicializaciones.<br>
> ><br>
> ><br>
> > Perfect, pues añadelas y si el problema anterior se<br>
> soluciona<br>
> > con <><br>
> > generas de nuevo el parche y listo :-)<br>
> ><br>
> > ><br>
> > > Lo siento por los fallos tontos, si tienes alguna<br>
> duda más<br>
> > dimelo y<br>
> > > así me quedan más claras las cosas para otra vez.<br>
> Gracias<br>
> > por las<br>
> > > explicaciones.<br>
> ><br>
> ><br>
> > Nada!! que lo sienta el que no envía parches, pero<br>
> no tú!<br>
> ><br>
> > No he visto ningún fallo más. Normalmente antes de<br>
> hacer un<br>
> > commit de un<br>
> > parche, suelo echar un ojo al código sin entrar<br>
> estrictamente<br>
> > en qué<br>
> > hace. Entonces cosas como lo del include, o la<br>
> inicialización<br>
> > a NULL,<br>
> > pues salta a la vista.<br>
> ><br>
> > un saludete!!<br>
> ><br>
> > ><br>
> > > Un saludo.<br>
> > ><br>
> > > Darío Rodríguez.<br>
> > ><br>
> > > El 21 de febrero de 2009 14:55, Roberto Calvo<br>
> > <<a href="mailto:rocapal@gsyc.es">rocapal@gsyc.es</a>><br>
> > > escribió:<br>
> > ><br>
> > > Buenas Dario!<br>
> > ><br>
> > > Muchas gracias por el parche! :-)<br>
> > ><br>
> > > Sólo una cosa, no cierres el ticket tan<br>
> pronto :-).<br>
> > La idea es<br>
> > > que el<br>
> > > ticket se cierra cuando se soluciona el<br>
> problema, y<br>
> > eso pasa<br>
> > > cuando el<br>
> > > parche se sube al repositorio y<br>
> posteriormente se<br>
> > prueba.<br>
> > > Entonces es<br>
> > > cuando tu mismo puedes probarlo y<br>
> verificar que<br>
> > funciona bien,<br>
> > > y<br>
> > > entonces cierras el parche.<br>
> > ><br>
> > > También puede pasar que el parche no esté<br>
> bien o<br>
> > incluso ni<br>
> > > compile (por<br>
> > > algún despiste) y entonces ni si quiera se<br>
> puede<br>
> > subir al svn,<br>
> > > por lo<br>
> > > que cerrar el ticket ahora es algo<br>
> prematuro.<br>
> > ><br>
> > > Sobre el parche, te comento 2 cosillas:<br>
> > ><br>
> > > * No entiendo muy bien estos cambios.<br>
> ¿Cuales el<br>
> > motivo de<br>
> > > usar " " en<br>
> > > vez de <>? Las comillas "" lo único que<br>
> hacen es<br>
> > buscar en el<br>
> > > directorio<br>
> > > actual de compilación antes de ir al del<br>
> sistema, y<br>
> > ninguna de<br>
> > > esas<br>
> > > cabeceras van a estar en el directorio<br>
> actual de<br>
> > compilación<br>
> > > del<br>
> > > esquema. Usar una u otra en este caso, da<br>
> igual,<br>
> > funcionaría<br>
> > > bien, pero<br>
> > > es interesante saber si el cambio viene<br>
> por algún<br>
> > otro bug o<br>
> > > problema.<br>
> > ><br>
> > > -#include <jde.h><br>
> > > +#include "jde.h"<br>
> > > -#include <graphics_gtk.h><br>
> > > +#include "graphics_gtk.h"<br>
> > > -#include <wiimote.h><br>
> > > +#include "wiimote.h"<br>
> > ><br>
> > ><br>
> > > * En el siguiente cambio que has hecho, no<br>
> los<br>
> > inicializas a<br>
> > > NULL por<br>
> > > algún motivo?? Es buena práctica<br>
> inicializar todo a<br>
> > NULL,<br>
> > > además ya<br>
> > > estaba puesto en la versión anterior del<br>
> código.<br>
> > ¿viste algún<br>
> > > problema<br>
> > > con esto?<br>
> > ><br>
> > > -runFn motors_run=NULL;<br>
> > > -stopFn motors_stop=NULL;<br>
> > > +runFn motors_resume;<br>
> > > +stopFn motors_suspend;<br>
> > ><br>
> > ><br>
> > > He reabierto el ticket, y en cuanto<br>
> quedamos en algo<br>
> > sobre los<br>
> > > 2 puntos<br>
> > > que te he comentado arriba, subimos el<br>
> parche al svn<br>
> > y después<br>
> > > de<br>
> > > comprobar que funciona bien el esquema,<br>
> cerramos<br>
> > ticket :-)<br>
> > ><br>
> > ><br>
> > > un saludete!!<br>
> > ><br>
> > > El sáb, 21-02-2009 a las 10:12 +0100,<br>
> Darío<br>
> > Rodríguez<br>
> > > escribió:<br>
> > ><br>
> > > > Hola a todos:<br>
> > > ><br>
> > > > El viernes estuve probando el esquema<br>
> wiioperator<br>
> > en el<br>
> > > bicho y parece<br>
> > > > que ya tira bien, hemos corregido lo de<br>
> los<br>
> > límites y ya no<br>
> > > se queda<br>
> > > > pillado en ninguno de los dos modos.<br>
> > > ><br>
> > > > Adjunto el diff para que pongais la<br>
> última versión<br>
> > y pongo<br>
> > > el tikect<br>
> > > > como fixed.<br>
> > > ><br>
> > > > Un saludo.<br>
> > > ><br>
> > > > Darío Rodríguez.<br>
> > ><br>
> > > ><br>
> _______________________________________________<br>
> > > > Jde-developers mailing list<br>
> > > > <a href="mailto:Jde-developers@gsyc.es">Jde-developers@gsyc.es</a><br>
> > > ><br>
> > ><br>
> ><br>
> <a href="http://gsyc.escet.urjc.es/cgi-bin/mailman/listinfo/jde-developers" target="_blank">http://gsyc.escet.urjc.es/cgi-bin/mailman/listinfo/jde-developers</a><br>
> > > --<br>
> > > Roberto Calvo Palomino | Libre<br>
> Software<br>
> > Engineering<br>
> > > Lab (GSyC)<br>
> > > Tel: (+34) 91 488 85 23 |<br>
> Universidad Rey<br>
> > Juan Carlos<br>
> > > Tel: (+34) 91 488 81 05 | Edif.<br>
> > Departamental II -<br>
> > > Despacho 116<br>
> > > <a href="mailto:rocapal@gsyc.es">rocapal@gsyc.es</a> |<br>
> c/Tulipán s/n<br>
> > 28933 Móstoles<br>
> > > (Madrid)<br>
> > > <a href="http://libresoft.es/" target="_blank">http://libresoft.es/</a><br>
> > ><br>
> > > GPG-KEY:<br>
> <a href="http://gsyc.es/%7Erocapal/rocapal.gpg" target="_blank">http://gsyc.es/~rocapal/rocapal.gpg</a><br>
> > ><br>
> ><br>
> > --<br>
> ><br>
> > Roberto Calvo Palomino | Libre Software<br>
> Engineering<br>
> > Lab (GSyC)<br>
> > Tel: (+34) 91 488 85 23 | Universidad Rey<br>
> Juan Carlos<br>
> > Tel: (+34) 91 488 81 05 | Edif.<br>
> Departamental II -<br>
> > Despacho 116<br>
> > <a href="mailto:rocapal@gsyc.es">rocapal@gsyc.es</a> | c/Tulipán s/n<br>
> 28933 Móstoles<br>
> > (Madrid)<br>
> > <a href="http://libresoft.es/" target="_blank">http://libresoft.es/</a><br>
> ><br>
> > GPG-KEY: <a href="http://gsyc.es/%7Erocapal/rocapal.gpg" target="_blank">http://gsyc.es/~rocapal/rocapal.gpg</a><br>
> ><br>
> ><br>
><br>
> --<br>
><br>
> Roberto Calvo Palomino | Libre Software Engineering<br>
> Lab (GSyC)<br>
> Tel: (+34) 91 488 85 23 | Universidad Rey Juan Carlos<br>
> Tel: (+34) 91 488 81 05 | Edif. Departamental II -<br>
> Despacho 116<br>
> <a href="mailto:rocapal@gsyc.es">rocapal@gsyc.es</a> | c/Tulipán s/n 28933 Móstoles<br>
> (Madrid)<br>
> <a href="http://libresoft.es/" target="_blank">http://libresoft.es/</a><br>
><br>
> GPG-KEY: <a href="http://gsyc.es/%7Erocapal/rocapal.gpg" target="_blank">http://gsyc.es/~rocapal/rocapal.gpg</a><br>
><br>
><br>
</div></div>--<br>
<div><div></div><div class="Wj3C7c">Roberto Calvo Palomino | Libre Software Engineering Lab (GSyC)<br>
Tel: (+34) 91 488 85 23 | Universidad Rey Juan Carlos<br>
Tel: (+34) 91 488 81 05 | Edif. Departamental II - Despacho 116<br>
<a href="mailto:rocapal@gsyc.es">rocapal@gsyc.es</a> | c/Tulipán s/n 28933 Móstoles (Madrid)<br>
<a href="http://libresoft.es/" target="_blank">http://libresoft.es/</a><br>
<br>
GPG-KEY: <a href="http://gsyc.es/%7Erocapal/rocapal.gpg" target="_blank">http://gsyc.es/~rocapal/rocapal.gpg</a><br>
</div></div></blockquote></div><br>