[Merge] lp:~phablet-team/qt-halide/unit-tests into lp:qt-halide

Ugo Riboni ugo.riboni at canonical.com
Mon Mar 30 11:01:49 UTC 2015


6 to 14 all done except: 

> 7) set_packed is only necessary for the AOT generators, for JIT generators it
> is done dynamically

Since we are only testing AOT now, does this still apply ?
 
> 9) is there any way to have a test/check target and possibly a coverage target
> appearing in QtCreator?

Not that I know of, sorry. Zoltan seems to agree that there is no way to add targets there from cmake.
 
> 10) why not use compare or fuzzyCompare (e.g.compare(image.pixel(20, 20),
> Qt.rgba(255, 0, 0, 255)) instead of creating our own Util.checkColor?

First there is a documentation mismatch: http://doc.qt.io/qt-5/qml-qtqml-qt.html#rgba-method and http://doc.qt.io/qt-5/qml-qttest-testcase.html#grabImage-method disagree on whether Qt.rgba should use 0-255 or 0.0-1.0 values to define colors. 

And more importantly I tried using compare with Qt.color and it never worked reliably, so after wasting several hours I just decided to get done with it and use our own function. I will revisit this when I have time to dig a bit better, but does not seem like a priority ATM.
 
-- 
https://code.launchpad.net/~phablet-team/qt-halide/unit-tests/+merge/254064
Your team Ubuntu Phablet Team is subscribed to branch lp:~phablet-team/qt-halide/unit-tests.



More information about the Ubuntu-reviews mailing list