[Gvsig_desarrolladores] Cambio 32111

[Francisco José Peñarrubia]

Hola Fernando.

Cuando puedas, mira el bug 776. Lo he estado mirando, y al probar con un 
DGN he visto que no salían bien los textos, y eso es grave porque hay 
muchos dgn del IGN circulando por ahí (las hojas MTN, etc).
Al final lo que he hecho es pintar sin centrar el texto en el punto, y 
para solucionar lo del símbolo con background, desplazar el rectángulo 
internamente.

Prueba a ver si con esos cambios tu caso de uso está ok, y lo pones en 
el ticket.

Saludos.

Fran.

El 02/07/2012 10:03, Fernando González escribió:
> 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
> _______________________________________________
> 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