<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/122153/">https://git.reviewboard.kde.org/r/122153/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 16th, 2016, 12:31 a.m. UTC, <b>Jarosław Staniek</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">@Friedrich What to do with this patch? Is it needed now or?</p></pre>
</blockquote>
<p>On January 21st, 2016, 10:12 a.m. UTC, <b>Friedrich W. H. Kossebau</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Guess so. Still missing proper sets of files/data to test it, so can only blindly commit it, but perhaps better than to ignore. Will do in the next days.</p></pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I submitted the patch last year. I first submitted it to kdab.com, where it became issue KDCH-1020. The Status there is CLOSED, and the Resolution is FIXED. The Assignee was KDAB staffer Andreas Hartmetz. Here is his final comment, dated 14/Jan/15:</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">“I've changed the h/v line layout items to only grow in the lengthwise direction and removed the comments. The comments didn't add anything of value. I've also changed "< 360" to "<= 360". The latter is more consistent with line 211 and also with the previous if condition, which is inclusive at the edges of the value range (0...360). It should not change the behavior at all, due to reversedOrder = false by default. Well, actually the whole if and dependent code is redundant, it's really just documentation.”</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Andreas said, “I've also changed "< 360" to "<= 360".”
I (Stephen) first suggested to kdab.com that "< 0" be changed to "< 360". I then slightly modified my suggestion to "<= 360". Here are the original and changed lines:
if ( ( angle >= 90 && angle < 180 ) || ( angle >= 270 && angle < 0 ) )
<em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">change to</em>:
if ( ( angle >= 90 && angle < 180 ) || ( angle >= 270 && angle <= 360 ) )</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Andreas mentioned line 211 in his comment. Here is the line:
} else if ( angle >= 270.0 && angle <= 360.0 ) {</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Their line 211 corresponds to line 209 in Calligra’s repository at
https://projects.kde.org/projects/calligra/calligra/repository/entry/3rdparty/kdchart/src/KDChartStockDiagram_p.cpp?rev=calligra%2F2.9</p></pre>
<br />
<p>- Stephen</p>
<br />
<p>On December 3rd, 2015, 7:58 a.m. UTC, Stephen Leibowitz wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Calligra and Jarosław Staniek.</div>
<div>By Stephen Leibowitz.</div>
<p style="color: grey;"><i>Updated Dec. 3, 2015, 7:58 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
calligra
</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">These patches are also being made at kdab.com. Those who have a KDAB account can see the discussion in “Suggested Changes to KD Chart” at
https://quality.kdab.com/browse/KDCH-1020</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Calligra’s kdchart and kdgantt files are out-of-date, even with the patches from the above paragraph. For example, “Compiler warnings” at
http://mail.kde.org/pipermail/calligra-devel/2015-January/012762.html
mentioned an error in KDChartPieDiagram.cpp. But the error is in a private function that was removed from the latest version (2.5.1) of KD Chart. KDAB will not patch previous versions. See “PieDiagram::drawPieSurface” at
https://quality.kdab.com/browse/KDCH-1023</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">KDAB makes available source code for the latest and earlier versions of its KD Chart and other GPL licensed software at http://docs.kdab.com/</p></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>3rdparty/kdchart/src/KDChartLayoutItems.cpp <span style="color: grey">(095d2cd)</span></li>
<li>3rdparty/kdchart/src/KDChartStockDiagram_p.cpp <span style="color: grey">(d8636d7)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/122153/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>