From e38064f698b03a3357943ed4c8e0d25fa6695418 Mon Sep 17 00:00:00 2001 From: Marc Mutz Date: Fri, 5 Aug 2016 23:05:40 +0300 Subject: tst_utf8: use QScopedPointer, not QSharedPointer There's no sharing, and the use of QSharedPointer(T*) triggers my tree's static analyzer. Easiest fix is to port to QScopedPointer, which is the correct smart pointer to begin with. Change-Id: I105c1a334c3d6712a475600c8394b0bebc420677 Reviewed-by: Lars Knoll Reviewed-by: Thiago Macieira --- tests/auto/corelib/codecs/utf8/tst_utf8.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'tests/auto/corelib/codecs') diff --git a/tests/auto/corelib/codecs/utf8/tst_utf8.cpp b/tests/auto/corelib/codecs/utf8/tst_utf8.cpp index 16265c6187..5666726a8c 100644 --- a/tests/auto/corelib/codecs/utf8/tst_utf8.cpp +++ b/tests/auto/corelib/codecs/utf8/tst_utf8.cpp @@ -29,7 +29,7 @@ #include #include -#include +#include static const char utf8bom[] = "\xEF\xBB\xBF"; @@ -180,7 +180,7 @@ void tst_Utf8::charByChar() { // from utf16 to utf8 char by char: - QSharedPointer encoder = QSharedPointer(codec->makeEncoder()); + const QScopedPointer encoder(codec->makeEncoder()); QByteArray encoded; for (int i = 0; i < utf16.length(); ++i) { @@ -194,7 +194,7 @@ void tst_Utf8::charByChar() } { // from utf8 to utf16 char by char: - QSharedPointer decoder = QSharedPointer(codec->makeDecoder()); + const QScopedPointer decoder(codec->makeDecoder()); QString decoded; for (int i = 0; i < utf8.length(); ++i) { @@ -219,7 +219,7 @@ void tst_Utf8::invalidUtf8() QFETCH(QByteArray, utf8); QFETCH_GLOBAL(bool, useLocale); - QSharedPointer decoder = QSharedPointer(codec->makeDecoder()); + const QScopedPointer decoder(codec->makeDecoder()); decoder->toUnicode(utf8); // Only enforce correctness on our UTF-8 decoder @@ -280,7 +280,7 @@ void tst_Utf8::nonCharacters() QFETCH(QString, utf16); QFETCH_GLOBAL(bool, useLocale); - QSharedPointer decoder = QSharedPointer(codec->makeDecoder()); + const QScopedPointer decoder(codec->makeDecoder()); decoder->toUnicode(utf8); // Only enforce correctness on our UTF-8 decoder @@ -289,7 +289,7 @@ void tst_Utf8::nonCharacters() else if (decoder->hasFailure()) qWarning("System codec reports failure when it shouldn't. Should report bug upstream."); - QSharedPointer encoder(codec->makeEncoder()); + const QScopedPointer encoder(codec->makeEncoder()); encoder->fromUnicode(utf16); if (!useLocale) QVERIFY(!encoder->hasFailure()); -- cgit v1.2.3