KDiagram libs (KChart, KGantt) in KDE Review

[Adriaan de Groot]

On Monday 09 February 2015 01:50:03 Friedrich W. H. Kossebau wrote:
> Yes, nearly copy&paste: the copies of KDChart in Calligra & KMyMoney are
> older  (2.4.1, based on Qt4) versions, while the copy of KDChart in
> Massif-Visualizer matches the version of the KChart lib in KDiagram.

I've tried compiling the code on FreeBSD 10.1-RELEASE with Clang 3.4.1 (I'm 
assuming that's a supported compiler -- on techbase, searching for "supported 
compiler" doesn't give me much compatibility information newer than KDE 4.2).

 - I need to add /usr/local/include to the include search path; this is not 
kdiagram specific, but seems to be a general Qt5 issue on FreeBSD.
 
 - TestDrawIntoPainter seems to hang; after 2 min at 100% CPU I killed it. I 
ran it separately by hand and get output about missing OpenGL drivers (which 
is true, I'm building kdiagram in a restricted environment; I didn't 
originally expect to need to have DBUS running to be able to do the tests 
either) and debug output like:
    QDEBUG : TestDrawIntoPainter::testTest() Time for drawing pixmap :/test: 
53682 ms
  Is that test supposed to take so much longer (minutes) than the other tests 
(deciseconds)?

So from a it-compiles-and-the-tests-pass point of view on my platform, it 
looks good.

[ade]