[Gvsig_desarrolladores] Cambio 32111

[Fernando González]

2012/7/1 Francisco José Peñarrubia <fpenarru en gmail.com>:
> Hola Fernando.
>
> Pues probablemente poco pelo.... igual que ahora :-p

Jajaja.

Vale, ahora en serio.

Mi problema es que esa clase se usa potencialmente desde mil sitios y
es imposible hacer una evaluación del impacto que puede tener adaptar
esas líneas a mi caso de uso. Por eso te preguntaba si te acordabas
del motivo del cambio.

Veo dos opciones.

 (a) Dejar el código así para siempre

 (b) enviar un patch que corrija mi caso de uso y que incluya un
comentario sobre la motivación de mi cambio y como testearlo, con
referencia al bug en cuestión en el redmine.

En (b), si se arregló algo con ese cambio, mi patch lo volverá a
estropear, pero el siguiente que vuelva a arreglarlo se encontrará mi
comentario y, esperemos, respetará mi cambio, con lo que acabaremos
con un código válido para ambos casos de uso.

Saludos.

>
> Saludos!.
>
>
> Fran.
>
> El 28/06/2012 18:11, Fernando González escribió:
>> Hola, estoy mirando el bug 776[1] y la causa del mismo es el commit
>> 32111 hecho por Fran Peñarrubia. El diff podéis obtenerlo con este
>> comando[2]. La parte significativa es esta[3]. También creo que el
>> código anterior no era del todo correcto, he obtenido algún resultado
>> mejor que revertiendo el commit.
>>
>> Por una parte, depurando he llegado a la conclusión de que esas líneas
>> no eran correctas (para mi caso de uso). Por otra, como el bug no se
>> daba en la versión 1.9 he hecho un diff de SimpleTextSymbol.java desde
>> la revisión 31573 (1.9) hasta HEAD[4] y sólo están cambiadas esas
>> líneas, por lo que se confirma mi sospecha.
>>
>> El problema es que no sé si Fran habrá hecho ese cambio por algún
>> motivo o se le ha colado involuntariamente. El mensaje del commit no
>> ayuda.
>>
>> Fran, ¿¿te acuerdas de qué tenías en la cabeza aquel 16 de febrero de
>> 2010 a las 10:57 de la mañana?? xD xD
>>
>> Saludos!
>>
>> [1] https://devel.gvsig.org/redmine/issues/776
>> [2] svn diff -r32111
>> https://devel.gvsig.org/svn/gvsig-desktop/trunk/libraries/libFMap/src/com/iver/cit/gvsig/fmap/core/symbols/SimpleTextSymbol.java
>> [3]
>> -             g.drawString(getText(), 0, (int)-bounds.getY());
>> +             g.drawString(getText(), -((int) bounds.getWidth()/2), 0);
>> //(int)-bounds.getY());
>> [4] svn diff -r31573:HEAD
>> https://devel.gvsig.org/svn/gvsig-desktop/trunk/libraries/libFMap/src/com/iver/cit/gvsig/fmap/core/symbols/SimpleTextSymbol.java
>> _______________________________________________
>> gvSIG_desarrolladores mailing list
>> gvSIG_desarrolladores en listserv.gva.es
>> Para ver histórico de mensajes, editar sus preferencias de usuario o darse de baja en esta lista, acuda a la siguiente dirección: http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_desarrolladores
>
> --
> Fran Peñarrubia
> Scolab
> www.scolab.es
>
> Asociación gvSIG
> www.gvsig.com
>
> _______________________________________________
> gvSIG_desarrolladores mailing list
> gvSIG_desarrolladores en listserv.gva.es
> Para ver histórico de mensajes, editar sus preferencias de usuario o darse de baja en esta lista, acuda a la siguiente dirección: http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_desarrolladores