[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