[Jde-dev] [JDE-dev] Ticket #215

[Roberto Calvo]

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
------------ 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/20090221/f637e815/attachment.pgp