[ http://jira.codehaus.org/browse/GEOT-2538?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Andrea Aime reopened GEOT-2538: ------------------------------- So, it turns out I messed up and committed and incomplete patch, and thus the problem is still there :( > GEOT-2073 fix intoduced severere performance degradation due to open SUN Bug > ID:6568969 > ---------------------------------------------------------------------------------------- > > Key: GEOT-2538 > URL: http://jira.codehaus.org/browse/GEOT-2538 > Project: GeoTools > Issue Type: Bug > Components: core render, core styling > Environment: WinXP, java 1.6 > Reporter: Per S Hustad > Assignee: Andrea Aime > Fix For: 2.5.6 > > Attachments: patch-stroke-bug.txt, StrokeBugDemo.java > > > The Stroke fix for GEOT-2073 introduced in 2.5.2, may cause severe rendering > degradation. This should be seen in conjunction with [SUN Bug > ID:6568969|http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6568969] > reporting that drawing long dashed lines takes extremely long time. > Drawing very long lines will typically happen when zooming far into a > relatively large feature. > Before the 2073 patch, plain solid SLD strokes were just converted to > {{java.awt.BasicStroke}} using a {{null dash}} array. However, after the > 2073 patch, a dash array of {{[1.0, 0.0]}} is passed to the {{BasicStroke}} > constructor if the original stroke is solid. Running the attached test > program shows the output from drawing a very long text line on my 8 core > Intel Xeon 3.37 Ghz machine with an ATI FireGL V7350 graphics card: > {noformat} > Drawing with dash= [1.0, 0.0] took 14570.986102 ms > Drawing with dash= null took 0.326722 ms > {noformat} > I.e. the difference in performance is immense. Note also there is no > workaround for this. Even if you specify a solid stroke, geotools will > automatically > create a dummy [1,0] stroke for you. > We have created a patch for the 2.5.5 version. The patch substitutes any > {{[1.0, 0.0]}} dash array with a null argument before constructing the > BasicStroke. All existing unit tests in the gt-render module pass with the > patch installed and our application performs extremely much better when > zooming in. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira ------------------------------------------------------------------------------ Crystal Reports - New Free Runtime and 30 Day Trial Check out the new simplified licensing option that enables unlimited royalty-free distribution of the report engine for externally facing server and web deployment. http://p.sf.net/sfu/businessobjects _______________________________________________ Geotools-devel mailing list Geotools-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel