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