Re: Review Request 122153: KD Chart

2016-01-29 Thread Stephen Leibowitz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122153/ --- (Updated Jan. 29, 2016, 7:29 p.m.) Status -- This change has been ma

Re: Review Request 122153: KD Chart

2016-01-21 Thread Stephen Leibowitz
> On Jan. 16, 2016, 12:31 a.m., Jarosław Staniek wrote: > > @Friedrich What to do with this patch? Is it needed now or? > > Friedrich W. H. Kossebau wrote: > Guess so. Still missing proper sets of files/data to test it, so can only > blindly commit it, but perhaps better than to ignore. Wil

Re: Review Request 122153: KD Chart

2016-01-21 Thread Friedrich W. H. Kossebau
> On Jan. 16, 2016, 12:31 a.m., Jarosław Staniek wrote: > > @Friedrich What to do with this patch? Is it needed now or? 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. - Friedrich W. H.

Re: Review Request 122153: KD Chart

2016-01-15 Thread Jarosław Staniek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122153/#review91168 --- @Friedrich What to do with this patch? Is it needed now or?

Re: Review Request 122153: KD Chart

2015-02-25 Thread Stephen Leibowitz
> On Feb. 14, 2015, 5:32 p.m., Friedrich W. H. Kossebau wrote: > > 3rdparty/kdchart/src/KDChartStockDiagram_p.cpp, line 293 > > > > > > `<= 360` or `< 360`? The KDChart 2.5.1 dump used for KDiagram has `< > > 36

Re: Review Request 122153: KD Chart

2015-02-14 Thread Friedrich W. H. Kossebau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122153/#review76031 --- Ship it! I do not have a KDAB account. But I see those change

Review Request 122153: KD Chart

2015-01-19 Thread Stephen Leibowitz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122153/ --- Review request for Calligra, Boudewijn Rempt and Jarosław Staniek. Reposi