-----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?

- - 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

Reply via email to