Hi,

Sascha L. Teichmann wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> ... done.
> 
> A very big "thank you" to Larry for supporting me in this. :-)
> 
> - From the point of view of the Print/Layout plug-in this makes it
> a lot easier to produce hi quality output without tampering
> with the size of the LayerViewPanel or other ugly workarounds.
> 
> Of cause on the callers side you cannot assume that the
> set() call is already there [1]. Even if SkyJUMP and OpenJUMP
> features it we have a bunch of other derivates.
> 
> It would be very nice to have it at least in deeJUMP or PirolJUMP
> as well.
> 
> Ugo, Michael (BrĂ¼ning), any comments about this topic?

Not really. I agree with the majority here ;-)

I can only test any changes (in deejump) in the next few weeks.

Cheers,

Ugo

> 
> - - Sascha
> 
> 
> [1] java.lang.reflect is our friend here to check if it exists
>     and call it if it's possible.
> 
> 
> Stefan Steiniger schrieb:
>> ok.. if Larry (from my point of view the expert) agrees, then you can 
>> commit your patch.
>>
>> stefan
>>
>> Sascha L. Teichmann schrieb:
>>> Hallo together,
>>>
>>> It would be of great if the Viewport class [1] would has a
>>> setJava2DConverter(Java2DConverter) method.
>>> Java2DConverter [2] objects are used to convert JTS
>>> geometries into Java2D shapes.
>>>
>>> Why should it be possible to exchange the converter? For the
>>> simple reason that the default one does some coordinate rounding/cutoffs
>>> that make it hard to produce a quality output for printing, SVG export
>>> and so on. We discussed that earlier on the list. If it would be
>>> possible to set a custom (actually a subclass of Java2DConverter)
>>> converter you can temporally replace the original one for a rendering
>>> cycle to get non-truncated vertices. Even more: you can install a
>>> converter that does some controlled simplifications to reduce the
>>> amount of data that is send to the output devices. I've tried both
>>> and it works very well without having a big impact onto the rendering
>>> process as such.
>>>
>>> I've attached a patch to add the setter to Viewport. It would be
>>> really kind if you test it. I don't want to simply let my commit
>>> speak. ;-)
>>>
>>> Thanks in advance,
>>>   Sascha
>>>
>>>
>>> [1] com.vividsolutions.jump.workbench.ui.Viewport
>>> [2] com.vividsolutions.jump.workbench.ui.renderer.java2D.Java2DConverter
>>>
>>>
>>> ------------------------------------------------------------------------
>>>
>>> Index: src/com/vividsolutions/jump/workbench/ui/Viewport.java
>>> ===================================================================
>>> RCS file: 
>>> /cvsroot/jump-pilot/openjump/src/com/vividsolutions/jump/workbench/ui/Viewport.java,v
>>> retrieving revision 1.1
>>> diff -u -r1.1 Viewport.java
>>> --- src/com/vividsolutions/jump/workbench/ui/Viewport.java  16 Jun 2005 
>>> 22:11:47 -0000      1.1
>>> +++ src/com/vividsolutions/jump/workbench/ui/Viewport.java  19 Mar 2007 
>>> 10:45:39 -0000
>>> @@ -97,6 +97,10 @@
>>>          return java2DConverter;
>>>      }
>>>  
>>> +    public void setJava2DConverter(Java2DConverter converter) {
>>> +        java2DConverter = converter;
>>> +    }
>>> +
>>>      public ZoomHistory getZoomHistory() {
>>>          return zoomHistory;
>>>      }
>>>
>>>
>>> ------------------------------------------------------------------------
>>>
>>> -------------------------------------------------------------------------
>>> Take Surveys. Earn Cash. Influence the Future of IT
>>> Join SourceForge.net's Techsay panel and you'll get the chance to share your
>>> opinions on IT & business topics through brief surveys-and earn cash
>>> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
>>>
>>>
>>> ------------------------------------------------------------------------
>>>
>>> _______________________________________________
>>> Jump-pilot-devel mailing list
>>> Jump-pilot-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>> -------------------------------------------------------------------------
>> Take Surveys. Earn Cash. Influence the Future of IT
>> Join SourceForge.net's Techsay panel and you'll get the chance to share your
>> opinions on IT & business topics through brief surveys-and earn cash
>> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
>> _______________________________________________
>> Jump-pilot-devel mailing list
>> Jump-pilot-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.2.2 (GNU/Linux)
> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
> 
> iD8DBQFF/3APsrvOlFf8EzcRAhWTAJsEJn1WSbZhWaksv3/iTYvejmQGhgCeI2so
> iF8PDtj9vJjeFzuG59PKyZA=
> =lTUC
> -----END PGP SIGNATURE-----
> 
> -------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share your
> opinions on IT & business topics through brief surveys-and earn cash
> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> _______________________________________________
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> 

-- 
l a t / l o n  GmbH
Aennchenstrasse 19           53177 Bonn, Germany
phone ++49 +228 184960       fax ++49 +228 1849629
http://www.lat-lon.de        http://www.deegree.org

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to