<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/116059/">https://git.reviewboard.kde.org/r/116059/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">There's a syntax error in the .kml file. The " characters are still escaped, i.e. there's \" where " is needed (copy-paste from C++), so it's not valid XML. Just remove those characters.
xmllint --noout some/file/to/test.xml
is a good way to find such errors.
With that done the test will still not succeed directly. If you use kml2kml and diff the output (after feeding through xmllint --format) you'll get this:
@@ -1,14 +1,14 @@
<?xml version="1.0" encoding="UTF-8"?>
-<kml xmlns="http://www.opengis.net/kml/2.2" xmlns:gx="http://www.google.com/kml/ext/2.2">
+<kml xmlns="http://earth.google.com/kml/2.2" xmlns:gx="http://www.google.com/kml/ext/2.2">
<Folder>
<PhotoOverlay>
- <rotation>12.455</rotation>
+ <rotation>12.4549999237061</rotation>
<ViewVolume>
+ <near>1000</near>
<leftFov>-60</leftFov>
<rightFov>60</rightFov>
<bottomFov>-45</bottomFov>
<topFov>45</topFov>
- <near>1000</near>
</ViewVolume>
<ImagePyramid>
<tileSize>219</tileSize>
@@ -17,7 +17,7 @@
<gridOrigin>lowerLeft</gridOrigin>
</ImagePyramid>
<Point>
- <coordinates>45.78665,0.6565</coordinates>
+ <coordinates>45.7866500000,0.6565000000</coordinates>
</Point>
<shape>sphere</shape>
</PhotoOverlay>
These are shortcomings of our string based comparison. To work around them until we have operator== everywhere available, do this:
- move the <near> tag on top of <leftFov>
- Choose some other value of <rotation> that is not affected by string/qreal conversion accuracy problems
- Use coordinate values with 10 digit precision
</pre>
<br />
<p>- Dennis Nienhüser</p>
<br />
<p>On February 26th, 2014, 10:32 a.m. UTC, Cruceru Calin-Cristian wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Marble.</div>
<div>By Cruceru Calin-Cristian.</div>
<p style="color: grey;"><i>Updated Feb. 26, 2014, 10:32 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=331467">331467</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
marble
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Extend KmlPhotoOverlayWriter.cpp to also write the following elements (and their children): - rotation - ViewVolume - ImagePyramid - shape
I fail the test with tests/TestGeoDataWriter so I'm waiting for review with what I'm doing wrong.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/lib/marble/geodata/writers/kml/KmlPhotoOverlayWriter.cpp <span style="color: grey">(5903654)</span></li>
<li>tests/TestGeoDataWriter.cpp <span style="color: grey">(b1acbc9)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/116059/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<ul>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2014/02/25/091ac6a9-8b67-434a-a678-d15a7d87f4c5__0002-Update.patch">The testing file</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>