[Marble-devel] Review Request: Fix painting of polygons with holes

Commit Hook null at kde.org
Wed May 23 20:45:45 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105019/#review14099
-----------------------------------------------------------


This review has been submitted with commit aee75aed399ac169c8dccd16ee300515ec5e24e6 by Dennis Nienhüser to branch master.

- Commit Hook


On May 23, 2012, 7 p.m., Dennis Nienhüser wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105019/
> -----------------------------------------------------------
> 
> (Updated May 23, 2012, 7 p.m.)
> 
> 
> Review request for Marble.
> 
> 
> Description
> -------
> 
> Currently GeoPainter renders Polygons wrong if they have an outline pen and are filled (think of buildings): A QPolygonF is created from the outer boundary and the inner rings are subtracted. Now QPainter paints the stroke of such Polygons by connecting each point, leading to unwanted connections between the outer boundary and the inner holes. See http://i.imgur.com/25rjs.png (from Ander).
> 
> The patch changes this to paint the background (brush) as previous, but paints the outer and inner rings' outlines individually. Please review with performance in mind. Any ideas how to paint everything in one go (should be faster)?
> 
> 
> Diffs
> -----
> 
>   src/lib/GeoPainter.cpp aa18e47 
> 
> Diff: http://git.reviewboard.kde.org/r/105019/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dennis Nienhüser
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20120523/07cc0b49/attachment.html>


More information about the Marble-devel mailing list