Ticket #2768 (closed feature: fixed)
Only call dashStyle when strokeDashstyle is set
| Reported by: | ahocevar | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 2.10 Release |
| Component: | Renderer | Version: | 2.9 |
| Keywords: | Cc: | ||
| State: | Complete |
Description
Currently, with every feature that has a stroke symbolizer set, the dashStyle method will be called to determine the dash style. This is not only relatively expensive, it also requires the strokeDashstyle to have a non-null default in the new Symbolizer classes.
The attached patch removes strokeDasharray from the renderer's defaultSymbolizer, adds an additional check in the renderers to only call dashStyle when the strokeDashstyle symbolizer property is set, and sets the strokeDashstyle default to null in the new Symbolizer classes.
Tests still pass, thanks for any review.
Attachments
Change History
Note: See
TracTickets for help on using
tickets.

