<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META content="text/html; charset=utf-8" http-equiv=Content-Type>
<META name=GENERATOR content="MSHTML 10.00.9200.16635">
<STYLE></STYLE>
</HEAD>
<BODY bgColor=#ffffff>
<DIV><FONT size=2 face=Arial>Hi Adrian,</FONT></DIV>
<DIV><FONT size=2 face=Arial></FONT> </DIV>
<DIV><FONT size=2 face=Arial>i tested you first version, it works.</FONT></DIV>
<DIV><FONT size=2 face=Arial>After updating to this modified version i am facing
a problem.</FONT></DIV>
<DIV><FONT size=2 face=Arial></FONT> </DIV>
<DIV><FONT size=2 face=Arial>In TextureLayer::addGroundOverlays(...) i allways
get NULL for this cast:</FONT></DIV>
<DIV><FONT color=#0000ff size=2><FONT color=#0000ff size=2>
<P>const</FONT></FONT><FONT size=2> GeoDataGroundOverlay *overlay = </FONT><FONT
color=#0000ff size=2><FONT color=#0000ff size=2>static_cast</FONT></FONT><FONT
size=2><GeoDataGroundOverlay *>(</FONT><FONT color=#0000ff size=2><FONT
color=#0000ff size=2>qvariant_cast</FONT></FONT><FONT size=2><GeoDataObject
*>( index.data( MarblePlacemarkModel::ObjectPointerRole ) ) );</FONT></P>
<P><FONT size=2><FONT face=Arial>The previous casting version
worked.<BR></FONT></FONT><FONT size=2><FONT face=Arial>Like mentioned by dennis
adding :</FONT><FONT size=2></P>
<P><FONT face=Arial>Q_DECLARE_METATYPE( Marble::GeoDataGroundOverlay*
)</FONT></P>
<P></FONT><FONT face=Arial>to GroundOverlay header seems not to be
enough.<BR>Are this cast is working for you ? I tested on
Win7,</FONT></P>
<P><FONT face=Arial>br, konrad</FONT> </P></FONT></DIV>
<BLOCKQUOTE
style="PADDING-LEFT: 5px; MARGIN-LEFT: 5px; BORDER-LEFT: #000000 2px solid; PADDING-RIGHT: 0px; MARGIN-RIGHT: 0px">
<DIV style="FONT: 10pt arial">----- Original Message ----- </DIV>
<DIV
style="BACKGROUND: #e4e4e4; FONT: 10pt arial; font-color: black"><B>From:</B>
<A title=draghici.adrian.b@gmail.com
href="mailto:draghici.adrian.b@gmail.com">Adrian Draghici</A> </DIV>
<DIV style="FONT: 10pt arial"><B>To:</B> <A title=tgridel@free.fr
href="mailto:tgridel@free.fr">Thibaut Gridel</A> ; <A title=tackat@kde.org
href="mailto:tackat@kde.org">Torsten Rahn</A> ; <A
title=bbeschow@cs.tu-berlin.de href="mailto:bbeschow@cs.tu-berlin.de">Bernhard
Beschow</A> ; <A title=earthwings@gentoo.org
href="mailto:earthwings@gentoo.org">Dennis Nienhüser</A> </DIV>
<DIV style="FONT: 10pt arial"><B>Cc:</B> <A title=marble-devel@kde.org
href="mailto:marble-devel@kde.org">Marble</A> </DIV>
<DIV style="FONT: 10pt arial"><B>Sent:</B> Saturday, July 20, 2013 2:02
PM</DIV>
<DIV style="FONT: 10pt arial"><B>Subject:</B> Re: [Marble-devel] Review
Request 111499: GroundOverlay Rendering</DIV>
<DIV><BR></DIV>
<DIV style="FONT-FAMILY: Verdana, Arial, Helvetica, Sans-Serif">
<TABLE
style="BORDER-TOP: #c9c399 1px solid; BORDER-RIGHT: #c9c399 1px solid; BORDER-BOTTOM: #c9c399 1px solid; BORDER-LEFT: #c9c399 1px solid"
cellPadding=8 width="100%" bgColor=#f9f3c9>
<TBODY>
<TR>
<TD>This is an automatically generated e-mail. To reply, visit: <A
href="http://git.reviewboard.kde.org/r/111499/">http://git.reviewboard.kde.org/r/111499/</A>
</TD></TR></TBODY></TABLE><BR>
<TABLE
style="BORDER-TOP: black 1px solid; BORDER-RIGHT: black 1px solid; BACKGROUND-IMAGE: url(http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png); BACKGROUND-REPEAT: repeat-x; BORDER-BOTTOM: black 1px solid; BACKGROUND-POSITION: left top; BORDER-LEFT: black 1px solid"
cellSpacing=0 cellPadding=8 width="100%" bgColor=#fefadf>
<TBODY>
<TR>
<TD>
<DIV>Review request for Marble, Bernhard Beschow, Dennis Nienhüser,
Torsten Rahn, and Thibaut Gridel.</DIV>
<DIV>By Adrian Draghici.</DIV>
<P style="COLOR: grey"><I>Updated July 20, 2013, 12:02 p.m.</I></P>
<H1
style="FONT-SIZE: 10pt; COLOR: #575012; MARGIN-TOP: 1.5em">Changes</H1>
<TABLE
style="BORDER-TOP: #b8b5a0 1px solid; BORDER-RIGHT: #b8b5a0 1px solid; BORDER-BOTTOM: #b8b5a0 1px solid; BORDER-LEFT: #b8b5a0 1px solid"
cellSpacing=0 cellPadding=10 width="100%" bgColor=#ffffff>
<TBODY>
<TR>
<TD><PRE style="WORD-WRAP: break-word; PADDING-BOTTOM: 0px; PADDING-TOP: 0px; PADDING-LEFT: 0px; MARGIN: 0px; PADDING-RIGHT: 0px">This version removes unnecessary cast and doesn't cache QImages anymore, using qLowerBound to search for overlays when adding and removing.
Points left are: fixing the rendering in Mercator projection and taking care not to render on the Atlas Map.</PRE></TD></TR></TBODY></TABLE>
<H1
style="FONT-SIZE: 10pt; COLOR: #575012; MARGIN-TOP: 1.5em">Description
</H1>
<TABLE
style="BORDER-TOP: #b8b5a0 1px solid; BORDER-RIGHT: #b8b5a0 1px solid; BORDER-BOTTOM: #b8b5a0 1px solid; BORDER-LEFT: #b8b5a0 1px solid"
cellSpacing=0 cellPadding=10 width="100%" bgColor=#ffffff>
<TBODY>
<TR>
<TD><PRE style="WORD-WRAP: break-word; PADDING-BOTTOM: 0px; PADDING-TOP: 0px; PADDING-LEFT: 0px; MARGIN: 0px; PADDING-RIGHT: 0px">This patch adds support for rendering GroundOverlays. A model with filtered GroundOverlays is saved in MarbleModel and used in TextureLayer, where the cache of GroundOverlays is handled. The actual rendering takes place in MergedLayerDecorator.</PRE></TD></TR></TBODY></TABLE>
<H1 style="FONT-SIZE: 10pt; COLOR: #575012; MARGIN-TOP: 1.5em">Testing
</H1>
<TABLE
style="BORDER-TOP: #b8b5a0 1px solid; BORDER-RIGHT: #b8b5a0 1px solid; BORDER-BOTTOM: #b8b5a0 1px solid; BORDER-LEFT: #b8b5a0 1px solid"
cellSpacing=0 cellPadding=10 width="100%" bgColor=#ffffff>
<TBODY>
<TR>
<TD><PRE style="WORD-WRAP: break-word; PADDING-BOTTOM: 0px; PADDING-TOP: 0px; PADDING-LEFT: 0px; MARGIN: 0px; PADDING-RIGHT: 0px">Tested with multiple overlapping overlays, different rotation angles.</PRE></TD></TR></TBODY></TABLE>
<H1 style="FONT-SIZE: 10pt; COLOR: #575012; MARGIN-TOP: 1.5em">Diffs</B>
(updated)</H1>
<UL style="PADDING-LEFT: 0px; MARGIN-LEFT: 3em">
<LI>src/lib/MarbleMap.cpp <SPAN style="COLOR: grey">(102ed0d)</SPAN>
<LI>src/lib/MarbleModel.h <SPAN style="COLOR: grey">(44a5f61)</SPAN>
<LI>src/lib/MarbleModel.cpp <SPAN style="COLOR: grey">(3c44f9d)</SPAN>
<LI>src/lib/MergedLayerDecorator.h <SPAN
style="COLOR: grey">(2823cfc)</SPAN>
<LI>src/lib/MergedLayerDecorator.cpp <SPAN
style="COLOR: grey">(6363d8e)</SPAN>
<LI>src/lib/geodata/data/GeoDataGroundOverlay.h <SPAN
style="COLOR: grey">(2e144af)</SPAN>
<LI>src/lib/layers/GeometryLayer.cpp <SPAN
style="COLOR: grey">(20fd6a7)</SPAN>
<LI>src/lib/layers/TextureLayer.h <SPAN
style="COLOR: grey">(1dbd114)</SPAN>
<LI>src/lib/layers/TextureLayer.cpp <SPAN
style="COLOR: grey">(5b9e9c9)</SPAN> </LI></UL>
<P><A style="MARGIN-LEFT: 3em"
href="http://git.reviewboard.kde.org/r/111499/diff/">View
Diff</A></P></TD></TR></TBODY></TABLE></DIV>
<P>
<HR>
<P></P>_______________________________________________<BR>Marble-devel mailing
list<BR>Marble-devel@kde.org<BR>https://mail.kde.org/mailman/listinfo/marble-devel<BR></BLOCKQUOTE></BODY></HTML>