diff options
author | Jason McDonald <jason.mcdonald@nokia.com> | 2011-10-07 10:55:06 +1000 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2011-10-07 04:30:30 +0200 |
commit | b4de0b17211c58fdecece48d065e8b2dac53711c (patch) | |
tree | 2cefda5796193a770b08e2d4ff42da8c1a96b8f7 /tests/auto/gui | |
parent | 924d810dbdcd5b5b0fa860922b2487ea9062d002 (diff) |
No need to return after QSKIP.
QSKIP already causes the test function that calls it to return, so the
returns removed by this commit were unreachable.
Change-Id: I1fa2f3a3271927d8a600b02d8b31bd81db9146b1
Reviewed-on: http://codereview.qt-project.org/6188
Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
Diffstat (limited to 'tests/auto/gui')
-rw-r--r-- | tests/auto/gui/text/qfontdatabase/tst_qfontdatabase.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/tests/auto/gui/text/qfontdatabase/tst_qfontdatabase.cpp b/tests/auto/gui/text/qfontdatabase/tst_qfontdatabase.cpp index fe37c77dd5..af6d822666 100644 --- a/tests/auto/gui/text/qfontdatabase/tst_qfontdatabase.cpp +++ b/tests/auto/gui/text/qfontdatabase/tst_qfontdatabase.cpp @@ -253,10 +253,8 @@ void tst_QFontDatabase::addAppFont() QCOMPARE(fontDbChangedSpy.count(), 1); // addApplicationFont is supported on Mac, don't skip the test if it breaks. #ifndef Q_WS_MAC - if (id == -1) { + if (id == -1) QSKIP("Skip the test since app fonts are not supported on this system", SkipSingle); - return; - } #endif const QStringList addedFamilies = QFontDatabase::applicationFontFamilies(id); |