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